[Bps-public-commit] r10828 - RT-Extension-SLA/lib/RT/Action

ruz at bestpractical.com ruz at bestpractical.com
Thu Feb 14 01:50:05 EST 2008


Author: ruz
Date: Thu Feb 14 01:50:05 2008
New Revision: 10828

Modified:
   RT-Extension-SLA/lib/RT/Action/SLA_SetDue.pm

Log:
* we check that level is set in Prepare code and return error otherwise,
  so we don't need to check it in commit code

Modified: RT-Extension-SLA/lib/RT/Action/SLA_SetDue.pm
==============================================================================
--- RT-Extension-SLA/lib/RT/Action/SLA_SetDue.pm	(original)
+++ RT-Extension-SLA/lib/RT/Action/SLA_SetDue.pm	Thu Feb 14 01:50:05 2008
@@ -34,14 +34,8 @@
     my $self = shift;
 
     my $ticket = $self->TicketObj;
-
-    my $level = $ticket->FirstCustomFieldValue('SLA');
-    unless ( $level ) {
-        $RT::Logger->debug('Ticket #'. $ticket->id .' has no service level defined');
-        return 1;
-    }
-
     my $txn = $self->TransactionObj;
+    my $level = $ticket->FirstCustomFieldValue('SLA');
 
     my $last_reply = $self->LastRequestorsEffectiveAct;
     $RT::Logger->debug('Last effective requestors\' reply to ticket #'. $ticket->id .' is txn #'. $last_reply->id )



More information about the Bps-public-commit mailing list