From 5caf2d0cef47f1057b5d4cd5f7a508d4a7692252 Mon Sep 17 00:00:00 2001
From: stekkel <stekkel@7612ce4b-ef26-0410-bec9-ea0150e637f0>
Date: Wed, 31 Jul 2002 16:47:55 +0000
Subject: [PATCH] Fix for incorrect header in case of message/rfc822
 attachments. Thnx Wouter.

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@3202 7612ce4b-ef26-0410-bec9-ea0150e637f0
---
 src/printer_friendly_bottom.php | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/src/printer_friendly_bottom.php b/src/printer_friendly_bottom.php
index 9f5a0978..0ff5dd9d 100644
--- a/src/printer_friendly_bottom.php
+++ b/src/printer_friendly_bottom.php
@@ -27,30 +27,30 @@ $mailbox = urldecode($mailbox);
 $imapConnection = sqimap_login($username, $key, $imapServerAddress, $imapPort, 0);
 sqimap_mailbox_select($imapConnection, $mailbox);
 $message = sqimap_get_message($imapConnection, $passed_id, $mailbox);
+$id = $passed_id;
+if (isset($passed_ent_id)) {
+   $message = $message->getEntity($passed_ent_id);
+}
+
 
 /* --start display setup-- */
 
+
 /* From and Date are usually fine as they are... */
 $from = decodeHeader($message->header->getAddr_s('from'));
 $date = getLongDateString($message->header->date);
+$subject = trim(decodeHeader($message->header->subject));
 
 /* we can clean these up if the list is too long... */
 $cc = decodeHeader($message->header->getAddr_s('cc'));
 $to = decodeHeader($message->header->getAddr_s('to'));
-//$cc = decodeHeader(getLineOfAddrs($message->header->cc));
-//$to = decodeHeader(getLineOfAddrs($message->header->to));
 
-/* and Body and Subject could easily stream off the page... */
-$id = $passed_id;
-if (isset($passed_ent_id)) {
-   $message = $message->getEntity($passed_ent_id);
-}
 $ent_ar = $message->findDisplayEntity();
-//$ent_num = $ent_ar[0];
 $body = '';
 if ($ent_ar[0] != '') {
   for ($i = 0; $i < count($ent_ar); $i++) {
      $body .= formatBody($imapConnection, $message, $color, $wrap_at, $ent_ar[$i], $passed_id, $mailbox);
+     $body .= '<hr noshade size=1>';
   }
   $hookResults = do_hook('message_body', $body);
   $body = $hookResults[1];
@@ -58,8 +58,6 @@ if ($ent_ar[0] != '') {
   $body = _("Message not printable");
 }
 
-$subject = trim(decodeHeader($message->header->subject));
-
  /* now, if they choose to, we clean up the display a bit... */
  
 if ( empty($pf_cleandisplay) || $pf_cleandisplay != 'no' ) {
@@ -71,7 +69,9 @@ if ( empty($pf_cleandisplay) || $pf_cleandisplay != 'no' ) {
     $to = pf_clean_string(str_replace(',,', ',', $to), $num_leading_spaces);
 
      // the body should have no leading zeros
-    $body = pf_clean_string($body, 0);
+    // disabled because it destroys html mail
+
+//    $body = pf_clean_string($body, 0);
 
      // clean up everything else...
     $subject = pf_clean_string($subject, $num_leading_spaces);
-- 
2.25.1