diff options
author | Alexander Tereschenko <aleksandr.v.tereschenko@linux.intel.com> | 2012-12-17 23:38:24 +0100 |
---|---|---|
committer | Frédéric Buclin <LpSolit@gmail.com> | 2012-12-17 23:38:24 +0100 |
commit | 64e21e0be7743a07841682c69f062b962550048e (patch) | |
tree | 1bc5035e7b2807306f45893da5fd903c2680f5b5 /Bugzilla/Bug.pm | |
parent | db3413beb4cc8f6aef95a47fed231df17f8f0063 (diff) | |
download | bugs-64e21e0be7743a07841682c69f062b962550048e.tar bugs-64e21e0be7743a07841682c69f062b962550048e.tar.gz bugs-64e21e0be7743a07841682c69f062b962550048e.tar.bz2 bugs-64e21e0be7743a07841682c69f062b962550048e.tar.xz bugs-64e21e0be7743a07841682c69f062b962550048e.zip |
Bug 818890: Bugzilla doesn't obey the "Comment required on status transition" for {Start}-> transition (for new bugs)
r/a=LpSolit
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 f8b4d2142..efad66f62 100644 --- a/Bugzilla/Bug.pm +++ b/Bugzilla/Bug.pm @@ -1323,11 +1323,12 @@ sub _check_bug_status { } # Check if a comment is required for this change. - if ($new_status->comment_required_on_change_from($old_status) && !$comment) + if ($new_status->comment_required_on_change_from($old_status) + && !$comment->{'thetext'}) { ThrowUserError('comment_required', - { old => $old_status->name, new => $new_status->name, - field => 'bug_status' }); + { old => $old_status ? $old_status->name : undef, + new => $new_status->name, field => 'bug_status' }); } if (ref $invocant |