[Rt-commit] [svn] r1239 - rt/branches/3.2-RELEASE/lib/RT/Action

[alexmv at pallas.eruditorum.org]

Author: alexmv
Date: Wed Jul 14 11:15:20 2004
New Revision: 1239

Modified:
   rt/branches/3.2-RELEASE/lib/RT/Action/SendEmail.pm
Log:
 * Fix error of "No recipients" if FriendlyToLine is set, but all
   recipients are masked


Modified: rt/branches/3.2-RELEASE/lib/RT/Action/SendEmail.pm
==============================================================================
--- rt/branches/3.2-RELEASE/lib/RT/Action/SendEmail.pm	(original)
+++ rt/branches/3.2-RELEASE/lib/RT/Action/SendEmail.pm	Wed Jul 14 11:15:20 2004
@@ -116,10 +116,11 @@
       if ( !$MIMEObj->head->get('Bcc') && $self->{'Bcc'} && @{ $self->{'Bcc'} } );
 
     # PseudoTo	(fake to headers) shouldn't get matched for message recipients.
-    # If we don't have any 'To' header, drop in the pseudo-to header.
+    # If we don't have any 'To' header (but do have other reciepients), drop in
+    # the pseudo-to header.
     $self->SetHeader( 'To', join ( ', ', @{ $self->{'PseudoTo'} } ) )
       if ( $self->{'PseudoTo'} && ( @{ $self->{'PseudoTo'} } )
-        and ( !$MIMEObj->head->get('To') ) );
+        and ( !$MIMEObj->head->get('To') ) ) and ( $MIMEObj->head->get('Cc') or $MIMEObj->head->get('Bcc'));
 
     # We should never have to set the MIME-Version header
     $self->SetHeader( 'MIME-Version', '1.0' );
@@ -273,6 +274,7 @@
 # }}}
 
 # {{{ AddAttachments 
+
 =head2 AddAttachments
 
 Takes any attachments to this transaction and attaches them to the message
@@ -325,6 +327,7 @@
 # }}}
 
 # {{{ RecordOutgoingMailTransaction
+
 =head2 RecordOutgoingMailTransaction MIMEObj
 
 Record a transaction in RT with this outgoing message for future record-keeping purposes