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

[bugs at horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/11073
------------------------------------------------------------------------------
  Ticket             | 11073
  Created By         | ak at lps.komi.ru
  Summary            | Charset is not set for the "View Source"
  Queue              | IMP
  Version            | Git master
  Type               | Bug
  State              | Unconfirmed
  Priority           | 3. High
  Milestone          |
  Patch              |
  Owners             |
------------------------------------------------------------------------------


ak at lps.komi.ru (2012-03-13 08:59) wrote:

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;}
         }
     }
     fseek($msg, 0, SEEK_END);
     $headertext='text/plain';
     if (isset($ct))$headertext.=$ct;
     $browser->downloadHeaders('Message Source', $headertext, true,  
ftell($msg));

INSTEAD OF

case 'view_source':
     $msg = $contents->fullMessageText(array('stream' => true));
     fseek($msg, 0, SEEK_END);
     $browser->downloadHeaders('Message Source', 'text/plain', true,  
ftell($msg));