[#7730] Tickets created by email are not correctly encoded
Summary Tickets created by email are not correctly encoded
Queue Whups
Queue Version 1.0
Type Bug
State Resolved
Priority 2. Medium
Owners Jan Schneider <jan (at) horde (dot) org>
Requester thomas (at) gelf (dot) net
Created 11/27/08 (474 days ago)
Due
Updated 03/16/09 (365 days ago)
Assigned 11/30/08 (471 days ago)
Resolved 03/16/09 (365 days ago)
Attachments
Milestone
Patch No

History
03/16/09 Jan Schneider Comment #4
State ⇒ Resolved		 Reply to this comment
Committed, thanks.
03/16/09 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
11/30/08 Chuck Hagenbuch State ⇒ Assigned
Summary ⇒ Tickets created by email are not correctly encoded
Assigned to Jan Schneider
 
11/27/08 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/08 thomas (at) gelf (dot) net Comment #1
State ⇒ Unconfirmed
Patch ⇒
Milestone ⇒
Queue ⇒ Whups
Summary ⇒ Tickets created by mail are not correctly encoded
Type ⇒ Bug
Priority ⇒ 2. Medium		 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...