Hello!

I have a site called Tim's Diary. Whenever Comment Mail sends me an email notifying me of a new comment, the subject reads as follows:

[Tim& #039;s Diary] New Comment posted on (I had to insert a space between the & and the # otherwise Drupal kept rendering it correctly.)

Is it possible to make it so that these characters are parsed and mailed correctly in the subject field?

Regards,
Tim

CommentFileSizeAuthor
#1 273784_commentmail_subject_has_htmlentities.patch933 bytesmaartenvg

Comments

maartenvg’s picture

maartenvg commented
Version: 5.x-1.0-beta » master
Status: Active » Needs review
StatusFileSize
new 273784_commentmail_subject_has_htmlentities.patch933 bytes

Attached patch fixes this.

There were 2 problems with that line:
- uses '@' placeholders instead of '!' placeholders, this will perform check_plain on the replacements, which is not wanted in a e-mail subject.
- @site content was wrapped in a additional check_plain(), which is overkill because the '@' already does that and unnecessary because of what I said above.

maartenvg’s picture

maartenvg commented
Version: master » 5.x-1.x-dev
Status: Needs review » Fixed

Fixed this in CVS and will find its way into the next version of Comment mail.

Thanks for reporting this!

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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