Tickets :: Comment on [#11073] Charset is not set for the "View Source"

* Your Email Address
* Spam protection	Enter the letters below: .__ .__ . .__.\ / \| \[__)\| \| \| >< \|__/[__)\|___\|__\/ \
Comment	>> One more needful thing for non-latin alphabet's users. >> When there is generates view of "View Source" [actionID=view_source] >> there is setting of the content type of 'text/plain' without charset. >> For correctness file /webroot/imp/view.php, lines after 194 should be: >> >> case 'view_source': >> $msg = $contents->fullMessageText(array('stream' => true)); >> rewind($msg); >> while (!feof($msg)) { >> $line=fgets($msg, 4096); >> if ($line !== false){ >> if (strlen($line)==0) break 1; >> if (strpos($line ,"charset=")>0 && ($pos=strpos($line >> ,";"))>0){$ct=substr($line,$pos);break 1;} >> } >> } > > No. A MIME message could theoretically have a different charset for > each part - you can't just choose the first part and use that as the > charset. Additionally, the charset is worthless information since > most messages will be encoded in US-ASCII. The charset parameter of > Content-Type deals with the UNencoded data. > > To accurately download the contents of a message, you need to use the > "Save As" feature. View Source is not meant to be a canonical > representation of the data.
Attachment
Watch this ticket