[Rt-commit] r11383 - rt/branches/3.6-RELEASE/html/Ticket

[ruz at bestpractical.com]

Author: ruz
Date: Wed Apr  2 04:39:47 2008
New Revision: 11383

Modified:
   rt/branches/3.6-RELEASE/html/Ticket/Display.html

Log:
* don't write an empty comment iif only change is update of
  TimeWorked field, instead just update the field.
  Thanks to Emmanuel Lacour

Modified: rt/branches/3.6-RELEASE/html/Ticket/Display.html
==============================================================================
--- rt/branches/3.6-RELEASE/html/Ticket/Display.html	(original)
+++ rt/branches/3.6-RELEASE/html/Ticket/Display.html	Wed Apr  2 04:39:47 2008
@@ -135,11 +135,10 @@
     }
 
     $ARGS{'UpdateContent'} =~ s/\r\n/\n/g if defined $ARGS{'UpdateContent'};
-    if ( $ARGS{'UpdateTimeWorked'} || (
-        defined $ARGS{'UpdateContent'}
+    if ( defined $ARGS{'UpdateContent'}
         && $ARGS{'UpdateContent'} ne ''
 	    && $ARGS{'UpdateContent'} ne "-- \n"
-	       . $session{'CurrentUser'}->UserObj->Signature ) )
+	       . $session{'CurrentUser'}->UserObj->Signature )
     {
 	    $ARGS{UpdateAttachments} = $session{'Attachments'};
 	    ProcessUpdateMessage(
@@ -148,6 +147,9 @@
 			TicketObj => $TicketObj,
         );
 	    delete $session{'Attachments'};
+    } elsif ( $ARGS{'UpdateTimeWorked'} ) {
+        # Add UpdateTimeWorked to TimeWorked (processed below with ProcessTicketBasics)
+        $ARGS{'TimeWorked'} = $TicketObj->TimeWorked + $ARGS{'UpdateTimeWorked'};
     }
     #Process status updates
     my @PeopleActions = ProcessTicketWatchers(ARGSRef => \%ARGS, TicketObj=>$TicketObj);