summaryrefslogtreecommitdiff
path: root/rt/lib/RT
diff options
context:
space:
mode:
authorMark Wells <mark@freeside.biz>2014-04-07 16:19:09 -0700
committerMark Wells <mark@freeside.biz>2014-04-07 16:19:09 -0700
commit2ea729e55651a288bbf3826c888f430806aab04c (patch)
treed51e0afe42ca79397b1e5feb1d1bbb1280111b24 /rt/lib/RT
parenta172aa2d549f2c62978fa1f6b6851a68ae0f2a4e (diff)
avoid creating Set TimeWorked transactions where new value is empty, #28459
Diffstat (limited to 'rt/lib/RT')
-rw-r--r--rt/lib/RT/Interface/Web_Vendor.pm8
1 files changed, 8 insertions, 0 deletions
diff --git a/rt/lib/RT/Interface/Web_Vendor.pm b/rt/lib/RT/Interface/Web_Vendor.pm
index e9c634661..245df1259 100644
--- a/rt/lib/RT/Interface/Web_Vendor.pm
+++ b/rt/lib/RT/Interface/Web_Vendor.pm
@@ -284,6 +284,14 @@ sub ProcessTicketBasics {
}
}
+ # RT core _will_ allow Set transactions that change these
+ # 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;
+ }
+
my @results = UpdateRecordObject(
AttributesRef => \@attribs,
Object => $TicketObj,