From 5f030a6f4394c45fdf58029d2d4a509d8289424b Mon Sep 17 00:00:00 2001 From: "lpsolit%gmail.com" <> Date: Sat, 28 Jul 2007 17:47:55 +0000 Subject: =?UTF-8?q?Bug=20389916:=20The=20"remaining=5Ftime=5Fzeroed"=20mes?= =?UTF-8?q?sage=20is=20displayed=20even=20if=20none=20of=20the=20bugs=20be?= =?UTF-8?q?ing=20changed=20has=20its=20remaining=20time=20>=200=20-=20Patc?= =?UTF-8?q?h=20by=20Fr=C3=83=C2=A9d=C3=83=C2=A9ric=20Buclin=20=20r=3Dmkanat=20a=3DLpSolit?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- Bugzilla/Bug.pm | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/Bugzilla/Bug.pm b/Bugzilla/Bug.pm index 4793c87da..5f37e9ff4 100755 --- a/Bugzilla/Bug.pm +++ b/Bugzilla/Bug.pm @@ -2003,8 +2003,13 @@ sub check_status_change_triggers { $vars->{DuplicateUserConfirm} = 1; # DUPLICATE bugs should have no time remaining. - $_->_zero_remaining_time() foreach @$bugs; - $vars->{'message'} = "remaining_time_zeroed"; + foreach my $bug (@$bugs) { + # Note that 0.00 is *true* for Perl! + next unless ($bug->remaining_time > 0); + $bug->_zero_remaining_time; + $vars->{'message'} = "remaining_time_zeroed" + if Bugzilla->user->in_group(Bugzilla->params->{'timetrackinggroup'}); + } } elsif ($action eq 'change_resolution' || !is_open_state($action)) { # don't resolve as fixed while still unresolved blocking bugs @@ -2038,8 +2043,11 @@ sub check_status_change_triggers { if ($action ne 'change_resolution') { foreach my $b (@$bugs) { if ($b->bug_status ne $action) { + # Note that 0.00 is *true* for Perl! + next unless ($b->remaining_time > 0); $b->_zero_remaining_time; - $vars->{'message'} = "remaining_time_zeroed"; + $vars->{'message'} = "remaining_time_zeroed" + if Bugzilla->user->in_group(Bugzilla->params->{'timetrackinggroup'}); } } } -- cgit v1.2.1