Removed a bunch of Ctrl-Ms at the ends of lines
[squirrelmail.git] / src / read_body.php
index 56563bab8f350ee6f5fd1dcf25c5f168a007b733..f9c22d88823740390a645728e6ee4f4e108e4c22 100644 (file)
@@ -18,37 +18,69 @@ require_once('../functions/mime.php');
 require_once('../functions/date.php');
 require_once('../functions/url_parser.php');
 require_once('../functions/smtp.php');
+require_once('../functions/html.php');
+require_once('../src/view_header.php');
 
 /**
-* Given an IMAP message id number, this will look it up in the cached
-* and sorted msgs array and return the index. Used for finding the next
-* and previous messages.
-*
-* returns the index of the next valid message from the array
-*/
+ * Given an IMAP message id number, this will look it up in the cached
+ * and sorted msgs array and return the index. Used for finding the next
+ * and previous messages.
+ *
+ * @return the index of the next valid message from the array
+ */
 function findNextMessage() {
-    global $msort, $currentArrayIndex, $msgs, $sort;
+    global $msort, $currentArrayIndex, $msgs, $sort, 
+           $thread_sort_messages, $allow_server_sort,
+           $server_sort_array;
+    if (!is_array($server_sort_array)) {
+        $thread_sort_messages = 0;
+        $allow_server_sort = FALSE;
+    }
     $result = -1;
-
-    if ($sort == 6) {
+    if ($thread_sort_messages == 1 || $allow_server_sort == TRUE) {
+        reset($server_sort_array);
+        while(list($key, $value) = each ($server_sort_array)) {
+            if ($currentArrayIndex == $value) {
+                if ($key == (count($server_sort_array) - 1)) {
+                    $result = -1;
+                    break;
+                }
+                $result = $server_sort_array[$key + 1];
+                break; 
+            }
+        }
+    } 
+    elseif ($sort == 6 && $allow_server_sort != TRUE &&
+            $thread_sort_messages != 1) {
         if ($currentArrayIndex != 1) {
             $result = $currentArrayIndex - 1;
         }
-    } else {
+    }
+    elseif ($allow_server_sort != TRUE && $thread_sort_messages != 1 ) {
+        if (!is_array($msort)) {
+            return -1;
+        }
         for (reset($msort); ($key = key($msort)), (isset($key)); next($msort)) {
             if ($currentArrayIndex == $msgs[$key]['ID']) {
                 next($msort);
                 $key = key($msort);
-                if (isset($key))
+                if (isset($key)){
                     $result = $msgs[$key]['ID'];
                     break;
+                }
             }
         }
     }
     return ($result);
 }
 
-/** Removes just one address from the list of addresses. */
+/**
+ * Removes just one address from the list of addresses. 
+ * 
+ * @param  &$addr_list  a by-ref array of addresses
+ * @param  $addr        an address to remove
+ * @return              void, since it operates on a by-ref param
+ */
 function RemoveAddress(&$addr_list, $addr) {
     if ($addr != '') {
         foreach (array_keys($addr_list, $addr) as $key_to_delete) {
@@ -60,16 +92,37 @@ function RemoveAddress(&$addr_list, $addr) {
 /** returns the index of the previous message from the array. */
 function findPreviousMessage() {
     global $msort, $currentArrayIndex, $sort, $msgs, $imapConnection,
-           $mailbox, $data_dir, $username;
-
+           $mailbox, $data_dir, $username, $thread_sort_messages,
+           $allow_server_sort, $server_sort_array;
     $result = -1;
-
-    if ($sort == 6) {
+    if (!is_array($server_sort_array)) {
+        $thread_sort_messages = 0;
+        $allow_server_sort = FALSE;
+    }
+    if ($thread_sort_messages == 1 || $allow_server_sort == TRUE) {
+        reset($server_sort_array);
+        while(list($key, $value) = each ($server_sort_array)) {
+            if ($currentArrayIndex == $value) {
+                if ($key == 0) {
+                    $result = -1;
+                    break;
+                }
+                $result = $server_sort_array[$key -1];
+                break;
+            }
+        }
+    }
+    elseif ($sort == 6 && $allow_server_sort != TRUE && 
+            $thread_sort_messages != 1) {
         $numMessages = sqimap_get_num_messages($imapConnection, $mailbox);
         if ($currentArrayIndex != $numMessages) {
             $result = $currentArrayIndex + 1;
         }
-    } else {
+    } 
+    elseif ($thread_sort_messages != 1 && $allow_server_sort != TRUE) {
+        if (!is_array($msort)) {
+            return -1;
+        }
         for (reset($msort); ($key = key($msort)), (isset($key)); next($msort)) {
             if ($currentArrayIndex == $msgs[$key]['ID']) {
                 prev($msort);
@@ -85,9 +138,9 @@ function findPreviousMessage() {
 }
 
 /**
-* Displays a link to a page where the message is displayed more
-* "printer friendly".
-*/
+ * Displays a link to a page where the message is displayed more
+ * "printer friendly".
+ */
 function printer_friendly_link() {
     global $passed_id, $mailbox, $ent_num, $color,
            $pf_subtle_link,
@@ -98,20 +151,19 @@ function printer_friendly_link() {
     }
 
     $params = '?passed_ent_id=' . $ent_num .
-              '&mailbox=' . urlencode($mailbox) .
-              '&passed_id=' . $passed_id;
+        '&mailbox=' . urlencode($mailbox) .
+        '&passed_id=' . $passed_id;
 
     $print_text = _("View Printable Version");
 
     if (!$pf_subtle_link) {
         /* The link is large, on the bottom of the header panel. */
-        $result =       '<tr bgcolor="' . $color[0] . '">' .
-                        '<td class="medText" align="right" valign="top">' .
-                          '&nbsp;' .
-                        '</td><td class="medText" valign="top" colspan="2">'."\n";
+        $result = html_tag( 'tr', '', '', $color[0] ) .
+            html_tag( 'td', '&nbsp;', 'right', '', 'class="medText" valign="top"' ) .
+            html_tag( 'td', '', 'left', '', 'class="medText" valign="top" colspan="2"' ) . "\n";
     } else {
         /* The link is subtle, below "view full header". */
-        $result = "<BR>\n";
+        $result = "<br>\n";
     }
 
     /* Output the link. */
@@ -124,10 +176,10 @@ function printer_friendly_link() {
                 "  }\n" .
                 "// -->\n" .
                 "</script>\n" .
-                "<A HREF=\"javascript:printFormat();\">$print_text</A>\n";
+                "<a href=\"javascript:printFormat();\">$print_text</a>\n";
     } else {
-        $result .= '<A TARGET="_blank" HREF="../src/printer_friendly_bottom.php' .
-                "$params\">$print_text</A>\n";
+        $result .= '<A target="_blank" HREF="../src/printer_friendly_bottom.php' .
+                "$params\">$print_text</a>\n";
     }
 
     if (!$pf_subtle_link) {
@@ -163,6 +215,8 @@ function SendMDN ( $recipient , $sender) {
     $subject = $header->subject;
     $now = getLongDateString( time() );
 
+    set_my_charset();
+
     $body = _("Your message") . "\r\n\r\n" .
             "\t" . _("To:") . ' ' . $to . "\r\n" .
             "\t" . _("Subject:") . ' ' . $subject . "\r\n" .
@@ -195,23 +249,23 @@ function SendMDN ( $recipient , $sender) {
     $newAttachment = array();
     $newAttachment['localfilename'] = $localfilename;
     $newAttachment['type'] = "message/disposition-notification";
-
+    $newAttachment['session']=-1;
     $attachments[] = $newAttachment;
     $MDN_to = trim($recipient);
     $reply_id = 0;
 
-    return (SendMessage($MDN_to,'','', _("Read:") . ' ' . $subject, $body,$reply_id, True, 3) );
+    return (SendMessage($MDN_to, '', '', _("Read:") . ' ' . $subject, 
+                        $body, $reply_id, True, 3, -1) );
 }
 
 
 function ToggleMDNflag ( $set ) {
-    global $imapConnection, $passed_id, $mailbox;
+    global $imapConnection, $passed_id, $mailbox, $uid;
     sqimap_mailbox_select($imapConnection, $mailbox);
-    
     $sg =  $set?'+':'-';
     $cmd = 'STORE ' . $passed_id . ' ' . $sg . 'FLAGS ($MDNSent)';
     $read = sqimap_run_command ($imapConnection, $cmd, true, $response, 
-                                $readmessage);
+                                $readmessage, $uid);
 }
 
 function ClearAttachments() {
@@ -219,17 +273,90 @@ function ClearAttachments() {
 
         $hashed_attachment_dir = getHashedDir($username, $attachment_dir);
 
+       $rem_attachments = array();
         foreach ($attachments as $info) {
-            $attached_file = "$hashed_attachment_dir/$info[localfilename]";
-            if (file_exists($attached_file)) {
-                unlink($attached_file);
-            }
-        }
+           if ($info['session'] == -1) {
+               $attached_file = "$hashed_attachment_dir/$info[localfilename]";
+               if (file_exists($attached_file)) {
+                   unlink($attached_file);
+               }
+           } else {
+               $rem_attachments[] = $info;
+           }
+       }
+        $attachments = $rem_attachments;
+}
 
-        $attachments = array();
+function formatRecipientString($recipients, $item ) {
+    global $base_uri, $passed_id, $urlMailbox, $startMessage, $show_more_cc, 
+           $echo_more, $echo_less, $show_more, $show_more_bcc, $sort, $passed_ent_id,
+          $PHP_SELF;
+
+    $i = 0;
+    $url_string = '';    
+    if ((is_array($recipients)) && (isset($recipients[0]))) {
+        $string = '';
+        $ary = $recipients;
+       $show = false;
+
+        if ($item == 'to') {
+          if ($show_more) {
+              $show = true;
+              $url = set_url_var($PHP_SELF, 'show_more',0);
+          } else {
+              $url = set_url_var($PHP_SELF, 'show_more',1);
+          }       
+       } else if ($item == 'cc') {
+          if ($show_more_cc) {
+              $url = set_url_var($PHP_SELF, 'show_more_cc',0);
+              $show = true;
+          } else {
+             $url = set_url_var($PHP_SELF, 'show_more_cc',1);
+          }
+       } else if ($item == 'bcc') {
+          if ($show_more_bcc) {
+             $url = set_url_var($PHP_SELF, 'show_more_bcc',0);
+             $show = true;
+          } else {
+             $url = set_url_var($PHP_SELF, 'show_more_bcc',1);
+          }       
+       }
+
+       $cnt = count($ary);
+        while ($i < $cnt) {
+           $addr_o = $ary[$i];
+           $ary[$i] = $addr_o->getAddress();
+           $ary[$i] = decodeHeader(htmlspecialchars($ary[$i]));
+           $url_string .= $ary[$i];
+           if ($string) {
+                $string .= '<BR>'.$ary[$i];
+           } else {
+                $string = $ary[$i];
+               if ($cnt>1) {
+                    $string .= '&nbsp;(<A HREF="'.$url;
+                   if ($show) {
+                      $string .= '">'.$echo_less.'</A>)';
+                   } else {
+                      $string .= '">'.$echo_more.'</A>)';
+                      break;
+                   }
+               } 
+           }
+           $i++;
+        }
+    }
+    else {
+        $string = '';
+    }
+    $url_string = urlencode($url_string);
+    $result = array();
+    $result['str'] = $string;
+    $result['url_str'] = $url_string;
+    return $result;
 }
 
 
+
 /*
  *   Main of read_boby.php  --------------------------------------------------
  */
@@ -240,88 +367,70 @@ function ClearAttachments() {
     $passed_id
 */
 
-if ( isset( $mailbox ) ) {
+global $uid_support, $sqimap_capabilities;
+
+if (isset($mailbox)){
     $mailbox = urldecode( $mailbox );
 }
-$imapConnection = sqimap_login($username, $key, $imapServerAddress, $imapPort, 0);
-$read = sqimap_mailbox_select($imapConnection, $mailbox, false, false, true);
+
+$imapConnection = sqimap_login($username, $key, $imapServerAddress, 
+                               $imapPort, 0);
+
+$mbx_response = sqimap_mailbox_select($imapConnection, $mailbox, false, false, true);
+
+if (!isset($messages)) {
+    $messages = array();
+    session_register('messages');
+}
+
+/**
+ * $message contains all information about the message
+ * including header and body
+ */
+
+$uidvalidity = $mbx_response['UIDVALIDITY'];
+if (!isset($messages[$uidvalidity])) {
+   $messages[$uidvalidity] = array();
+}  
+if (!isset($messages[$uidvalidity][$passed_id])) {
+    $message = sqimap_get_message($imapConnection, $passed_id, $mailbox);
+    $messages[$uidvalidity][$passed_id] = $message;
+    $header = $message->header;
+} else {
+    $message = $messages[$uidvalidity][$passed_id];
+//    $message = sqimap_get_message($imapConnection, $passed_id, $mailbox);
+    if (isset($passed_ent_id)) {
+       $message = $message->getEntity($passed_ent_id);
+       $message->id = $passed_id;
+       $message->mailbox = $mailbox;
+    }
+    $header = $message->header;
+}
 
 do_hook('html_top');
 
 /*
  * The following code sets necesarry stuff for the MDN thing
  */
-if( $default_use_mdn &&
-    ( $mdn_user_support = getPref($data_dir, $username, 'mdn_user_support', $default_use_mdn) ) ) {
-
-    $supportMDN = ServerMDNSupport($read["PERMANENTFLAGS"]);
-    $flags = sqimap_get_flags ($imapConnection, $passed_id);
-    $FirstTimeSee = !(in_array( 'Seen', $flags ));
+if($default_use_mdn &&
+   ($mdn_user_support = getPref($data_dir, $username, 'mdn_user_support', 
+                                $default_use_mdn))) {
+    $supportMDN = ServerMDNSupport($mbx_response["PERMANENTFLAGS"]);
+    $FirstTimeSee = !$message->is_seen;
 }
 
 displayPageHeader($color, $mailbox);
 
-
 /*
  * The following code shows the header of the message and then exit
  */
 if (isset($view_hdr)) {
-    $read = sqimap_run_command ($imapConnection, "FETCH $passed_id BODY[HEADER]", true, $a, $b);
-
-    echo '<BR>' .
-        '<TABLE WIDTH="100%" CELLPADDING="2" CELLSPACING="0" BORDER="0" ALIGN="CENTER">' . "\n" .
-        "   <TR><TD BGCOLOR=\"$color[9]\" WIDTH=\"100%\" ALIGN=\"CENTER\"><B>" . _("Viewing Full Header") . '</B> - '.
-        '<a href="' . $base_uri . "src/read_body.php?mailbox=".urlencode($mailbox);
-    if (isset($where) && isset($what)) {
-        // Got here from a search
-        echo "&amp;passed_id=$passed_id&amp;where=".urlencode($where)."&amp;what=".urlencode($what).'">';
-    } else {
-        echo "&amp;passed_id=$passed_id&amp;startMessage=$startMessage&amp;show_more=$show_more\">";
-    }
-    echo _("View message") . "</a></b></td></tr></table>\n" .
-         "<table width=\"99%\" cellpadding=2 cellspacing=0 border=0 align=center>\n" .
-         '<tr><td>';
-
-    $cnum = 0;
-    for ($i=1; $i < count($read); $i++) {
-        $line = htmlspecialchars($read[$i]);
-        if (eregi("^&gt;", $line)) {
-            $second[$i] = $line;
-            $first[$i] = '&nbsp;';
-            $cnum++;
-        } else if (eregi("^[ |\t]", $line)) {
-            $second[$i] = $line;
-            $first[$i] = '';
-        } else if (eregi("^([^:]+):(.+)", $line, $regs)) {
-            $first[$i] = $regs[1] . ':';
-            $second[$i] = $regs[2];
-            $cnum++;
-        } else {
-            $second[$i] = trim($line);
-            $first[$i] = '';
-        }
-    }
-    for ($i=0; $i < count($second); $i = $j) {
-        if (isset($first[$i])) {
-            $f = $first[$i];
-        }
-        if (isset($second[$i])) {
-            $s = nl2br($second[$i]);
-        }
-        $j = $i + 1;
-        while (($first[$j] == '') && ($j < count($first))) {
-            $s .= '&nbsp;&nbsp;&nbsp;&nbsp;' . nl2br($second[$j]);
-            $j++;
-        }
-        parseEmail($s);
-        if (isset($f)) {
-               echo "<nobr><tt><b>$f</b>$s</tt></nobr>";
-        }
-    }
-    echo "</td></tr></table>\n" .
-         '</body></html>';
-    sqimap_logout($imapConnection);
-    exit;
+   $template_vars = array();
+   parse_viewheader($imapConnection,$passed_id,&$template_vars);
+   $template_vars['return_address'] = set_url_var($PHP_SELF, 'view_hdr');
+   view_header($template_vars, '', '</body></html>');
+   exit;
 }
 
 if (isset($msgs)) {
@@ -329,84 +438,130 @@ if (isset($msgs)) {
 } else {
     $currentArrayIndex = -1;
 }
+$msgs[$passed_id]['FLAG_SEEN'] = true;
 
-for ($i = 0; $i < count($msgs); $i++) {
-    if ($msgs[$i]['ID'] == $passed_id) {
-        $msgs[$i]['FLAG_SEEN'] = true;
-    }
+
+/*
+ * The following code sets necesarry stuff for the MDN thing
+ */
+if($default_use_mdn &&
+   ($mdn_user_support = getPref($data_dir, $username, 'mdn_user_support', 
+                                $default_use_mdn))) {
+    $supportMDN = ServerMDNSupport($mbx_response["PERMANENTFLAGS"]);
+    $FirstTimeSee = !$message->is_seen;
 }
 
-// $message contains all information about the message
-// including header and body
-$message = sqimap_get_message($imapConnection, $passed_id, $mailbox);
+/*
+ * The following code shows the header of the message and then exit
+ */
+if (isset($view_hdr)) {
+   $template_vars = array();
+   parse_viewheader($imapConnection,$passed_id,&$template_vars);
+   $template_vars['return_address'] = set_url_var($PHP_SELF, 'view_hdr');
+   view_header($template_vars, '', '</body></html>');
+   exit;
+}
+
+if (isset($msgs)) {
+    $currentArrayIndex = $passed_id;
+} else {
+    $currentArrayIndex = -1;
+}
+$msgs[$passed_id]['FLAG_SEEN'] = true;
 
 /** translate the subject and mailbox into url-able text **/
-$url_subj = urlencode(trim($message->header->subject));
+$url_subj = urlencode(trim($header->subject));
 $urlMailbox = urlencode($mailbox);
 $url_replyto = '';
-if (isset($message->header->replyto)) {
-    $url_replyto = urlencode($message->header->replyto);
+if (isset($header->replyto)) {
+    $addr_o = $header->replyto;
+    $addr_s = $addr_o->getAddress();
+    $url_replyto = urlencode($addr_s);
 }
 
-$url_replytoall   = $url_replyto;
-
-// If we are replying to all, then find all other addresses and
-// add them to the list.  Remove duplicates.
-// This is somewhat messy, so I'll explain:
-// 1) Take all addresses (from, to, cc) (avoid nasty join errors here)
-$url_replytoall_extra_addrs = array_merge(
-    array($message->header->from),
-    $message->header->to,
-    $message->header->cc
-);
+$url_replytoall = $url_replyto;
 
-// 2) Make one big string out of them
-$url_replytoall_extra_addrs = join(';', $url_replytoall_extra_addrs);
-
-// 3) Parse that into an array of addresses
-$url_replytoall_extra_addrs = parseAddrs($url_replytoall_extra_addrs);
+/**
+ * If we are replying to all, then find all other addresses and
+ * add them to the list.  Remove duplicates.
+ */
 
-// 4) Make them unique -- weed out duplicates
-// (Coded for PHP 4.0.0)
-$url_replytoall_extra_addrs =
-    array_keys(array_flip($url_replytoall_extra_addrs));
+$excl_arr = array();
 
-// 5) Remove the addresses we'll be sending the message 'to'
+/**
+ * 1) Remove the addresses we'll be sending the message 'to'
+ */
 $url_replytoall_avoid_addrs = '';
-if (isset($message->header->replyto)) {
-    $url_replytoall_avoid_addrs = $message->header->replyto;
+if (isset($header->replyto)) {
+    $excl_ar = $header->getAddr_a('replyto');
 }
 
-$url_replytoall_avoid_addrs = parseAddrs($url_replytoall_avoid_addrs);
-foreach ($url_replytoall_avoid_addrs as $addr) {
-    RemoveAddress($url_replytoall_extra_addrs, $addr);
-}
 
-// 6) Remove our identities from the CC list (they still can be in the
-// TO list) only if $include_self_reply_all is turned off
+/**
+ * 2) Remove our identities from the CC list (they still can be in the
+ * TO list) only if $include_self_reply_all is turned off
+ */
 if (!$include_self_reply_all) {
-    RemoveAddress($url_replytoall_extra_addrs,
-                getPref($data_dir, $username, 'email_address'));
+    $email_address = trim(getPref($data_dir, $username, 'email_address'));
+    $excl_ar[$email_address] = '';
+    
     $idents = getPref($data_dir, $username, 'identities');
     if ($idents != '' && $idents > 1) {
         for ($i = 1; $i < $idents; $i ++) {
-            $cur_email_address = getPref($data_dir, $username, 'email_address' . $i);
-            RemoveAddress($url_replytoall_extra_addrs, $cur_email_address);
+            $cur_email_address = getPref($data_dir, $username, 
+                                         'email_address' . $i);
+            $cur_email_address = strtolower($cur_email_address);
+           $excl_ar[$cur_email_address] = '';
         }
     }
 }
 
-// 7) Smoosh back into one nice line
-$url_replytoallcc = getLineOfAddrs($url_replytoall_extra_addrs);
+/** 
+ * 3) get the addresses.
+ */
+$url_replytoall_ar = $header->getAddr_a(array('from','to','cc'), $excl_ar);
 
-// 8) urlencode() it
+/** 
+ * 4) generate the string.
+ */
+$url_replytoallcc = '';
+foreach( $url_replytoall_ar as $email => $personal) {
+    if ($personal) {
+       $url_replytoallcc .= ", \"$personal\" <$email>";
+    } else {
+       $url_replytoallcc .= ', '. $email;    
+    }
+}
+$url_replytoallcc = substr($url_replytoallcc,2);
+
+/**
+ * 5) urlencode() it
+ */
 $url_replytoallcc = urlencode($url_replytoallcc);
 
-$dateString = getLongDateString($message->header->date);
+$dateString = getLongDateString($header->date);
+
+/**
+ * What do we reply to -- text only, if possible
+ */
+$body = ''; 
+    
+/* first step in displaying multiple entities */
+
+    $ent_ar = findDisplayEntity($message, false);
+    $i = 0;
+    for ($i = 0; $i < count($ent_ar); $i++) {
+       $body .= formatBody($imapConnection, $message, $color, $wrap_at, $ent_ar[$i]);
+    }
+
 
-// What do we reply to -- text only, if possible
-$ent_num = findDisplayEntity($message);
+$ent_ar = findDisplayEntity($message,true);
 
+$ent_num = $ent_ar[0];
+for ($i = 1 ; $i < count($ent_ar); $i++) {
+    $ent_num .= '_'.$ent_ar[$i];
+}
 /** TEXT STRINGS DEFINITIONS **/
 $echo_more = _("more");
 $echo_less = _("less");
@@ -415,147 +570,30 @@ if (!isset($show_more_cc)) {
     $show_more_cc = FALSE;
 }
 
-/** FORMAT THE TO STRING **/
-$i = 0;
-$to_string = '';
-$url_to_string = '';
-$to_ary = $message->header->to;
-while ($i < count($to_ary)) {
-    $to_ary[$i] = htmlspecialchars(decodeHeader($to_ary[$i]));
-    $url_to_string .= $to_ary[$i];
-
-    if ($to_string) {
-        $to_string = "$to_string<BR>$to_ary[$i]";
-    } else {
-        $to_string = "$to_ary[$i]";
-    }
-
-    $i++;
-    if (count($to_ary) > 1) {
-        if ($show_more == false) {
-            if ($i == 1) {
-                /* From a search... */
-                $to_string .= '&nbsp;(<A HREF="' . $base_uri .
-                             "src/read_body.php?mailbox=$urlMailbox&amp;passed_id=$passed_id&amp;";
-                if (isset($where) && isset($what)) {
-                    $to_string .= 'where='.urlencode($where)."&amp;what=".urlencode($what)."&amp;show_more=1&amp;show_more_cc=$show_more_cc\">$echo_more</A>)";
-                } else {
-                    $to_string .= "sort=$sort&amp;startMessage=$startMessage&amp;show_more=1&amp;show_more_cc=$show_more_cc\">$echo_more</A>)";
-                }
-                $i = count($to_ary);
-            }
-        } else if ($i == 1) {
-            /* From a search... */
-            $to_string .= '&nbsp;(<A HREF="' . $base_uri .
-                         "src/read_body.php?mailbox=$urlMailbox&amp;passed_id=$passed_id&amp;";
-            if (isset($where) && isset($what)) {
-                $to_string .= 'where='.urlencode($where)."&amp;what=".urlencode($what)."&amp;show_more=0&amp;show_more_cc=$show_more_cc\">$echo_less</A>)";
-            } else {
-                $to_string .= "sort=$sort&amp;startMessage=$startMessage&amp;show_more=0&amp;show_more_cc=$show_more_cc\">$echo_less</A>)";
-            }
-        }
-    }
+if (!isset($show_more_bcc)) {
+    $show_more_bcc = FALSE;
 }
-$url_to_string = urlencode($url_to_string);
+
+/** FORMAT THE TO STRING **/
+$to = formatRecipientString($message->header->to, "to");
+$to_string = $to['str'];
+$url_to_string = $to['url_str'];
 
 
 /** FORMAT THE CC STRING **/
-$i = 0;
-$url_cc_string = '';
-if (isset ($message->header->cc[0]) && trim($message->header->cc[0])) {
-    $cc_string = '';
-    $cc_ary = $message->header->cc;
-    while ($i < count(decodeHeader($cc_ary))) {
-        $cc_ary[$i] = htmlspecialchars($cc_ary[$i]);
-        $url_cc_string .= $cc_ary[$i];
-
-        if ($cc_string) {
-            $cc_string = "$cc_string<BR>$cc_ary[$i]";
-        } else {
-            $cc_string = "$cc_ary[$i]";
-        }
 
-        $i++;
-        if (count($cc_ary) > 1) {
-            if ($show_more_cc == false) {
-                if ($i == 1) {
-                    /* From a search... */
-                    $cc_string .= '&nbsp;(<A HREF="' . $base_uri .
-                                  "src/read_body.php?mailbox=$urlMailbox&passed_id=$passed_id";
-                    if (isset($where) && isset($what)) {
-                        $cc_string .= '&amp;what='.urlencode($what)."&amp;where=".urlencode($where)."&amp;show_more_cc=1&amp;show_more=$show_more\">$echo_more</A>)";
-                    } else {
-                        $cc_string .= "&amp;sort=$sort&amp;startMessage=$startMessage&amp;show_more_cc=1&amp;show_more=$show_more\">$echo_more</A>)";
-                    }
-                    $i = count($cc_ary);
-                }
-            } else if ($i == 1) {
-                /* From a search... */
-                $cc_string .= '&nbsp;(<A HREF="' . $base_uri .
-                              "src/read_body.php?mailbox=$urlMailbox&amp;passed_id=$passed_id&amp;";
-                if (isset($where) && isset($what)) {
-                    $cc_string .= 'what=' . urlencode($what)."&amp;where=".urlencode($where)."&amp;show_more_cc=0&amp;show_more=$show_more\">$echo_less</A>)";
-                } else {
-                    $cc_string .= "sort=$sort&amp;startMessage=$startMessage&amp;show_more_cc=0&amp;show_more=$show_more\">$echo_less</A>)";
-                }
-            }
-        }
-    }
-}
-else {
-    $cc_string = '';
-}
-$url_cc_string = urlencode($url_cc_string);
+$cc = formatRecipientString($header->cc, "cc");
+$cc_string = $cc['str'];
+$url_cc_string = $cc['url_str'];
 
 /** FORMAT THE BCC STRING **/
-$i = 0;
-$url_bcc_string = '';
-if (isset ($message->header->bcc[0]) && trim($message->header->bcc[0])){
-    $bcc_string = '';
-    $bcc_ary = $message->header->bcc;
-    while ($i < count(decodeHeader($bcc_ary))) {
-        $bcc_ary[$i] = htmlspecialchars($bcc_ary[$i]);
-        $url_bcc_string .= $bcc_ary[$i];
-        if ($bcc_string) {
-            $bcc_string = "$bcc_string<BR>$bcc_ary[$i]";
-        } else {
-            $bcc_string = "$bcc_ary[$i]";
-        }
 
-        $i++;
-        if (count($bcc_ary) > 1) {
-            if ($show_more_cc == false) {
-                if ($i == 1) {
-                    /* From a search... */
-                    $bcc_string .= '&nbsp;(<A HREF="' . $base_uri .
-                                   "src/read_body.php?mailbox=$urlMailbox&amp;passed_id=$passed_id&amp;";
-                    if (isset($where) && isset($what)) {
-                        $bcc_string .= 'what=' . urlencode($what)."&amp;where=".urlencode($where)."&amp;show_more_cc=1&amp;show_more=$show_more\">$echo_more</A>)";
-                    } else {
-                        $bcc_string .= "sort=$sort&amp;startMessage=$startMessage&amp;show_more_cc=1&amp;show_more=$show_more\">$echo_more</A>)";
-                    }
-                    $i = count($bcc_ary);
-                }
-            } else if ($i == 1) {
-                /* From a search... */
-                $bcc_string .= '&nbsp;(<A HREF="' . $base_uri .
-                               "src/read_body.php?mailbox=$urlMailbox&amp;passed_id=$passed_id&amp;";
-                if (isset($where) && isset($what)) {
-                    $bcc_string .= 'what=' . urlencode($what)."&amp;where=".urlencode($where)."&amp;show_more_cc=0&amp;show_more=$show_more\">$echo_less</A>)";
-                } else {
-                    $bcc_string .= "sort=$sort&amp;startMessage=$startMessage&amp;show_more_cc=0&amp;show_more=$show_more\">$echo_less</A>)";
-                }
-            }
-        }
-    }
-}
-else {
-    $bcc_string = '';
-}
-$url_bcc_string = urlencode($url_bcc_string);
+$bcc = formatRecipientString($header->bcc, "bcc");
+$bcc_string = $bcc['str'];
+$url_bcc_string = $bcc['url_str'];
 
 if ($default_use_priority) {
-    $priority_level = substr($message->header->priority,0,1);
+    $priority_level = substr($header->priority,0,1);
 
     switch($priority_level) {
         /* check for a higher then normal priority. */
@@ -581,130 +619,190 @@ if ($default_use_priority) {
 }
 
 /** make sure everything will display in HTML format **/
-$from_name = decodeHeader(htmlspecialchars($message->header->from));
+
+$from_o = $header->from;
+if (is_object($from_o)) {
+    $from_name = $from_o->getAddress();
+} else {
+    $from_name = _("Unknown sender");
+}
+$from_name = decodeHeader(htmlspecialchars($from_name));
 $subject = decodeHeader(htmlspecialchars($message->header->subject));
+$identity = '';
+$idents = getPref($data_dir, $username, 'identities');
+if (!empty($idents) && $idents > 1) {
+    for ($i = 1; $i < $idents; $i++) {
+        $enc_from_name = '"'. 
+            encodeHeader(getPref($data_dir, 
+                                 $username, 
+                                 'full_name' . $i)) .
+            '" <' . getPref($data_dir, $username, 
+                            'email_address' . $i) . '>';
+        if (htmlspecialchars($enc_from_name) == $from_name) {
+            $identity = $i;
+            break;
+        }
+    }
+}
 
 do_hook('read_body_top');
-echo '<BR>' .
-     '<TABLE CELLSPACING="0" WIDTH="100%" BORDER="0" ALIGN="CENTER" CELLPADDING="0">' .
-        '<TR><TD BGCOLOR="' . $color[9] . '" WIDTH="100%">' .
-           '<TABLE WIDTH="100%" CELLSPACING="0" BORDER="0" CELLPADDING="3">' .
-              '<TR>' .
-                 '<TD ALIGN="LEFT" WIDTH="33%">' .
-                    '<SMALL>' .
-     '<A HREF="' . $base_uri . 'src/';
+
+echo '<br>' .
+     html_tag( 'table', '', 'center', '', 'width="100%" cellpadding="0" cellspacing="0" border="0"' ) . "\n" .
+     html_tag( 'tr' ) .
+     html_tag( 'td', '', 'left', $color[9], 'width="100%"' ) .
+     html_tag( 'table', '', '', '', 'width="100%" cellpadding="3" cellspacing="0" border="0"' ) . "\n" .
+     html_tag( 'tr' ) .
+     html_tag( 'td', '', 'left', '', 'width="33%"' ) .
+     '<small>' .
+     '<a href="' . $base_uri . 'src/';
 
 if ($where && $what) {
-    if( $pos == '' ) {
-        $pos = 0;
+    if ($pos == '') {
+        $pos=0;
     }
-    echo "search.php?where$pos=".urlencode($where)."&amp;pos=$pos&amp;what$pos=".urlencode($what)."&amp;mailbox=$urlMailbox\">";
+    echo "search.php?where=".urlencode($where)."&amp;pos=$pos&amp;what=".urlencode($what)."&amp;mailbox=$urlMailbox\">";
 } else {
     echo "right_main.php?sort=$sort&amp;startMessage=$startMessage&amp;mailbox=$urlMailbox\">";
 }
 echo _("Message List") .
-     '</A>&nbsp;|&nbsp;' .
-     '<A HREF="' . $base_uri . "src/delete_message.php?mailbox=$urlMailbox&amp;message=$passed_id&amp;";
+     '</a>&nbsp;|&nbsp;' .
+     '<a href="' . $base_uri . "src/delete_message.php?mailbox=$urlMailbox&amp;message=$passed_id&amp;";
 if ($where && $what) {
     echo 'where=' . urlencode($where) . '&amp;what=' . urlencode($what) . '">';
 } else {
     echo "sort=$sort&amp;startMessage=$startMessage\">";
 }
-echo _("Delete") . '</A>&nbsp;';
+echo _("Delete") . '</a>&nbsp;';
 if (($mailbox == $draft_folder) && ($save_as_draft)) {
-    echo '|&nbsp;<A HREF="' . $base_uri .
-         "src/compose.php?mailbox=$mailbox&amp;send_to=$url_to_string&amp;send_to_cc=$url_cc_string&amp;send_to_bcc=$url_bcc_string&amp;subject=$url_subj&amp;draft_id=$passed_id&amp;ent_num=$ent_num" . '"';
+    $comp_uri = $base_uri . "src/compose.php?mailbox=$mailbox&amp;".
+                "identity=$identity&amp;send_to=$url_to_string&amp;".
+                "send_to_cc=$url_cc_string&amp;send_to_bcc=$url_bcc_string&amp;".
+                "subject=$url_subj&amp;mailprio=$priority_level&amp;".
+                "draft_id=$passed_id&amp;ent_num=$ent_num";
+
     if ($compose_new_win == '1') {
-        echo 'TARGET="compose_window" onClick="comp_in_new()"';
+        echo "<a href=\"javascript:void(0)\" onclick=\"comp_in_new(false,'$comp_uri')\"";
+    } else {
+        echo '|&nbsp;<a href="' . $comp_uri .'"';
     }
     echo '>'.
          _("Resume Draft") . '</a>';
 }
+if ($mailbox == $sent_folder) {
+    $comp_uri = $base_uri . "src/compose.php?mailbox=$mailbox&amp;".
+                "identity=$identity&amp;send_to=$url_to_string&amp;".
+                "send_to_cc=$url_cc_string&amp;send_to_bcc=$url_bcc_string&amp;".
+                "subject=$url_subj&amp;mailprio=$priority_level&amp;".
+                "ent_num=$ent_num&amp;passed_id=$passed_id&amp;edit_as_new=1";
+
+    if ($compose_new_win == '1') {
+        echo "<a href=\"javascript:void(0)\" onclick=\"comp_in_new(false,'$comp_uri')\"";
+    } else {
+        echo '|&nbsp;<a href="' . $comp_uri .'"';
+    }
+    echo '>'.
+          _("Edit Message as New") . '</a>';
+}
 
 echo '&nbsp;&nbsp;' .
-                   '</SMALL>' .
-                '</TD>' .
-                '<TD WIDTH="33%" ALIGN="CENTER">' .
-                   '<SMALL>';
+                   '</small>' .
+                '</td>' .
+                html_tag( 'td', '', 'center', '', 'width="33%"' ) .
+                   '<small>';
 
 if ( !($where && $what) ) {
-
     if ($currentArrayIndex == -1) {
         echo 'Previous&nbsp;|&nbsp;Next';
     } else {
-        $prev = findPreviousMessage();
+        $prev = findPreviousMessage($mbx_response['EXISTS']);
         $next = findNextMessage();
 
         if ($prev != -1) {
-            echo '<a href="' . $base_uri . "src/read_body.php?passed_id=$prev&amp;mailbox=$urlMailbox&amp;sort=$sort&amp;startMessage=$startMessage&amp;show_more=0\">" . _("Previous") . "</A>&nbsp;|&nbsp;";
+            echo '<a href="' . $base_uri . "src/read_body.php?passed_id=$prev&amp;mailbox=$urlMailbox&amp;sort=$sort&amp;startMessage=$startMessage&amp;show_more=0\">" . _("Previous") . "</a>&nbsp;|&nbsp;";
         } else {
             echo _("Previous") . '&nbsp;|&nbsp;';
         }
 
         if ($next != -1) {
-            echo '<a href="' . $base_uri . "src/read_body.php?passed_id=$next&amp;mailbox=$urlMailbox&amp;sort=$sort&amp;startMessage=$startMessage&amp;show_more=0\">" . _("Next") . "</A>";
+            echo '<a href="' . $base_uri . "src/read_body.php?passed_id=$next&amp;mailbox=$urlMailbox&amp;sort=$sort&amp;startMessage=$startMessage&amp;show_more=0\">" . _("Next") . "</a>";
         } else {
             echo _("Next");
         }
     }
 }
 
-echo                '</SMALL>' .
-                '</TD><TD WIDTH="33%" ALIGN="RIGHT">' .
-                   '<SMALL>' .
-                   '<A HREF="' . $base_uri . "src/compose.php?forward_id=$passed_id&amp;forward_subj=$url_subj&amp;".
-                    ($default_use_priority?"mailprio=$priority_level&amp;":'')
-                    ."mailbox=$urlMailbox&amp;ent_num=$ent_num\"";
-    if ($compose_new_win == '1') {
-        echo 'TARGET="compose_window" onClick="comp_in_new()"';
-    }
+echo                '</small>' .
+                '</td>' .
+                html_tag( 'td', '', 'right', '', 'width="33%"' ) .
+                   '<small>' ;
+$comp_uri = $base_uri . "src/compose.php?forward_id=$passed_id&amp;".
+            "forward_subj=$url_subj&amp;".
+            ($default_use_priority?"mailprio=$priority_level&amp;":'').
+            "mailbox=$urlMailbox&amp;ent_num=$ent_num";
+
+if ($compose_new_win == '1') {
+    echo "<a href=\"javascript:void(0)\" onclick=\"comp_in_new(false,'$comp_uri')\"";
+} else {
+    echo '|&nbsp;<a href="' . $comp_uri .'"';
+}
+
     echo '>'.
     _("Forward") .
-    '</A>&nbsp;|&nbsp;' .
-                   '<A HREF="' . $base_uri . "src/compose.php?send_to=$url_replyto&amp;reply_subj=$url_subj&amp;".
-                    ($default_use_priority?"mailprio=$priority_level&amp;":'').
-                    "reply_id=$passed_id&amp;mailbox=$urlMailbox&amp;ent_num=$ent_num\"";
-    if ($compose_new_win == '1') {
-        echo 'TARGET="compose_window" onClick="comp_in_new()"';
-    }
+    '</a>&nbsp;|&nbsp;';
+
+$comp_uri = $base_uri . "src/compose.php?send_to=$url_replyto&amp;".
+            "reply_subj=$url_subj&amp;".
+            ($default_use_priority?"mailprio=$priority_level&amp;":'').
+            "reply_id=$passed_id&amp;mailbox=$urlMailbox&amp;ent_num=$ent_num";
+
+if ($compose_new_win == '1') {
+    echo "<a href=\"javascript:void(0)\" onclick=\"comp_in_new(false,'$comp_uri')\"";
+} else {
+    echo '|&nbsp;<a href="' . $comp_uri .'"';
+}
+
     echo '>'.
     _("Reply") .
-    '</A>&nbsp;|&nbsp;' .
-                   '<A HREF="' . $base_uri . "src/compose.php?send_to=$url_replytoall&amp;send_to_cc=$url_replytoallcc&amp;reply_subj=$url_subj&amp;".
-                    ($default_use_priority?"mailprio=$priority_level&amp;":'').
-                    "reply_id=$passed_id&amp;mailbox=$urlMailbox&amp;ent_num=$ent_num\"";
-    if ($compose_new_win == '1') {
-        echo 'TARGET="compose_window" onClick="comp_in_new()"';
-    }
+    '</a>&nbsp;|&nbsp;';
+
+$comp_uri = $base_uri . "src/compose.php?send_to=$url_replytoall&amp;".
+            "send_to_cc=$url_replytoallcc&amp;reply_subj=$url_subj&amp;".
+            ($default_use_priority?"mailprio=$priority_level&amp;":'').
+            "reply_id=$passed_id&amp;mailbox=$urlMailbox&amp;ent_num=$ent_num";
+
+if ($compose_new_win == '1') {
+    echo "<a href=\"javascript:void(0)\" onclick=\"comp_in_new(false,'$comp_uri')\"";
+} else {
+    echo '|&nbsp;<a href="' . $comp_uri .'"';
+}
+
     echo '>'.
     _("Reply All") .
-    '</A>&nbsp;&nbsp;' .
-                   '</SMALL>' .
-                '</TD>' .
-             '</TR>' .
-          '</TABLE>' .
-       '</TD></TR>' .
-       '<TR><TD WIDTH="100%">' .
-       '<TABLE WIDTH="100%" BORDER="0" CELLSPACING="0" CELLPADDING="3">' . "\n" .
-          '<TR>' . "\n";
+    '</a>&nbsp;&nbsp;' .
+                   '</small>' .
+                '</td>' .
+             '</tr>' .
+          '</table>' .
+       '</td></tr>' .
+       html_tag( 'tr' ) .
+       html_tag( 'td', '', 'left', '', 'width="100%"' ) .
+       html_tag( 'table', '', '', '', 'width="100%" border="0" cellspacing="0" cellpadding="3"' ) .
+       html_tag( 'tr' ) . "\n";
 
 /** subject **/
-echo          "<TD BGCOLOR=\"$color[0]\" WIDTH=\"10%\" ALIGN=\"right\" VALIGN=\"top\">\n" .
-    _("Subject:") .
-             "</TD><TD BGCOLOR=\"$color[0]\" WIDTH=\"80%\" VALIGN=\"top\">\n" .
-                "<B>$subject</B>&nbsp;\n" .
-             "</TD>\n" .
-             '<TD ROWSPAN="4" width="10%" BGCOLOR="' . $color[0] .
-    '" ALIGN=right VALIGN=top NOWRAP><small>'.
-    '<A HREF="' . $base_uri . "src/read_body.php?mailbox=$urlMailbox&amp;passed_id=$passed_id&amp;";
+echo html_tag( 'td', _("Subject:"), 'right', $color[0], 'width="10%" valign="top"' ) .
+        html_tag( 'td', '<b>' . $subject . '</b>&nbsp;' . "\n", 'left', $color[0], 'width="80%" valign="top"' ) .
+        html_tag( 'td', '', 'right', $color[0], 'rowspan="4" width="10%" valign="top" nowrap' ) .
+             '<a href="' . $base_uri . "src/read_body.php?mailbox=$urlMailbox&amp;passed_id=$passed_id&amp;";
 
 /* From a search... */
 if ($where && $what) {
     echo 'where=' . urlencode($where) . '&amp;what=' . urlencode($what) .
-         "&amp;view_hdr=1\">" . _("View Full Header") . "</A>\n";
+         "&amp;view_hdr=1\">" . _("View Full Header") . "</a>\n";
 } else {
     echo "startMessage=$startMessage&amp;show_more=$show_more&amp;view_hdr=1\">" .
-         _("View Full Header") . "</A>\n";
+         _("View Full Header") . "</a>\n";
 }
 
 /* Output the printer friendly link if we are in subtle mode. */
@@ -713,77 +811,68 @@ if ($pf_subtle_link) {
 }
 
 do_hook("read_body_header_right");
-echo '</small></TD>' .
-    ' </TR>';
+echo '</small></td>' .
+    ' </tr>';
 
 /** from **/
-echo       '<TR>' .
-             '<TD BGCOLOR="' . $color[0] . '" ALIGN="RIGHT">' .
-    _("From:") .
-             '</TD><TD BGCOLOR="' . $color[0] . '">' .
-                "<B>$from_name</B>&nbsp;\n" .
-             '</TD>' .
-          '</TR>';
+echo html_tag( 'tr', "\n" .
+            html_tag( 'td', _("From:"), 'right', $color[0], 'valign="top"' ) .
+            html_tag( 'td',
+                '<b>' . $from_name . '</b>&nbsp;' . "\n" ,
+            'left', $color[0] )
+       ) . "\n";
+       do_hook("read_body_after_from");
 /** date **/
-echo       '<TR>' . "\n" .
-             '<TD BGCOLOR="' . $color[0] . '" ALIGN="RIGHT">' . "\n" .
-    _("Date:") .
-             "</TD><TD BGCOLOR=\"$color[0]\">\n" .
-                "<B>$dateString</B>&nbsp;\n" .
-             '</TD>' . "\n" .
-          '</TR>' . "\n";
-
+echo html_tag( 'tr', "\n" .
+            html_tag( 'td', _("Date:"), 'right', $color[0], 'valign="top"' ) .
+            html_tag( 'td',
+                '<b>' . $dateString . '</b>&nbsp;' . "\n" ,
+            'left', $color[0] )
+       ) . "\n";
 /** to **/
-echo       "<TR>\n" .
-             "<TD BGCOLOR=\"$color[0]\" ALIGN=RIGHT VALIGN=TOP>\n" .
-    _("To:") .
-             '</TD><TD BGCOLOR="' . $color[0] . '" VALIGN="TOP">' . "\n" .
-                "<B>$to_string</B>&nbsp;\n" .
-             '</TD>' . "\n" .
-          '</TR>' . "\n";
+echo html_tag( 'tr', "\n" .
+            html_tag( 'td', _("To:"), 'right', $color[0], 'valign="top"' ) .
+            html_tag( 'td',
+                '<b>' . $to_string . '</b>&nbsp;' . "\n" ,
+            'left', $color[0] )
+       ) . "\n";
 /** cc **/
 if (isset($cc_string) && $cc_string <> '') {
-    echo       '<TR>' .
-                 "<TD BGCOLOR=\"$color[0]\" ALIGN=RIGHT VALIGN=TOP>" .
-                    'Cc:' .
-                 "</TD><TD BGCOLOR=\"$color[0]\" VALIGN=TOP colspan=2>" .
-                    "<B>$cc_string</B>&nbsp;" .
-                 '</TD>' .
-              '</TR>' . "\n";
+    echo html_tag( 'tr', "\n" .
+                html_tag( 'td', _("Cc:"), 'right', $color[0], 'valign="top"' ) .
+                html_tag( 'td',
+                    '<b>' . $cc_string . '</b>&nbsp;' . "\n" ,
+                'left', $color[0], 'colspan="2" valign="top"' )
+           ) . "\n";
 }
 
 /** bcc **/
 if (isset($bcc_string) && $bcc_string <> '') {
-    echo       '<TR>'.
-                 "<TD BGCOLOR=\"$color[0]\" ALIGN=RIGHT VALIGN=TOP>" .
-                    'Bcc:' .
-                 "</TD><TD BGCOLOR=\"$color[0]\" VALIGN=TOP colspan=2>" .
-                    "<B>$bcc_string</B>&nbsp;" .
-                 '</TD>' .
-              '</TR>' . "\n";
+    echo html_tag( 'tr', "\n" .
+                html_tag( 'td', _("Bcc:"), 'right', $color[0], 'valign="top"' ) .
+                html_tag( 'td',
+                    '<b>' . $bcc_string . '</b>&nbsp;' . "\n" ,
+                'left', $color[0], 'colspan="2" valign="top"' )
+           ) . "\n";
 }
 if ($default_use_priority && isset($priority_string) && $priority_string <> '' ) {
-    echo       '<TR>' .
-                 "<TD BGCOLOR=\"$color[0]\" ALIGN=RIGHT VALIGN=TOP>" .
-                       _("Priority") . ': '.
-                 "</TD><TD BGCOLOR=\"$color[0]\" VALIGN=TOP colspan=2>" .
-                    "<B>$priority_string</B>&nbsp;" .
-                 '</TD>' .
-              "</TR>" . "\n";
+    echo html_tag( 'tr', "\n" .
+                html_tag( 'td', _("Priority") . ':', 'right', $color[0], 'valign="top"' ) .
+                html_tag( 'td',
+                    '<b>' . $priority_string . '</b>&nbsp;' . "\n" ,
+                'left', $color[0], 'colspan="2" valign="top"' )
+           ) . "\n";
 }
 
 if ($show_xmailer_default) {
-    $read = sqimap_run_command ($imapConnection, "FETCH $passed_id BODY.PEEK[HEADER.FIELDS (X-Mailer User-Agent)]", true,
-                            $response, $readmessage);
-    $mailer = substr($read[1], strpos($read[1], " "));
+    $mailer = $header->xmailer;
     if (trim($mailer)) {
-        echo       '<TR>' .
-                     "<TD BGCOLOR=\"$color[0]\" ALIGN=RIGHT VALIGN=TOP>" .
-                           _("Mailer") . ': '.
-                     "</TD><TD BGCOLOR=\"$color[0]\" VALIGN=TOP colspan=2>" .
-                        "<B>$mailer</B>&nbsp;" .
-                     '</TD>' .
-                  "</TR>" . "\n";
+       echo html_tag( 'tr', "\n" .
+                   html_tag( 'td', _("Mailer") . ':', 'right', $color[0], 'valign="top"' ) .
+                   html_tag( 'td',
+                       '<b>' . $mailer . '</b>&nbsp;' ,
+                   'left', $color[0], 'colspan="2" valign="top"' )
+              ) . "\n";
     }
 }
 
@@ -797,69 +886,59 @@ if ($default_use_mdn) {
 
         // debug gives you the capability to remove mdn-flags
         // $MDNDebug = false;
-        $read = sqimap_run_command ($imapConnection, "FETCH $passed_id BODY.PEEK[HEADER.FIELDS (Disposition-Notification-To)]", true,
-                                $response, $readmessage);
-        $MDN_to = substr($read[1], strpos($read[1], ' '));
-        $MDN_flag_present = false;
 
-        $read = sqimap_run_command ($imapConnection, "FETCH $passed_id FLAGS", true,
-                                $response, $readmessage);
-
-        $MDN_flag_present = preg_match( '/.*\$MDNSent/i', $read[0]);
-
-        if (trim($MDN_to) &&
-            (!isset( $sendreceipt ) || $sendreceipt == '' )  ) {
+        if ($header->dnt) {
+          $MDN_to_o = $header->dnt;
+          $MDN_to = $MDN_to_o->getAddress();
+       } else {
+          $MDN_to = '';   
+        }
 
-            if ( $MDN_flag_present && $supportMDN) {
+        if ($MDN_to && (!isset( $sendreceipt ) || $sendreceipt == '' )  ) {
+            if ( $message->is_mdnsent && $supportMDN) {
                 $sendreceipt = 'removeMDN';
+
                 $url = "\"read_body.php?mailbox=$mailbox&amp;passed_id=$passed_id&amp;startMessage=$startMessage&amp;show_more=$show_more&amp;sendreceipt=$sendreceipt\"";
                 $sendreceipt='';
                 /*
                 if ($MDNDebug ) {
-                    echo       '<TR>' .
-                                 "<TD BGCOLOR=\"$color[9]\"  ALIGN=RIGHT VALIGN=TOP>" .
-                                       _("Read receipt") . ': ' .
-                                 "</TD><TD BGCOLOR=\"$color[9]\" VALIGN=TOP colspan=2>" .
-                                    '<B>' .
-                                    _("send") .
-                                    "</B> <a href=$url>[" . _("Remove MDN flag") . ']  </a>'  .
-                                 '</TD>' .
-                             '</TR>' . "\n";
+                    echo html_tag( 'tr', "\n" .
+                                html_tag( 'td', _("Read receipt") . ':', 'right', $color[9], 'valign="top"' ) .
+                                html_tag( 'td',
+                                    '<b>' . _("send") . '</b>&nbsp;<a href="' . $url . '">[' . _("Remove MDN flag") . ']  </a>&nbsp;' ,
+                                'left', $color[9], 'colspan="2" valign="top"' )
+                            ) . "\n";
                 } else {
                 */
-                echo       '<TR>' .
-                             "<TD BGCOLOR=\"$color[9]\"  ALIGN=RIGHT VALIGN=TOP>" .
-                                   _("Read receipt") . ': ' .
-                             "</TD><TD BGCOLOR=\"$color[9]\" VALIGN=TOP colspan=2>" .
-                                '<B>'._("send").'</B>'.
-                             '</TD>' .
-                         '</TR>' . "\n";
+                echo html_tag( 'tr', "\n" .
+                            html_tag( 'td', _("Read receipt") . ':', 'right', $color[9], 'valign="top"' ) .
+                                html_tag( 'td',
+                                    '<b>' . _("send") . '</b>&nbsp;' ,
+                                'left', $color[9], 'colspan="2" valign="top"' )
+                            ) . "\n";
                 /*
                 }
                 */
 
             } // when deleted or draft flag is set don't offer to send a MDN response
-            else if ( ereg('\\Draft',$read[0] || ereg('\\Deleted',$read[0])) ) {
-                echo       '<TR>' .
-                            "<TD BGCOLOR=\"$color[9]\"  ALIGN=RIGHT VALIGN=TOP>" .
-                                _("Read receipt") . ': '.
-                            "</TD><TD BGCOLOR=\"$color[9]\" VALIGN=TOP colspan=2>" .
-                                '<B>' . _("requested") . "</B>" .
-                            '</TD>' .
-                        '</TR>' . "\n";
+            else if ( $message->is_draft || $message->is_deleted)  {
+                echo html_tag( 'tr', "\n" .
+                            html_tag( 'td', _("Read receipt") . ':', 'right', $color[9], 'valign="top"' ) .
+                                html_tag( 'td',
+                                    '<b>' . _("requested") . '</b>&nbsp;' ,
+                                'left', $color[9], 'colspan="2" valign="top"' )
+                            ) . "\n";
             }
             // if no MDNsupport don't use the annoying popup messages
             else if (  !$FirstTimeSee ) {
                 $sendreceipt = 'send';
                 $url = "\"read_body.php?mailbox=$mailbox&passed_id=$passed_id&startMessage=$startMessage&show_more=$show_more&sendreceipt=$sendreceipt\"";
-                echo       '<TR>' .
-                            "<TD BGCOLOR=\"$color[9]\"  ALIGN=RIGHT VALIGN=TOP>" .
-                                _("Read receipt") . ': ' .
-                            "</TD><TD BGCOLOR=\"$color[9]\" VALIGN=TOP colspan=2>" .
-                                '<B>' . _("requested") .
-                                "</B> &nbsp; <a href=$url>[" . _("Send read receipt now") . "]</a>" .
-                            '</TD>' .
-                        '</TR>' . "\n";
+                echo html_tag( 'tr', "\n" .
+                            html_tag( 'td', _("Read receipt") . ':', 'right', $color[9], 'valign="top"' ) .
+                                html_tag( 'td',
+                                    '<b>' . _("requested") . '</b> &nbsp; <a href="' . $url . '">[' . _("Send read receipt now") . ']</a>',
+                                'left', $color[9], 'colspan="2" valign="top"' )
+                            ) . "\n";
                 $sendreceipt='';
             }
             else {
@@ -871,20 +950,17 @@ if ($default_use_mdn) {
                     "               if (window.confirm(\"" .
                     _("The message sender has requested a response to indicate that you have read this message. Would you like to send a receipt?") .
                     "\")) {  \n" .
-                    "                       window.location=($url); \n" .
-                    '                       window.reload()' . "\n" .
+                    '                       window.open('.$url.',"right");' . "\n" .
                     '               }' . "\n" .
                     '// -->' . "\n" .
                     '</script>' . "\n";
                 }
-                echo       '<TR>' .
-                            "<TD BGCOLOR=\"$color[9]\"  ALIGN=RIGHT VALIGN=TOP>" .
-                                    _("Read receipt") . ': ' .
-                            "</TD><TD BGCOLOR=\"$color[9]\" VALIGN=TOP colspan=2>" .
-                                '<B>' . _("requested") . "&nbsp&nbsp</B><a href=$url>" . '[' .
-                                _("Send read receipt now") . ']  </a>' ." \n" .
-                            '</TD>' .
-                            '</TR>' . "\n";
+                echo html_tag( 'tr', "\n" .
+                            html_tag( 'td', _("Read receipt") . ':', 'right', $color[9], 'valign="top"' ) .
+                                html_tag( 'td',
+                                    '<b>' . _("requested") . '</b>&nbsp&nbsp<a href="' . $url . '">[' . _("Send read receipt now") . ']</a>',
+                                'left', $color[9], 'colspan="2" valign="top"' )
+                            ) . "\n";
                 $sendreceipt = '';
             }
         }
@@ -898,36 +974,36 @@ if ($default_use_mdn) {
                     $final_recipient = getPref($data_dir, $username, 'email_address', '' );
                 }
 
+
                 $final_recipient = trim($final_recipient);
                 if ($final_recipient == '' ) {
                     $final_recipient = getPref($data_dir, $username, 'email_address', '' );
                 }
 
-                if ( SendMDN( $MDN_to, $final_recipient ) > 0 && $supportMDN ) {
+                if ( SendMDN( $MDN_to, $final_recipient, $message ) > 0 && $supportMDN ) {
                     ToggleMDNflag( true);
                 }
+               ClearAttachments();
             }
             $sendreceipt = 'removeMDN';
             $url = "\"read_body.php?mailbox=$mailbox&amp;passed_id=$passed_id&amp;startMessage=$startMessage&amp;show_more=$show_more&amp;sendreceipt=$sendreceipt\"";
             $sendreceipt='';
-                       /*
+            /*
             if ($MDNDebug && $supportMDN) {
-            echo "      <TR>\n" .
-                    "         <TD BGCOLOR=\"$color[9]\"  ALIGN=RIGHT VALIGN=TOP>\n" .
-                    "            "._("Read receipt").": \n".
-                    "         </TD><TD BGCOLOR=\"$color[9]\" VALIGN=TOP colspan=2>\n" .
-                    '            <B>'._("send").'</B>'." <a href=$url>" . '[' . _("Remove MDN flag") . ']  </a>'  . "\n" .
-                    '         </TD>' . "\n" .
-                    '     </TR>' . "\n";
+            echo html_tag( 'tr', "\n" .
+                       html_tag( 'td', _("Read receipt") . ':', 'right', $color[9], 'valign="top"' ) .
+                           html_tag( 'td',
+                               '<b>' . _("send") . '</b>&nbsp&nbsp<a href="' . $url . '">[' . _("Remove MDN flag") . ']</a>',
+                           'left', $color[9], 'colspan="2" valign="top"' )
+                       ) . "\n";
             } else {
             */
-            echo "      <TR>\n" .
-                    "         <TD BGCOLOR=\"$color[9]\"  ALIGN=RIGHT VALIGN=TOP>\n" .
-                    "            "._("Read receipt").": \n".
-                    "         </TD><TD BGCOLOR=\"$color[9]\" VALIGN=TOP colspan=2>\n" .
-                    '            <B>'._("send").'</B>'. "\n" .
-                    '         </TD>' . "\n" .
-                    '     </TR>' . "\n";
+            echo html_tag( 'tr', "\n" .
+                       html_tag( 'td', _("Read receipt") . ':', 'right', $color[9], 'valign="top"' ) .
+                           html_tag( 'td',
+                               '<b>' . _("send") . '</b>&nbsp',
+                           'left', $color[9], 'colspan="2" valign="top"' )
+                       ) . "\n";
             /*
             }
             */
@@ -937,14 +1013,12 @@ if ($default_use_mdn) {
 
             $sendreceipt = 'send';
                 $url = "\"read_body.php?mailbox=$mailbox&amp;passed_id=$passed_id&amp;startMessage=$startMessage&amp;show_more=$show_more&amp;sendreceipt=$sendreceipt\"";
-                echo       '<TR>'.
-                              "<TD BGCOLOR=\"$color[9]\"  ALIGN=RIGHT VALIGN=TOP>" .
-                                    _("Read receipt") . ': ' .
-                              "</TD><TD BGCOLOR=\"$color[9]\" VALIGN=TOP colspan=2>" .
-                                 '<B>' . _("requested") .
-                                 "</B> &nbsp; <a href=$url>[" . _("Send read receipt now") . "]</a>" .
-                              '</TD>' .
-                            '</TR>' . "\n";
+                echo html_tag( 'tr', "\n" .
+                           html_tag( 'td', _("Read receipt") . ':', 'right', $color[9], 'valign="top"' ) .
+                           html_tag( 'td',
+                               '<b>' . _("requested") . '</b> &nbsp; <a href="' . $url . '">[' . _("Send read receipt now") . ']</a>',
+                           'left', $color[9], 'colspan="2" valign="top"' )
+                       ) . "\n";
             $sendreceipt = '';
 
         }
@@ -953,19 +1027,23 @@ if ($default_use_mdn) {
 
 do_hook('read_body_header');
 
-echo '</TABLE>' .
-    '   </TD></TR>' .
-    '</TABLE>';
+echo '</table>' .
+    '   </td></tr>' .
+    '</table>';
 flush();
-
-echo "<TABLE CELLSPACING=0 WIDTH=\"97%\" BORDER=0 ALIGN=CENTER CELLPADDING=0>\n" .
-    "   <TR><TD BGCOLOR=\"$color[4]\" WIDTH=\"100%\">\n" .
-    '<BR>'.
-    formatBody($imapConnection, $message, $color, $wrap_at).
-    '</TD></TR></TABLE>' .
-    '<TABLE CELLSPACING="0" WIDTH="100%" BORDER="0" ALIGN="CENTER" CELLPADDING="0">' . "\n" .
-    "   <TR><TD BGCOLOR=\"$color[9]\">&nbsp;</TD></TR>" .
-    '</TABLE>' . "\n";
+echo html_tag( 'table', "\n" .
+            html_tag( 'tr', "\n" .
+                html_tag( 'td', '<br>' . "\n" . $body . "\n", 'left', $color[4]
+                )
+            ) ,
+        'center', '', 'cellspacing=0 width="97%" border="0" cellpadding="0"') .
+
+        html_tag( 'table', "\n" .
+                   html_tag( 'tr', "\n" .
+                       html_tag( 'td', '&nbsp;', 'left', $color[9]
+                       )
+                   ) ,
+        'center', '', 'cellspacing=0 width="100%" border="0" cellpadding="0"');
 
 /* show attached images inline -- if pref'fed so */
 if (($attachment_common_show_images) &&
@@ -979,14 +1057,12 @@ if (($attachment_common_show_images) &&
                 '&amp;passed_ent_id=' . urlencode($img['ent_id']) .
                 '&amp;absolute_dl=true';
 
-        echo "<TABLE BORDER=0 CELLSPACING=0 CELLPADDING=2 ALIGN=CENTER>\n" .
-              '<TR>' .
-                '<TD>' .
-                  "<img src=\"$imgurl\">\n" .
-                "</TD>\n" .
-              "</TR>\n" .
-            "</TABLE>\n";
-
+        echo html_tag( 'table', "\n" .
+                   html_tag( 'tr', "\n" .
+                       html_tag( 'td', '<img src="' . $imgurl . '">' ."\n", 'left'
+                       )
+                   ) ,
+        'center', '', 'cellspacing=0 border="0" cellpadding="2"');
     }
 }