diff options
| author | Byron Jones <glob@mozilla.com> | 2015-05-05 16:31:21 +0200 |
|---|---|---|
| committer | Byron Jones <glob@mozilla.com> | 2015-05-05 16:31:21 +0200 |
| commit | 27db7bb9aabe7a411d701ffbd9989e1ef11a9651 (patch) | |
| tree | bb4dda04d134cafcf25dce8f21d399cfd4124e57 /Bugzilla/Bug.pm | |
| parent | 99fc3b3c3db2e1ffa00a94d5e88cefc29f98c49b (diff) | |
| download | bugzilla-27db7bb9aabe7a411d701ffbd9989e1ef11a9651.tar.gz bugzilla-27db7bb9aabe7a411d701ffbd9989e1ef11a9651.tar.xz | |
Bug 1161568: unable to move non-public bugs between products when groups need updating (hitting save on intermediate page shows the intermediate page again)
Diffstat (limited to 'Bugzilla/Bug.pm')
| -rw-r--r-- | Bugzilla/Bug.pm | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/Bugzilla/Bug.pm b/Bugzilla/Bug.pm index 78bb1dff4..9534b49e2 100644 --- a/Bugzilla/Bug.pm +++ b/Bugzilla/Bug.pm @@ -2696,8 +2696,10 @@ sub _set_product { if $params->{other_bugs}; @idlist = uniq @idlist; + my $verified = $params->{product_change_confirmed}; + # BMO - if everything is ok then we can skip the verfication page - if ($component_ok && $version_ok && $milestone_ok) { + if (!$verified && $component_ok && $version_ok && $milestone_ok) { $invalid_groups = $self->get_invalid_groups({ bug_ids => \@idlist, product => $product }); my $has_invalid_group = 0; foreach my $group (@$invalid_groups) { @@ -2706,7 +2708,7 @@ sub _set_product { last; } } - $params->{product_change_confirmed} = + $verified = # always check for invalid groups !$has_invalid_group # never skip verification when changing multiple bugs @@ -2715,7 +2717,6 @@ sub _set_product { && (!@{ $self->groups_in } || Bugzilla->input_params->{group_verified}); } - my $verified = $params->{product_change_confirmed}; my %vars; if (!$verified || !$component_ok || !$version_ok || !$milestone_ok) { $vars{defaults} = { |