- Corrected message headers decoding when charset isn't specified and impro...

alecpl

2008-05-08 583850d00cb61771333a19ed57bef753ac140fe0

 program/steps/mail/show.inc

@@ -19,10 +19,7 @@

*/

require_once('Mail/mimeDecode.php');
require_once('lib/rc_mail_mime.inc');

$PRINT_MODE = $_action=='print' ? TRUE : FALSE;
$PRINT_MODE = $RCMAIL->action=='print' ? TRUE : FALSE;

// similar code as in program/steps/mail/get.inc
if ($_GET['_uid'])
@@ -33,16 +30,16 @@
  // set message charset as default
  if (!empty($MESSAGE['headers']->charset))
    $IMAP->set_charset($MESSAGE['headers']->charset);
  

  // go back to list if message not found (wrong UID)
  if (!$MESSAGE['headers'])
    {
    $OUTPUT->show_message('messageopenerror', 'error');
    if ($_action=='preview' && template_exists('messagepreview'))
        parse_template('messagepreview');
    if ($RCMAIL->action=='preview' && template_exists('messagepreview'))
        $OUTPUT->send('messagepreview');
    else
      {
      $_action = 'list';
      $RCMAIL->action = 'list';
      return;
      }
    }
@@ -62,9 +59,9 @@
  if ((bool)$MESSAGE['is_safe'])
    send_nocacheing_headers();
  else if (empty($CONFIG['devel_mode']))
    send_modified_header($_SESSION['login_time'], $etag);
    send_modified_header($_SESSION['login_time'], $etag, !$MESSAGE['headers']->seen);

  $MESSAGE['subject'] = rcube_imap::decode_mime_string($MESSAGE['headers']->subject, $MESSAGE['headers']->charset);
  $MESSAGE['subject'] = $IMAP->decode_header($MESSAGE['headers']->subject);
  $OUTPUT->set_pagetitle($MESSAGE['subject']);
  
  if ($MESSAGE['structure'] = $IMAP->get_structure($MESSAGE['UID']))
@@ -79,14 +76,14 @@

  // mark message as read
  if (!$MESSAGE['headers']->seen)
  {
    $marked = $IMAP->set_flag($MESSAGE['UID'], 'SEEN');
    if($RCMAIL->action == 'preview' && $marked != -1)
    {
      $marked = $IMAP->set_flag($MESSAGE['UID'], 'SEEN');
      if($_action == 'preview' && $marked != -1)
   {
   $OUTPUT->command('set_unread_count_from_preview', $mbox_name, $IMAP->messagecount($mbox_name, 'UNSEEN'), ($mbox_name == 'INBOX'));
   $OUTPUT->command('mark_as_read_from_preview', $MESSAGE['UID']);
   }
      $OUTPUT->command('set_unread_count_from_preview', $mbox_name, $IMAP->messagecount($mbox_name, 'UNSEEN'), ($mbox_name == 'INBOX'));
      $OUTPUT->command('mark_as_read_from_preview', $MESSAGE['UID']);
    }
  }

  // give message uid to the client
  $OUTPUT->set_env('uid', $MESSAGE['UID']);
@@ -206,10 +203,10 @@
  'blockedobjects' => 'rcmail_remote_objects_msg'));


if ($_action=='print' && template_exists('printmessage'))
  parse_template('printmessage');
else if ($_action=='preview' && template_exists('messagepreview'))
    parse_template('messagepreview');
if ($RCMAIL->action=='print' && template_exists('printmessage'))
  $OUTPUT->send('printmessage');
else if ($RCMAIL->action=='preview' && template_exists('messagepreview'))
    $OUTPUT->send('messagepreview');
else
  parse_template('message');
  $OUTPUT->send('message');
?>