[Rt-commit] r12758 - in rt/branches/3.8-TESTING: .

[elacour at bestpractical.com]

Author: elacour
Date: Fri May 30 05:43:41 2008
New Revision: 12758

Modified:
   rt/branches/3.8-TESTING/   (props changed)
   rt/branches/3.8-TESTING/lib/RT/Ticket_Overlay.pm

Log:
 r9024 at datura:  manu | 2008-05-30 11:43:06 +0200
 Don't block the queue ticket move if a reminder move failed, just drop an error
 in the log.


Modified: rt/branches/3.8-TESTING/lib/RT/Ticket_Overlay.pm
==============================================================================
--- rt/branches/3.8-TESTING/lib/RT/Ticket_Overlay.pm	(original)
+++ rt/branches/3.8-TESTING/lib/RT/Ticket_Overlay.pm	Fri May 30 05:43:41 2008
@@ -1706,8 +1706,8 @@
     # On queue change, change queue for reminders too
     my $reminder_collection = $self->Reminders->Collection;
     while ( my $reminder = $reminder_collection->Next ) {
-        my ($val, $msg) = $reminder->SetQueue($NewQueue);
-        return ( 0, $self->loc("Queue change failed for reminder [_1]: [_2]", $reminder->Id ,$msg) ) unless $val;
+        my ($status, $msg) = $reminder->SetQueue($NewQueue);
+        $RT::Logger->error('Queue change failed for reminder #' . $reminder->Id . ': ' . $msg) unless $status;
     }
 
     return ( $self->_Set( Field => 'Queue', Value => $NewQueueObj->Id() ) );