Changeset 5852c14d in github

Timestamp:

Jun 11, 2010 9:57:02 AM (3 years ago)

Author:

alecpl <alec@…>

Branches:

master, HEAD, courier-fix, dev-browser-capabilities, pdo, release-0.6, release-0.7, release-0.8

Children:

0133e6b

Parents:

93e3ae1

Message:

• Add option to set separate footer for HTML messages (#1486660)

Files:

• CHANGELOG (modified) (1 diff)
• config/main.inc.php.dist (modified) (1 diff)
• program/steps/mail/sendmail.inc (modified) (5 diffs)

CHANGELOG

@@ -2 +2 @@
 ===========================
 
+- Add option to set separate footer for HTML messages (#1486660)
 - Add real SMTP error description to displayed error messages (#1485927)
 - Fix some IMAP errors handling when opening the message (#1485443)

config/main.inc.php.dist

@@ -225 +225 @@
 // paths are relative to the RoundCube root folder
 $rcmail_config['generic_message_footer'] = '';
+
+// path to a text file which will be added to each sent HTML message
+// paths are relative to the RoundCube root folder
+$rcmail_config['generic_message_footer_html'] = '';
 
 // add a received header to outgoing mails containing the creators IP and hostname

program/steps/mail/sendmail.inc

@@ -385 +385 @@
         '<blockquote type="cite" style="'.$bstyle.'">', $message_body);
   }
+
   // generic footer for all messages
-  if (!empty($CONFIG['generic_message_footer'])) {
+  if ($isHtml && !empty($CONFIG['generic_message_footer_html'])) {
+      $footer = file_get_contents(realpath($CONFIG['generic_message_footer_html']));
+      $footer = rcube_charset_convert($footer, RCMAIL_CHARSET, $message_charset);
+  }
+  else if (!empty($CONFIG['generic_message_footer'])) {
     $footer = file_get_contents(realpath($CONFIG['generic_message_footer']));
     $footer = rcube_charset_convert($footer, RCMAIL_CHARSET, $message_charset);
-  }
+    if ($isHtml)
+      $footer = '<pre>'.$footer.'</pre>';
+  }
+  if ($footer)
+    $message_body .= "\r\n" . $footer;
 }
 
@@ -423 +432 @@
     array('body' => $message_body, 'type' => 'html', 'message' => $MAIL_MIME));
 
-  $MAIL_MIME->setHTMLBody($plugin['body'] . ($footer ? "\r\n<pre>".$footer.'</pre>' : ''));
+  $MAIL_MIME->setHTMLBody($plugin['body']);
 
   // add a plain text version of the e-mail as an alternative part.
   $h2t = new html2text($plugin['body'], false, true, 0);
-  $plainTextPart = rc_wordwrap($h2t->get_text(), $LINE_LENGTH, "\r\n") . ($footer ? "\r\n".$footer : '');
+  $plainTextPart = rc_wordwrap($h2t->get_text(), $LINE_LENGTH, "\r\n");
   $plainTextPart = wordwrap($plainTextPart, 998, "\r\n", true);
-  if (!strlen($plainTextPart)) {
+  if (!$plainTextPart) {
     // empty message body breaks attachment handling in drafts 
     $plainTextPart = "\r\n"; 
@@ -448 +457 @@
 }
 else {
-  if ($footer)
-    $message_body .= "\r\n" . $footer;
-  
+  $plugin = $RCMAIL->plugins->exec_hook('outgoing_message_body',
+    array('body' => $message_body, 'type' => 'plain', 'message' => $MAIL_MIME));
+
+  $message_body = $plugin['body'];
+
   // compose format=flowed content if enabled and not a reply message
   if (empty($_SESSION['compose']['reply_msgid']) && ($flowed = $RCMAIL->config->get('send_format_flowed', true)))
@@ -456 +467 @@
   else
     $message_body = rc_wordwrap($message_body, $LINE_LENGTH, "\r\n");
-  
+
   $message_body = wordwrap($message_body, 998, "\r\n", true);
   if (!strlen($message_body)) { 
@@ -463 +474 @@
   }
 
-  $plugin = $RCMAIL->plugins->exec_hook('outgoing_message_body',
-    array('body' => $message_body, 'type' => 'plain', 'message' => $MAIL_MIME));
-
-  $MAIL_MIME->setTXTBody($plugin['body'], false, true);
+  $MAIL_MIME->setTXTBody($message_body, false, true);
 }