[freeside-commits] branch master updated. bd4e387da29dc3dad44605572d531ab4f7ab4f4d

[Mark Wells]

The branch, master has been updated
       via  bd4e387da29dc3dad44605572d531ab4f7ab4f4d (commit)
      from  4e95cbcc7a876017487f098a06b957e79c0acf77 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit bd4e387da29dc3dad44605572d531ab4f7ab4f4d
Author: Mark Wells <mark at freeside.biz>
Date:   Fri Apr 11 08:20:26 2014 -0700

    fix accidental resetting of TimeWorked value, #28459

diff --git a/rt/lib/RT/Interface/Web_Vendor.pm b/rt/lib/RT/Interface/Web_Vendor.pm
index 245df12..c9bff6f 100644
--- a/rt/lib/RT/Interface/Web_Vendor.pm
+++ b/rt/lib/RT/Interface/Web_Vendor.pm
@@ -288,8 +288,10 @@ sub ProcessTicketBasics {
     # fields to empty strings, but internally change the values 
     # to zero.  This is sloppy and causes some problems.
     foreach my $field (qw(TimeWorked TimeEstimated TimeLeft)) {
-      $ARGSRef->{$field} =~ s/\s//g;
-      $ARGSRef->{$field} ||= 0;
+      if (exists $ARGSRef->{$field}) {
+        $ARGSRef->{$field} =~ s/\s//g;
+        $ARGSRef->{$field} ||= 0;
+      }
     }
 
     my @results = UpdateRecordObject(

-----------------------------------------------------------------------

Summary of changes:
 rt/lib/RT/Interface/Web_Vendor.pm |    6 ++++--
 1 files changed, 4 insertions(+), 2 deletions(-)