diff options
| author | Byron Jones <glob@mozilla.com> | 2014-06-18 05:46:04 +0200 |
|---|---|---|
| committer | Byron Jones <glob@mozilla.com> | 2014-06-18 05:46:04 +0200 |
| commit | 324f00d76a2f33421c047d4e86a17bb6fa34b416 (patch) | |
| tree | 8ef92e36fd83fde0eb962be42bcb78dc64368081 | |
| parent | c2fef4494755f341a1d17fc61f208b3cb650da1d (diff) | |
| download | bugzilla-324f00d76a2f33421c047d4e86a17bb6fa34b416.tar.gz bugzilla-324f00d76a2f33421c047d4e86a17bb6fa34b416.tar.xz | |
Bug 1026862: updating a bug via an api or bulk bug update clears the mentor field
| -rw-r--r-- | extensions/Review/Extension.pm | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/extensions/Review/Extension.pm b/extensions/Review/Extension.pm index ac7cb7233..4fd32e3bd 100644 --- a/extensions/Review/Extension.pm +++ b/extensions/Review/Extension.pm @@ -178,7 +178,7 @@ sub object_end_of_create { if (_is_countable_flag($object)) { $self->_log_flag_state_activity($object, $object->status); } - elsif ($object->isa('Bugzilla::Bug')) { + elsif ($object->isa('Bugzilla::Bug') && exists Bugzilla->input_params->{bug_mentors}) { my ($new, $new_users) = _new_users_from_input('bug_mentors'); _update_users($object, [], $new_users); } @@ -222,7 +222,7 @@ sub object_end_of_update { _adjust_request_count($object, +1); } } - elsif ($object->isa('Bugzilla::Bug')) { + elsif ($object->isa('Bugzilla::Bug') && exists Bugzilla->input_params->{bug_mentors}) { my ($new, $new_mentors) = _new_users_from_input('bug_mentors'); my $old = join(", ", map { $_->login } @{ $old_object->mentors }); if ($old ne $new) { |