From 9f774e4a6de07003873bb20a68989d8e7f6fd544 Mon Sep 17 00:00:00 2001 From: "mkanat%bugzilla.org" <> Date: Mon, 5 Nov 2007 01:17:57 +0000 Subject: Bug 401953: Move work_time validation from process_bug to Bugzilla::Bug Patch By Max Kanat-Alexander r=LpSolit, a=LpSolit --- Bugzilla/Bug.pm | 6 ++++++ 1 file changed, 6 insertions(+) (limited to 'Bugzilla/Bug.pm') diff --git a/Bugzilla/Bug.pm b/Bugzilla/Bug.pm index ba66e846f..7ffc3b52e 100755 --- a/Bugzilla/Bug.pm +++ b/Bugzilla/Bug.pm @@ -496,6 +496,10 @@ sub update { $dbh->do("INSERT INTO longdescs (bug_id, who, bug_when, $columns) VALUES (?,?,?,$qmarks)", undef, $self->bug_id, Bugzilla->user->id, $delta_ts, @values); + if ($comment->{work_time}) { + LogActivityEntry($self->id, "work_time", "", $comment->{work_time}, + Bugzilla->user->id, $delta_ts); + } } # Insert the values into the multiselect value tables @@ -1547,6 +1551,8 @@ sub add_comment { $params ||= {}; if (exists $params->{work_time}) { $params->{work_time} = $self->_check_work_time($params->{work_time}); + ThrowUserError('comment_required') + if $comment eq '' && $params->{work_time} != 0; } if (exists $params->{type}) { $params->{type} = $self->_check_comment_type($params->{type}); -- cgit v1.2.3-24-g4f1b