ZF-11265: Zend_Mime_Decode::decodeQuotedPrintable() doesn't work at all

Issue Type: Bug Created: 2011-04-05T15:40:05.000+0000 Last Updated: 2011-05-04T15:27:16.000+0000 Status: Resolved Fix version(s): - 1.11.6 (05/May/11)

Reporter: Volker Grabsch (vog) Assignee: Benjamin Eberlei (beberlei) Tags: - Zend_Mime

Related issues: - ZF-10236

Attachments: - remove_useless_method_Zend_Mime_Decode_decodeQuotedPrintable.patch


The method decodeQuotedPrintable() of Zend_Mime_Decode does not decode "quoted-printable" content.

It internally calls iconv_mime_decode(). However, this PHP function performs an entirely different action.

Instead, quoted_printable_decode() would have to be used.

However, decodeQuotedPrintable() is not used at any other place in the Zend framework. Also, it is unlikely that any other application uses this method, as it doesn't work. Since quoted_printable_decode() already performs this job very well, it is unclear to me why this method exists at all.

So I propose to simply remove this method, in order to not confuse programmers anymore.


Posted by Volker Grabsch (vog) on 2011-04-05T15:41:33.000+0000

New patch: remove_useless_method_Zend_Mime_Decode_decodeQuotedPrintable.patch

remove the useless and non-functioning method Zend_Mime_Decode::decodeQuotedPrintable()

Instead, programmers should use the simple (and working) quoted_printable_decode() function.

Posted by Ralph Schindler (ralph) on 2011-05-04T15:27:16.000+0000

Fixed along side linked issue ZF-10236

Have you found an issue?

See the Overview section for more details.


© 2006-2016 by Zend, a Rogue Wave Company. Made with by awesome contributors.

This website is built using zend-expressive and it runs on PHP 7.