projects / squirrelmail.git / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | inline | side by side
Formatting cleanups
[squirrelmail.git] / src / read_body.php
diff --git a/src/read_body.php b/src/read_body.php
index fca049e62e2b156e4bbfd443db85bf9e3b232bea..a85361f8664bd48a488d01f41cc728dd4fca7bdb 100644 (file)
--- a/src/read_body.php
+++ b/src/read_body.php
@@ -23,7 +23,6 @@ require_once(SM_PATH . 'functions/date.php');
 require_once(SM_PATH . 'functions/url_parser.php');
 require_once(SM_PATH . 'functions/html.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
@@ -121,18 +120,18 @@ function printer_friendly_link($mailbox, $passed_id, $passed_ent_id, $color) {
     $result = '';
     /* Output the link. */
     if ($javascript_on) {
-        $result .= '<script language="javascript" type="text/javascript">' . "\n" .
-                   '<!--' . "\n" .
-                   "  function printFormat() {\n" .
-                   '    window.open("../src/printer_friendly_main.php' .
-                   $params . '","Print","width=800,height=600");' . "\n".
-                   "  }\n" .
-                   "// -->\n" .
-                   "</script>\n" .
-                   "<a href=\"javascript:printFormat();\">$print_text</a>\n";
+        $result = '<script language="javascript" type="text/javascript">' . "\n" .
+                  '<!--' . "\n" .
+                  "  function printFormat() {\n" .
+                  '    window.open("../src/printer_friendly_main.php' .
+                  $params . '","Print","width=800,height=600");' . "\n".
+                  "  }\n" .
+                  "// -->\n" .
+                  "</script>\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";
     }
     return $result;
 }
@@ -306,7 +305,6 @@ function SendMDN ( $mailbox, $passed_id, $sender, $message, $imapConnection) {
     return $success;
 }
 
-
 function ToggleMDNflag ($set ,$imapConnection, $mailbox, $passed_id, $uid_support) {
     $sg   =  $set?'+':'-';
     $cmd  = 'STORE ' . $passed_id . ' ' . $sg . 'FLAGS ($MDNSent)';
@@ -445,8 +443,8 @@ function formatEnvheader($mailbox, $passed_id, $passed_ent_id, $message,
         }
     }
 
-    $s  = '<TABLE WIDTH="100%" CELLPADDING="0" CELLSPACING="0" BORDER="0"';
-    $s .= ' ALIGN="center" BGCOLOR="' . $color[0] . '">';
+    $s  = '<TABLE WIDTH="100%" CELLPADDING="0" CELLSPACING="2" BORDER="0"';
+    $s .= ' ALIGN="center" BGCOLOR="'.$color[0].'">';
     foreach ($env as $key => $val) {
         if ($val) {
             $s .= '<TR>';
@@ -455,10 +453,16 @@ function formatEnvheader($mailbox, $passed_id, $passed_ent_id, $message,
             $s .= '</TR>';
         }
     }
+    echo '<TABLE BGCOLOR="'.$color[9].'" WIDTH="100%" CELLPADDING="1"'.
+         ' CELLSPACING="0" BORDER="0" ALIIGN="center">'."\n";
+    echo '<TR><TD HEIGHT="5" COLSPAN="2" BGCOLOR="'.
+          $color[4].'"></TD></TR><TR><TD align=center>'."\n";
     echo $s;
     do_hook("read_body_header");
     formatToolbar($mailbox, $passed_id, $passed_ent_id, $message, $color);
     echo '</TABLE>';
+    echo '</TD></TR><TR><TD HEIGHT="5" COLSPAN="2" BGCOLOR="'.$color[4].'"></TD></TR>'."\n";
+    echo '</TABLE>';
 }
 
 function formatMenubar($mailbox, $passed_id, $passed_ent_id, $message, $mbx_response) {
@@ -482,7 +486,6 @@ function formatMenubar($mailbox, $passed_id, $passed_ent_id, $message, $mbx_resp
         $msgs_str  = _("Message List");
     }
     $s .= '<a href="' . $msgs_url . '">' . $msgs_str . '</a>';
-    $s .= $topbar_delimiter;
 
     $delete_url = $base_uri . 'src/delete_message.php?mailbox=' . $urlMailbox .
                   '&amp;message=' . $passed_id . '&amp;';
@@ -492,6 +495,7 @@ function formatMenubar($mailbox, $passed_id, $passed_ent_id, $message, $mbx_resp
         } else {
             $delete_url .= 'sort=' . $sort . '&amp;startMessage=' . $startMessage;
         }
+        $s .= $topbar_delimiter;
         $s .= '<a href="' . $delete_url . '">' . _("Delete") . '</a>';
     }
 
@@ -545,6 +549,32 @@ function formatMenubar($mailbox, $passed_id, $passed_ent_id, $message, $mbx_resp
         /* code for navigating through attached message/rfc822 messages */
         $url = set_url_var($PHP_SELF, 'passed_ent_id',0);
         $s .= '<a href="'.$url.'">'._("View Message").'</a>';
+        $entities     = array();
+        $entity_count = array();
+        $c = 0;
+
+        foreach($message->parent->entities as $ent) {
+            if ($ent->type0 == 'message' && $ent->type1 == 'rfc822') {
+                $c++;
+                $entity_count[$c] = $ent->entity_id;
+                $entities[$ent->entity_id] = $c;
+            }
+        }
+        $prev_link = _("Previous");
+        $next_link = _("Next");
+        if($entities[$passed_ent_id] > 1) {
+            $prev_ent_id = $entity_count[$entities[$passed_ent_id] - 1];
+            $prev_link   = '<a href="'
+                         . set_url_var($PHP_SELF, 'passed_ent_id', $prev_ent_id)
+                         . '">' . $prev_link . '</a>';
+        }
+        if($entities[$passed_ent_id] < $c) {
+            $next_ent_id = $entity_count[$entities[$passed_ent_id] + 1];
+            $next_link   = '<a href="'
+                         . set_url_var($PHP_SELF, 'passed_ent_id', $next_ent_id)
+                         . '">' . $next_link . '</a>';
+        }
+        $s .= $topbar_delimiter . $prev_link;
         $par_ent_id = $message->parent->entity_id;
         if ($par_ent_id) {
             $par_ent_id = substr($par_ent_id,0,-2);
@@ -552,24 +582,25 @@ function formatMenubar($mailbox, $passed_id, $passed_ent_id, $message, $mbx_resp
             $url = set_url_var($PHP_SELF, 'passed_ent_id',$par_ent_id);
             $s .= '<a href="'.$url.'">'._("Up").'</a>';
         }
+        $s .= $topbar_delimiter . $next_link;
     }
 
     $s .= '</small></td>' . "\n" . '<td align="right" width="33%" nowrap><small>';
     $comp_action_uri = $comp_uri . '&amp;action=forward';
     $s .= $link_open . $comp_action_uri . $link_close . _("Forward") . '</a>';
-    $s .= $topbar_delimiter;
 
     if ($enable_forward_as_attachment) {
         $comp_action_uri = $comp_uri . '&amp;action=forward_as_attachment';
-        $s .= $link_open . $comp_action_uri . $link_close . _("Forward as Attachment") . '</a>';
         $s .= $topbar_delimiter;
+        $s .= $link_open . $comp_action_uri . $link_close . _("Forward as Attachment") . '</a>';
     }
 
     $comp_action_uri = decodeHeader($comp_uri . '&amp;action=reply');
-    $s .= $link_open . $comp_action_uri . $link_close . _("Reply") . '</a>';
     $s .= $topbar_delimiter;
+    $s .= $link_open . $comp_action_uri . $link_close . _("Reply") . '</a>';
 
     $comp_action_uri = $comp_uri . '&amp;action=reply_all';
+    $s .= $topbar_delimiter;
     $s .= $link_open . $comp_action_uri . $link_close . _("Reply All") . '</a>';
     $s .= '</small></td></tr></table>';
     do_hook("read_body_menu_top");
@@ -605,12 +636,12 @@ function formatToolbar($mailbox, $passed_id, $passed_ent_id, $message, $color) {
 
 /* get the globals we may need */
 
-$username = $_SESSION['username'];
-$key = $_COOKIE['key'];
+$username   = $_SESSION['username'];
+$key        = $_COOKIE['key'];
 $onetimepad = $_SESSION['onetimepad'];
-$msgs = $_SESSION['msgs'];
-$base_uri = $_SESSION['base_uri'];
-$delimiter = $_SESSION['delimiter'];
+$msgs       = $_SESSION['msgs'];
+$base_uri   = $_SESSION['base_uri'];
+$delimiter  = $_SESSION['delimiter'];
 
 if (isset($_GET['passed_id'])) {
     $passed_id = $_GET['passed_id'];
@@ -697,7 +728,7 @@ if (isset($mailbox)) {
 }
 
 $imapConnection = sqimap_login($username, $key, $imapServerAddress, $imapPort, 0);
-$mbx_response = sqimap_mailbox_select($imapConnection, $mailbox, false, false, true);
+$mbx_response   = sqimap_mailbox_select($imapConnection, $mailbox, false, false, true);
 
 
 /**
@@ -706,10 +737,10 @@ $mbx_response = sqimap_mailbox_select($imapConnection, $mailbox, false, false, t
  */
 
 $uidvalidity = $mbx_response['UIDVALIDITY'];
- 
+
 if (!isset($messages[$uidvalidity])) {
    $messages[$uidvalidity] = array();
-}  
+}
 if (!isset($messages[$uidvalidity][$passed_id]) || !$uid_support) {
    $message = sqimap_get_message($imapConnection, $passed_id, $mailbox);
    $FirstTimeSee = !$message->is_seen;
@@ -767,8 +798,8 @@ if (isset($sendreceipt)) {
 /***********************************************/
 
 $msgs[$passed_id]['FLAG_SEEN'] = true;
- 
-$messagebody = ''; 
+
+$messagebody = '';
 do_hook('read_body_top');
 if ($show_html_default == 1) {
     $ent_ar = $message->findDisplayEntity(array());
@@ -792,20 +823,24 @@ echo '    <table width="100%" cellpadding="1" cellspacing="0" align="center" bor
 echo '      <tr><td>';
 echo '        <table width="100%" cellpadding="3" cellspacing="0" align="center" border="0">';
 echo '          <tr bgcolor="'.$color[4].'"><td>';
-echo '            <table cellpadding="5" cellspacing="0"  border="0">';
-echo '              <tr>'. html_tag( 'td', '<br>'. $messagebody."\n", 'left').'</td>';
+echo '            <table cellpadding="1" cellspacing="5" align="left" border="0">';
+echo '              <tr>' . html_tag( 'td', '<br>'. $messagebody."\n", 'left')
+                        . '</tr>';
 echo '            </table>';
 echo '          </td></tr>';      
 echo '        </table></td></tr>';
 echo '    </table>';
 echo '  </td></tr>';
 
+echo '<TR><TD HEIGHT="5" COLSPAN="2" BGCOLOR="'.
+          $color[4].'"></TD></TR>'."\n";
+
 $attachmentsdisplay = formatAttachments($message,$ent_ar,$mailbox, $passed_id);
 if ($attachmentsdisplay) {
    echo '  <tr><td>';
    echo '    <table width="100%" cellpadding="1" cellspacing="0" align="center"'.' border="0" bgcolor="'.$color[9].'">';
    echo '     <tr><td>';
-   echo '       <table width="100%" cellpadding="1" cellspacing="0" align="center" border="0" bgcolor="'.$color[4].'">';
+   echo '       <table width="100%" cellpadding="0" cellspacing="0" align="center" border="0" bgcolor="'.$color[4].'">';
    echo '        <tr><td ALIGN="left" bgcolor="'.$color[9].'">';
    echo '           <b>' . _("Attachments") . ':</b>';
    echo '        </td></tr>';
@@ -813,10 +848,11 @@ if ($attachmentsdisplay) {
    echo '          <table width="100%" cellpadding="2" cellspacing="2" align="center"'.' border="0" bgcolor="'.$color[0].'"><tr><td>';
    echo              $attachmentsdisplay;
    echo '          </td></tr></table>';
-   echo '        </table></td></tr>';
-   echo '       </table></td></tr>';
-   echo '    </table>';
+   echo '       </td></tr></table>';
+   echo '    </td></tr></table>';
    echo '  </td></tr>';
+   echo '<TR><TD HEIGHT="5" COLSPAN="2" BGCOLOR="'.
+          $color[4].'"></TD></TR>';
 }
 echo '</table>';
 
Unnamed repository; edit this file 'description' to name the repository.