Tickets :: Comment on [#8346] $conf['mailformat']['brokenrfc2231'] doesnt work, bug in Part.php

* Your Email Address
* Spam protection	Enter the letters below: . .. ..__.. .. . \|\/\|\| \|[__]\|__\|\|\/\| \| \|\|/\\|\| \|\| \|\| \|
Comment	> The workaround for the option brokenrfc2231 doesnt work, for longer > filenames at least (>50?). > > > > The check (strpos($encode_2231, '=') !== false)) will never return > true, simply because > > > > $encode_2231 = MIME::encodeRFC2231($key, $value, $charset); > > > > will not result in = but in 0= 1= etc - an encoded string looks like this: > > > > name0="aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa"; > > > name1="aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa"; > > > name2="aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa"; > > name3="aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa.pdf" > > > > there is no name= for longer filenames. Only for shorter > > > > Fix: > > > > replace if with > > > > if (!empty($GLOBALS['conf']['mailformat']['brokenrfc2231']) && > > ((strpos($encode_2231, '=') !== false) \|\| > (strpos($encode_2231, '*0=') !== false))) { > > $ctype .= '; ' . $key . '="' . MIME::encode($value, > $charset) . '"'; > > } > > > > in both places
Attachment
Watch this ticket