Tickets :: [#7730] Tickets created by email are not correctly encoded

6.0.0-beta1

11/5/25

Summary	Tickets created by email are not correctly encoded
Queue	Whups
Queue Version	1.0
Type	Bug
State	Resolved
Priority	2. Medium
Owners	jan (at) horde (dot) org
Requester	thomas (at) gelf (dot) net
Created	11/27/2008 (6187 days ago)
Due
Updated	03/16/2009 (6078 days ago)
Assigned	12/01/2008 (6183 days ago)
Resolved	03/16/2009 (6078 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

03/16/2009 11:52:34 AM	Jan Schneider	Comment #4 State ⇒ Resolved	Reply to this comment
Committed, thanks.

03/16/2009 11:52:27 AM	CVS Commit	Comment #3	Reply to this comment
Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/whups/docs/CHANGES?rt=horde&r1=1.218.2.11&r2=1.218.2.12&ty=u http://cvs.horde.org/diff.php/whups/lib/Mail.php?rt=horde&r1=1.46.2.1&r2=1.46.2.2&ty=u

12/01/2008 02:32:59 AM	Chuck Hagenbuch	Assigned to Jan Schneider Summary ⇒ Tickets created by email are not correctly encoded State ⇒ Assigned

11/27/2008 05:48:58 PM	thomas (at) gelf (dot) net	Comment #2	Reply to this comment
One additional note, there is this line of code in mail-filter.php: // Set charset to UTF-8 for most flexible conversion between email charset // and backend charset. NLS::setCharsetEnvironment('UTF-8'); So my guess regarding other systems may be wrong. This is also the reason why setting LC_ALL to different encodings had no impact in my tests.

11/27/2008 05:24:11 PM	thomas (at) gelf (dot) net	Comment #1 Priority ⇒ 2. Medium Type ⇒ Bug Summary ⇒ Tickets created by mail are not correctly encoded Queue ⇒ Whups Milestone ⇒ Patch ⇒ No State ⇒ Unconfirmed	Reply to this comment
Whups_Mail::processMail() is getting the mail body for comments by _buildComment(). This function doesn't seem to care about mail encoding - replacing $comment .= $part->transferDecode(); with $comment .= String::convertCharset($part->transferDecode(), $part->getCharset()); in Whups_Mail::_buildComment() fixes this for me. My system seems to be correctly configured, one reason for the fact that it works on other systems may be that today still most (European) systems out there are running their cron jobs as ISO8859 - and most mails have this very same encoding. Best regards, Thomas Gelf NB: Sorry for not attaching a unified diff for this one-line fix - I still need to clean up my code, remove debugging output etc...