Even though it's incorrect, many modules do not create the body as an array.

Thus the hook_mail_alter() code that says 

    $message['body'][] = ".... signature added";

produces a fatal error.

This patch makes the code a bit more generous.

CommentFileSizeAuthor
#2 examples.body_as_string_653504_02.patch895 bytesrfay
examples.body_as_string.patch892 bytesrfay
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented

Just make sure to use the proper coding standard with if/else. :)

rfay’s picture

rfay
Primary language English
Location Palisade, CO, USA
CreditAttribution: rfay commented
FileSize
examples.body_as_string_653504_02.patch895 bytes

Oops. Thanks, @davereid. Here's the replacement.

Note that this fix should be made in D7 as well.

ilo’s picture

ilo CreditAttribution: ilo commented

Reviewed, patched, tested, and committed to DRUPAL-6--1. Thank you all!

ilo’s picture

ilo CreditAttribution: ilo commented
Version: 6.x-1.x-dev »
Status: Needs review » Fixed

and committed also for D7

Status: Fixed » Closed (fixed)

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