diff options
author | Byron Jones <glob@mozilla.com> | 2014-08-26 09:17:34 +0200 |
---|---|---|
committer | Byron Jones <glob@mozilla.com> | 2014-08-26 09:17:34 +0200 |
commit | 726d3c1b09a6563c2a8364204a68f5f05c41caa0 (patch) | |
tree | e92558083054f71a597ab0ef9ca4111a9bfd68bb | |
parent | fe5deaa737630825b6012487ee5bc583d3c17343 (diff) | |
download | bugzilla-726d3c1b09a6563c2a8364204a68f5f05c41caa0.tar.gz bugzilla-726d3c1b09a6563c2a8364204a68f5f05c41caa0.tar.xz |
Bug 1051655: fix error thrown when users without editbugs comment on
bugs
-rwxr-xr-x | post_bug.cgi | 1 | ||||
-rwxr-xr-x | process_bug.cgi | 1 |
2 files changed, 2 insertions, 0 deletions
diff --git a/post_bug.cgi b/post_bug.cgi index ccdda3e2d..21b621acc 100755 --- a/post_bug.cgi +++ b/post_bug.cgi @@ -159,6 +159,7 @@ foreach my $field (@multi_selects) { # instead of $cgi->param to ensure we get the correct value. foreach my $field (keys %$user_match_fields) { next if exists $bug_params{$field}; + next unless $cgi->should_set($field); $bug_params{$field} = Bugzilla->input_params->{$field} // []; } diff --git a/process_bug.cgi b/process_bug.cgi index 39823f88e..75c59b084 100755 --- a/process_bug.cgi +++ b/process_bug.cgi @@ -379,6 +379,7 @@ foreach my $field (@custom_fields) { # instead of $cgi->param to ensure we get the correct value. foreach my $field (keys %$user_match_fields) { next if exists $set_all_fields{$field}; + next unless should_set($field, 1); $set_all_fields{$field} = Bugzilla->input_params->{$field} // []; } |