Ascending order - Click to sort in descending order
Hide
lobo Donald A. Lobo added a comment -

can u please investigate and submit a patch if needed. Can u also check other email programs

Show
lobo Donald A. Lobo added a comment - can u please investigate and submit a patch if needed. Can u also check other email programs
Hide
chrism Chris Mott added a comment -

ok, i can look further into this one

Show
chrism Chris Mott added a comment - ok, i can look further into this one
Show
lobo Donald A. Lobo added a comment - check: http://forums.codewalkers.com/pear-packages-47/email-bcc-header-confusion-840705.html http://pear.php.net/bugs/bug.php?id=8047
Hide
chrism Chris Mott added a comment -

Here is the fix:

It's actually quite simple as far as I can tell and not related to a PEAR shortcoming.

CRM/Utils/Mail.php near Line 140 (remove the BCC line)

$headers['To'] = self::encodeAddressHeader($toDisplayName, $toEmail);
$headers['Cc'] = $cc;

  • $headers['Bcc'] = $bcc;
    $headers['Subject'] = self::encodeSubjectHeader($subject);
    $headers['Content-Type'] = 'text/plain; charset=utf-8';
    $headers['Content-Disposition'] = 'inline';

The Bcc field does not need to be in the header. The recipients list is separately generated including the 'Bcc' addresses, so this header is just used to generate the text for the header of the message,

I'm usually nervous about changes seem too simple, since it usually means I didn't understand something. However I have tested on both SMTP and Sendmail, on Linux Debian Etch server, with Mail.app mail program.

Show
chrism Chris Mott added a comment - Here is the fix: It's actually quite simple as far as I can tell and not related to a PEAR shortcoming. CRM/Utils/Mail.php near Line 140 (remove the BCC line) $headers ['To'] = self::encodeAddressHeader($toDisplayName, $toEmail); $headers ['Cc'] = $cc; $headers ['Bcc'] = $bcc; $headers ['Subject'] = self::encodeSubjectHeader($subject); $headers ['Content-Type'] = 'text/plain; charset=utf-8'; $headers ['Content-Disposition'] = 'inline'; The Bcc field does not need to be in the header. The recipients list is separately generated including the 'Bcc' addresses, so this header is just used to generate the text for the header of the message, I'm usually nervous about changes seem too simple, since it usually means I didn't understand something. However I have tested on both SMTP and Sendmail, on Linux Debian Etch server, with Mail.app mail program.
Hide
sunil Sunil Pawar added a comment -

Verified in r18824.

Show
sunil Sunil Pawar added a comment - Verified in r18824.
Hide
kurund Kurund Jalmi added a comment -

Batch move to verification

Show
kurund Kurund Jalmi added a comment - Batch move to verification
Hide
neha.saraph Neha Kulkarni added a comment -

verified in rev 23722

Show
neha.saraph Neha Kulkarni added a comment - verified in rev 23722
Hide
nganivet Nicolas Ganivet added a comment -

Reopening because this bug was never fixed: it is still in 4.6 and 4.7. Full fix coming.

Show
nganivet Nicolas Ganivet added a comment - Reopening because this bug was never fixed: it is still in 4.6 and 4.7. Full fix coming.
Hide
nganivet Nicolas Ganivet added a comment -

Added links to similar bugs. It has always been difficult to fix since the behavior is different based on which method you are using to send emails: the PEAR library requires different arguments for the Mail_mail mailer as for all other mailers. See code comments in submitted fix.

Show
nganivet Nicolas Ganivet added a comment - Added links to similar bugs. It has always been difficult to fix since the behavior is different based on which method you are using to send emails: the PEAR library requires different arguments for the Mail_mail mailer as for all other mailers. See code comments in submitted fix.