CiviCRM Community Forums (archive)

Greetings,

The help text for the 'Email Delivery Settings' section in the 'Report Settings' includes the text, 'Multiple email addresses should be separated by commas.'  When I enter multiple bare addresses in the 'To' field separated by commas the resulting To: header in the outbound email is very poorly formed.

For example, The entry:

xxxx.yyy@foo.bar.com,brown@stat.duke.edu
results in a To: header that looks like:

To: <"xxxx.yyy@foo.bar.com, brown"@stat.duke.edu>
Somehow, CiviCRM manages to deliver the messages to the correct addresses in the SMTP transaction, but that To: header is very very broken.

I traced through the code and it looks like the mailReport function in CRM/Report/Utils/Report.php calls CRM_Utils_Mail::send to send the report email, but that the 'send' function is expecting a _single_ email address in the 'toEmail' variable.  The relevant bits of code are:

from Report.php 'mailReport' function:

        $params                = array( );
        $params['groupName'  ] = 'Report Email Sender';
        $params['from'       ] = '"' . $domainEmailName . '" <' . $domainEmailAddress . '>';
        $params['toName'     ] = ""; //$domainEmailName;
        $params['toEmail'    ] = CRM_Utils_Array::value( 'email_to', $instanceInfo );
        $params['cc'         ] = CRM_Utils_Array::value( 'email_cc', $instanceInfo );
        $params['subject'    ] = CRM_Utils_Array::value( 'email_subject', $instanceInfo );
        if ( !CRM_Utils_Array::value( 'attachments', $instanceInfo ) ) {
            $instanceInfo['attachments'] = array();
        }
        $params['attachments'] = array_merge(CRM_Utils_Array::value( 'attachments', $instanceInfo ), $attachments);
        $params['text'       ] = '';
        $params['html'       ] = $fileContent;

        require_once "CRM/Utils/Mail.php";
        return CRM_Utils_Mail::send( $params );

toName is blank and toEmail is set to the value of the 'To' field in the Report settings.

Then, in the CRM_Utils_Mail::send function:

        $headers['To']                        = self::formatRFC822Email( CRM_Utils_Array::value( 'toName', $params ),
                                                                         CRM_Utils_Array::value( 'toEmail', $params ),
                                                                         false );

This munges the toName and toEmail variables together into a supposedly  RFC822 legal email address, which in this case fails.

Either the help text in the Report Settings section needs to be changed or the mailReport and/or send functions need to be modified to support multiple 'To' addresses.

Thanks,
  --[Lance]

Issue created.  http://issues.civicrm.org/jira/browse/CRM-10392

--[Lance]