From bd698341c4871bdabd763601583f0e9ff7137a97 Mon Sep 17 00:00:00 2001
From: Aleksander Machniak <alec@alec.pl>
Date: Wed, 03 Apr 2013 03:17:03 -0400
Subject: [PATCH] Remove "HTML message" from attachments list while viewing a message in text mode (#1486939)

---
 CHANGELOG                               |    1 +
 program/lib/Roundcube/rcube_message.php |   12 ------------
 2 files changed, 1 insertions(+), 12 deletions(-)

diff --git a/CHANGELOG b/CHANGELOG
index 5309676..6593367 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -7,6 +7,7 @@
 - Call resize handler in intervals to prevent lags and double onresize calls in Chrome (#1489005)
 - Add rel="noreferrer" for links in displayed messages (#1484686)
 - Add ability to toggle between HTML and text while viewing a message (#1486939)
+- Remove "HTML message" from attachments list while viewing a message in text mode (#1486939)
 - Support IMAP MOVE extension [RFC 6851]
 - Add attachment menu with Open and Download options (#1488975)
 - Display user-friendly message on IMAP "over quota" errors (#1484164)
diff --git a/program/lib/Roundcube/rcube_message.php b/program/lib/Roundcube/rcube_message.php
index 42d7b9b..41a114f 100644
--- a/program/lib/Roundcube/rcube_message.php
+++ b/program/lib/Roundcube/rcube_message.php
@@ -486,14 +486,6 @@
                 $this->parts[] = $c;
             }
 
-            // add html part as attachment
-            if ($html_part !== null && $structure->parts[$html_part] !== $print_part) {
-                $html_part = $structure->parts[$html_part];
-                $html_part->mimetype = 'text/html';
-
-                $this->attachments[] = $html_part;
-            }
-
             // add unsupported/unrecognized parts to attachments list
             if ($attach_part) {
                 $this->attachments[] = $structure->parts[$attach_part];
@@ -576,10 +568,6 @@
 
                     // list as attachment as well
                     if (!empty($mail_part->filename)) {
-                        $this->attachments[] = $mail_part;
-                    }
-                    // list html part as attachment (here the part is most likely inside a multipart/related part)
-                    else if ($this->parse_alternative && ($secondary_type == 'html' && !$this->opt['prefer_html'])) {
                         $this->attachments[] = $mail_part;
                     }
                 }

--
Gitblit v1.9.1