Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.10.11

Bug #20226 Mail_mimePart::encodeHeader does not encode ISO-2022-JP string.
Submitted: 2014-03-21 09:36 UTC
From: zhk72mtq Assigned: alec
Status: Closed Package: Mail_Mime (version 1.8.8)
PHP Version: 5.5.10 OS: Windows 8 64bit
Roadmaps: (Not assigned)    
Subscription  


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 36 - 17 = ?

 
 [2014-03-21 09:36 UTC] zhk72mtq (Atsushi Suzuki)
Description: ------------ Mail_mimePart#encodeHeader and Mail_mime#headers does not encode ISO-2022-JP (or other 7bit encodings possibly) string but returns raw string. In mimePart.php, whether the value needs conversion or not is determined by existence of non-ascii character. however, since ISO-2022-JP is 7bit encoding, that should be determined by existence of non-ascii printable character. Test script: --------------- <?php require_once('Mail/mimePart.php'); // precondition assert(mb_internal_encoding() === 'UTF-8'); assert(mb_detect_encoding(file_get_contents(__FILE__)) === 'UTF-8'); $encoding = 'ISO-2022-JP'; $text = '????'; $mime = new Mail_mimePart(); $subject = $mime->encodeHeader( 'Subject', mb_convert_encoding($text, $encoding), $encoding, 'base64'); var_dump($subject); Expected result: ---------------- string(38): "'=?ISO-2022-JP?B?GyRCJT8lJCVIJWsbKEI=?=" Actual result: -------------- string(14): "?$B%?%$%H%k?(B"

Comments

 [2014-03-21 10:49 UTC] zhk72mtq (Atsushi Suzuki)
 [2014-03-21 10:50 UTC] zhk72mtq (Atsushi Suzuki)
 [2014-03-22 12:44 UTC] doconnor (Daniel O'Connor)
 [2014-03-22 12:44 UTC] doconnor (Daniel O'Connor)
-Status: Open +Status: Analyzed
 [2014-03-22 12:44 UTC] doconnor (Daniel O'Connor)
Any chance of a test case, and forking + updating the PR?
 [2014-05-14 14:01 UTC] alec (Aleksander Machniak)
-Status: Analyzed +Status: Closed -Assigned To: +Assigned To: alec
This bug has been fixed in SVN. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.