Use mime header encoding for all e-mail fields

[jksrecko]

Hello,

When we tested OJS (2.2.4.) for our journal, we noticed that non ASCII characters in people's names weren't displayed correctly virtually in any e-mail client.

I have a patch that mime encodes name part of all e-mail fields (From, To, Cc and Bcc), not just the Subject, using String::encode_mime_header function.

The patch has been tested on our production system (we'll register soon), and I'll be happy to provide it, as soon as my two posts are cleared. Although the patch is against 2.2.4 version, I think it should apply cleanly on the current head since as far as I can see classes/mail/Main.inc.php hasn't changed.

EDIT: Patch attached!

[asmecher]

Hi jksrecko,

Have you tried this with the current 2.3.x release? I believe this has been fixed since OJS 2.2.4. If you like, I can try to track down the relevant Bugzilla entry and linked patch.

Regards,
Alec Smecher
Public Knowledge Project Team

[jksrecko]

Hi,
When we updated to 2.3.3.2 this still wasn't working. The functionality was moved to lib/pkp though. So, here is the patch against the 2.3.3.2 version.

I've tried searching bugzilla for the patch you mentioned, but couldn't find it. Would you please track it down, so I can review it? Thanks.

[asmecher]

Hi jksrecko,

You're right -- this doesn't appear to have been fixed. (The issue I was recalling was related only to attachments.) However, are you using SMTP or a local MTA (i.e. is the "smtp" option enabled in your config.inc.php file)? I'm not sure whether this fix will work in both situations and might need some adjustment first.

Regards,
Alec Smecher
Public Knowledge Project Team

[jksrecko]

We are using SMTP.

[asmecher]

Hi jksrecko,

Thanks -- see http://pkp.sfu.ca/bugzilla/show_bug.cgi?id=6639. The release of OJS 2.3.5 is coming too soon for us to include this, but I'm filing against 2.3.x, which should get attention before OJS 2.4 is released.

Regards,
Alec Smecher
Public Knowledge Project Team

[aleskl]

I have tried the above patch for our 2.3.6 OJS installation and the patch went smoothly. The emails sent out of OJS seem to have UTF-8 encoded characters now, so I hope that fixed our problems with unencoded non-English characters in email headers. We were sending email to one server that rejected this kind of emails, so that was a big problem for us.

Srečko, thank you very much for the fix!