diff options
| author | Byron Jones <bjones@mozilla.com> | 2013-03-25 15:11:54 +0100 |
|---|---|---|
| committer | Byron Jones <bjones@mozilla.com> | 2013-03-25 15:11:54 +0100 |
| commit | 9adb2c722a30919e530d13d406d4727351255f40 (patch) | |
| tree | 0e641f5baabd8b71542f5fe453db1db9354ff0b5 | |
| parent | 58a67dbb625f0c17ba32e1a7965fe68f6ec5e8b0 (diff) | |
| download | bugzilla-9adb2c722a30919e530d13d406d4727351255f40.tar.gz bugzilla-9adb2c722a30919e530d13d406d4727351255f40.tar.xz | |
Bug 854386: needinfo doesn't work if user confirmation is required
| -rw-r--r-- | extensions/Needinfo/Extension.pm | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/extensions/Needinfo/Extension.pm b/extensions/Needinfo/Extension.pm index 255e08098..e88e764b2 100644 --- a/extensions/Needinfo/Extension.pm +++ b/extensions/Needinfo/Extension.pm @@ -57,8 +57,7 @@ sub bug_start_of_update { my $cgi = Bugzilla->cgi; my $params = Bugzilla->input_params; - my $needinfo = delete $params->{needinfo}; - if ($user->in_group('canconfirm') && $needinfo) { + if ($user->in_group('canconfirm') && $params->{needinfo}) { # do a match if applicable Bugzilla::User::match_field({ 'needinfo_from' => { 'type' => 'single' } @@ -70,6 +69,7 @@ sub bug_start_of_update { $params->{needinfo_done} = 1; Bugzilla->input_params($params); + my $needinfo = delete $params->{needinfo}; my $needinfo_from = delete $params->{needinfo_from}; my $needinfo_role = delete $params->{needinfo_role}; my $is_private = $params->{'comment_is_private'}; |