[Rt-commit] r7603 - in rt/branches/3.7-EXPERIMENTAL: .

ruz at bestpractical.com ruz at bestpractical.com
Sat Apr 21 11:43:58 EDT 2007


Author: ruz
Date: Sat Apr 21 11:43:56 2007
New Revision: 7603

Modified:
   rt/branches/3.7-EXPERIMENTAL/   (props changed)
   rt/branches/3.7-EXPERIMENTAL/lib/RT/Action/SendEmail.pm

Log:
 r5045 at cubic-pc:  cubic | 2007-04-21 18:45:37 +0400
 * SendEmail return -1 when there is no recipients


Modified: rt/branches/3.7-EXPERIMENTAL/lib/RT/Action/SendEmail.pm
==============================================================================
--- rt/branches/3.7-EXPERIMENTAL/lib/RT/Action/SendEmail.pm	(original)
+++ rt/branches/3.7-EXPERIMENTAL/lib/RT/Action/SendEmail.pm	Sat Apr 21 11:43:56 2007
@@ -241,20 +241,12 @@
         . $self->ScripObj->id . " "
         . ($self->ScripObj->Description || '') );
 
-    #If we don't have any recipients to send to, don't send a message;
-    unless ( $MIMEObj->head->get('To')
-        || $MIMEObj->head->get('Cc')
-        || $MIMEObj->head->get('Bcc') )
-    {
-        $RT::Logger->info( $msgid . " No recipients found. Not sending.\n" );
-        return (-1);
-    }
-
-    return(0) unless RT::Interface::Email::SendEmail(
+    my $status = RT::Interface::Email::SendEmail(
         Entity => $MIMEObj,
         Ticket => $self->TicketObj,
         Transaction => $self->TransactionObj,
     );
+    return $status unless $status > 0;
 
     my $success = $msgid . " sent ";
     foreach( qw(To Cc Bcc) ) {


More information about the Rt-commit mailing list