From dddc985ce24a9769e3b1a071b5d2fc654f56f4f7 Mon Sep 17 00:00:00 2001
From: Aleksander Machniak <alec@alec.pl>
Date: Sun, 16 Aug 2015 11:50:23 -0400
Subject: [PATCH] Make Mail_mime>=1.9.0 a requirement, fix compat. errors
---
program/lib/Roundcube/rcube.php | 21 ++++++---------------
INSTALL | 2 +-
composer.json-dist | 4 ++--
3 files changed, 9 insertions(+), 18 deletions(-)
diff --git a/INSTALL b/INSTALL
index f6a6746..f792687 100644
--- a/INSTALL
+++ b/INSTALL
@@ -17,7 +17,7 @@
- Libiconv, Zip (recommended)
- OpenSSL, Fileinfo, Mcrypt, mbstring (optional)
* PEAR packages distributed with Roundcube or external:
- - Mail_Mime 1.8.1 or newer
+ - Mail_Mime 1.9.0 or newer
- Net_SMTP (latest from https://github.com/pear/Net_SMTP/)
- Net_IDNA2 0.1.1 or newer
- Auth_SASL 1.0.6 or newer
diff --git a/composer.json-dist b/composer.json-dist
index 762dc6a..635ddba 100644
--- a/composer.json-dist
+++ b/composer.json-dist
@@ -34,8 +34,8 @@
"pear-pear.php.net/auth_sasl": ">=1.0.6",
"pear-pear.php.net/net_idna2": ">=0.1.1",
"pear-pear.php.net/net_sieve": ">=1.3.2",
- "pear/mail_mime": ">=1.8.9",
- "pear/net_smtp": "dev-master",
+ "pear-pear.php.net/mail_mime": ">=1.9.0",
+ "pear-pear.php.net/net_smtp": "dev-master",
"patchwork/utf8": "1.2.x"
},
"require-dev": {
diff --git a/program/lib/Roundcube/rcube.php b/program/lib/Roundcube/rcube.php
index 80fc2a0..1e3b366 100644
--- a/program/lib/Roundcube/rcube.php
+++ b/program/lib/Roundcube/rcube.php
@@ -1670,12 +1670,8 @@
if (strlen($headers['Bcc']))
$a_recipients[] = $headers['Bcc'];
- // clean Bcc from header for recipients
- $send_headers = $headers;
- unset($send_headers['Bcc']);
- // here too, it because txtHeaders() below use $message->_headers not only $send_headers
- unset($message->_headers['Bcc']);
-
+ // remove Bcc header and get the whole head of the message as string
+ $send_headers = array('Bcc' => null);
$smtp_headers = $message->txtHeaders($send_headers, true);
if ($message->getParam('delay_file_io')) {
@@ -1717,13 +1713,9 @@
// send mail using PHP's mail() function
else {
// unset some headers because they will be added by the mail() function
- $headers_enc = $message->headers($headers);
- $headers_php = $message->_headers;
- unset($headers_php['To'], $headers_php['Subject']);
-
- // reset stored headers and overwrite
- $message->_headers = array();
- $header_str = $message->txtHeaders($headers_php);
+ $headers_enc = $headers;
+ $headers_res = array('To' => null, 'Subject' => null);
+ $header_str = $message->txtHeaders($headers_res, true);
// #1485779
if (strtoupper(substr(PHP_OS, 0, 3)) === 'WIN') {
@@ -1796,8 +1788,7 @@
fclose($msg_body);
}
- $message->_headers = array();
- $message->headers($headers);
+ $message->headers($headers, true);
return $sent;
}
--
Gitblit v1.9.1