Skip to content

Commit

Permalink
don't reload the Tree/View pane after a Copy Branch operation; Copy B…
Browse files Browse the repository at this point in the history
…ranch doesn't mutate the asset tree being viewed so not needed
  • Loading branch information
scottwalters committed Sep 25, 2013
1 parent b3714e6 commit fbb1431
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion lib/WebGUI/AssetHelper/CopyBranch.pm
Original file line number Diff line number Diff line change
Expand Up @@ -163,6 +163,7 @@ sub copyBranch {

my $tree = WebGUI::ProgressTree->new($session, $assetIds );
my $maxValue = keys %{ $tree->flat };
$process->update(sub { $tree->json });
my $update_progress = sub {
# update the Fork's progress with how many are done
my $flat = $tree->flat;
Expand All @@ -172,7 +173,7 @@ sub copyBranch {
maxValue => $maxValue,
value => $current_value,
message => 'Copying...',
reload => 1, # this won't take effect until Fork.pm returns finished => 1 and this status is propogated to WebGUI.Admin.prototype.openForkDialog's callback
# reload => 1, # this won't take effect until Fork.pm returns finished => 1 and this status is propogated to WebGUI.Admin.prototype.openForkDialog's callback; Copy Branch is non-mutating of the Tree/View mode so we don't need reload
@_,
};
$info->{refresh} = 1 if $maxValue == $current_value;
Expand Down

0 comments on commit fbb1431

Please sign in to comment.