When receiving my newsletter, the title was "C'est chaud, c'est piquant!....." but should be "C'est chaud, c'est piquant! ...."

The online version is here http://www.salsamontreal.com/cest_chaud_cest_piquant_la_publication_grat...

This was working correctly in the 4.7 version (yeah, I know it's far)

Comments

alexandreracine’s picture

Title: Title in email is not correct » The newsletter title in the received email is not correct

Changed title to something more clear.

alexandreracine’s picture

StatusFileSize
new29.75 KB

Ho wow! Looking this issue back, drupal.org just translated back the caracters correctly, LOL. So here is a screenshot of the newsletter in my gmail account.

sutharsan’s picture

Status: Active » Postponed (maintainer needs more info)
StatusFileSize
new27.22 KB

Attached is my screen shot of a plain text newsletter send with the latest 6.x-1.x-dev release. Both in Gmail and mac's mail title and body look good. I can not reproduce you error.
Is it mime mail related, is it server related?

gdoteof’s picture

Ampersands even in your screen shot are being converted. This is happening both with titles and "from" names.

sutharsan’s picture

Status: Postponed (maintainer needs more info) » Active

You are right, the '&' was a last minute addition to the test and I overlooked it in the result. The html characters are the result of security measures (check_plain()) on the title. I will investigate if we can remove this.

sutharsan’s picture

Status: Active » Needs review
StatusFileSize
new1.42 KB

The attached patch should fix the issue. But I will wait committing until it gets 'security clearance'.

gdoteof’s picture

The '&' is still being converted on the 'From' name, though it is fixed in the title/subject with that patch, thanks

sutharsan’s picture

Title: The newsletter title in the received email is not correct » Raw HTML characters in newsletter title and from address
StatusFileSize
new1.7 KB

Patch updated for 'From' name.
Title changed to better describe the problem.

sutharsan’s picture

Status: Needs review » Fixed

I have gotten two confirmations that title and name are treated as plain text by mail clients. Therefore I conclude that it is safe to remove the check_plain() from both title and from-name.

Patch committed.

alexandreracine’s picture

Mmmm, I'll have to wait to test since I can't send email with my test machine. Sounds good :)

Status: Fixed » Closed (fixed)

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