The code for handling cc and bcc in smtp.mail.inc uses the PHP split() function which is deprecated preventing it from working in Drupal 7. It should use preg_find() instead.

CommentFileSizeAuthor
#1 1112828-fixed-deprecated-split-function-0.patch881 bytesMakubeX
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

MakubeX’s picture

MakubeX CreditAttribution: MakubeX commented
FileSize
1112828-fixed-deprecated-split-function-0.patch881 bytes

Patch uploaded. This is my first one, I hope I did it right.

PS. In the issue description above I meant to say it should be preg_split(), not preg_find()... it's late, sorry.

seattlehimay’s picture

seattlehimay CreditAttribution: seattlehimay commented

I wanted to make sure this was logged, and found your report. FIx worked for me.

Vlad Vinnikov’s picture

Vlad Vinnikov CreditAttribution: Vlad Vinnikov commented

explode can be used to fix this problem http://stackoverflow.com/questions/7259974/how-to-cc-on-drupal-mail

josesanmartin’s picture

josesanmartin CreditAttribution: josesanmartin commented

I believe explode() should have a better performance.

josesanmartin’s picture

josesanmartin CreditAttribution: josesanmartin commented
Assigned: Unassigned » josesanmartin

This has been fixed on 7.x branch, moving the ticket to 6.x branch.

josesanmartin’s picture

josesanmartin CreditAttribution: josesanmartin commented
Version: 7.x-1.x-dev » 6.x-1.0-beta5
josesanmartin’s picture

josesanmartin CreditAttribution: josesanmartin commented
Status: Patch (to be ported) » Fixed

Fixed for D6 as well.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.