summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authoralecpl <alec@alec.pl>2009-12-04 10:50:00 +0000
committeralecpl <alec@alec.pl>2009-12-04 10:50:00 +0000
commit19d6a89342cbbeff0d20884c177d996d249e445a (patch)
treed6c08ec7e8ae9934b0fe89b7c4d768658352739c
parent9a0f89d7af6cc815f21580049767cf721317d857 (diff)
- fix r3150 (better detection that message/rfc822 should be displayed as message/aternative)
-rw-r--r--CHANGELOG2
-rw-r--r--program/include/rcube_message.php20
2 files changed, 17 insertions, 5 deletions
diff --git a/CHANGELOG b/CHANGELOG
index 0971a185c..9d15fddca 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -1,7 +1,7 @@
CHANGELOG RoundCube Webmail
===========================
-- Fix displaying nested message parts (#1486246)
+- Fix displaying alternative parts in messages of type message/rfc822 (#1486246)
- Fix possible messages exposure when using Roundcube behind a proxy (#1486281)
- Fix unicode para and line separators in javascript response (#1486310)
- additional_message_headers: allow unsetting headers, support plugin's config file (#1486268)
diff --git a/program/include/rcube_message.php b/program/include/rcube_message.php
index 0b6fbb1e9..62f0fb0d8 100644
--- a/program/include/rcube_message.php
+++ b/program/include/rcube_message.php
@@ -225,7 +225,7 @@ class rcube_message
* @param object rcube_message_part Message structure node
* @param bool True when called recursively
*/
- private function parse_structure($structure, $recursive = false)
+ private function parse_structure($structure, $recursive = false, $alternative = false)
{
$message_ctype_primary = strtolower($structure->ctype_primary);
$message_ctype_secondary = strtolower($structure->ctype_secondary);
@@ -351,7 +351,8 @@ class rcube_message
($primary_type == 'message' && ($secondary_type == 'delivery-status' || $secondary_type == 'disposition-notification'))) {
// add text part if it matches the prefs
- if (($secondary_type == 'html' && $this->opt['prefer_html']) ||
+ if ((!$this->parse_alternative && !$alternative) ||
+ ($secondary_type == 'html' && $this->opt['prefer_html']) ||
($secondary_type == 'plain' && !$this->opt['prefer_html'])) {
$mail_part->type = 'content';
$this->parts[] = $mail_part;
@@ -363,8 +364,19 @@ class rcube_message
}
// part message/*
else if ($primary_type=='message') {
- $this->parse_structure($mail_part, true);
-
+ // let's try to find out if message/rfc822 is a multipart/alternative
+ if ($secondary_type == 'rfc822' && is_array($mail_part->parts) && count($mail_part->parts) > 1) {
+ $types = array();
+ // iterate over parts to find its types and count them by type
+ for ($j=0; $j < count($mail_part->parts); $j++) {
+ $_type = strtolower($mail_part->parts[$j]->ctype_primary).'/'.strtolower($mail_part->parts[$j]->ctype_secondary);
+ $types[$_type] = $types[$_type] ? $types[$_type]+1 : 1;
+ }
+ if ($types['text/plain'] == 1 && $types['text/html'] == 1)
+ $_alternative = true;
+ }
+ $this->parse_structure($mail_part, true, $_alternative);
+
// list as attachment as well (mostly .eml)
if (!empty($mail_part->filename))
$this->attachments[] = $mail_part;