From 1a2f8375ded7563964ea24c44c7874a92e6f7b77 Mon Sep 17 00:00:00 2001
From: alecpl <alec@alec.pl>
Date: Thu, 12 Aug 2010 03:11:28 -0400
Subject: [PATCH] - add message_part_structure hook also for text parts of mixed messages

---
 program/include/rcube_message.php |   49 ++++++++++++++++++++++++++-----------------------
 1 files changed, 26 insertions(+), 23 deletions(-)

diff --git a/program/include/rcube_message.php b/program/include/rcube_message.php
index fac7fe8..8ba607c 100644
--- a/program/include/rcube_message.php
+++ b/program/include/rcube_message.php
@@ -247,17 +247,11 @@
      */
     private function parse_structure($structure, $recursive = false)
     {
-        $message_ctype_primary = $structure->ctype_primary;
-        $message_ctype_secondary = $structure->ctype_secondary;
-        $mimetype = $structure->mimetype;
-
         // real content-type of message/rfc822 part
-        if ($mimetype == 'message/rfc822') {
-            if ($structure->real_mimetype) {
-                $mimetype = $structure->real_mimetype;
-                list($message_ctype_primary, $message_ctype_secondary) = explode('/', $mimetype);
-            }
-        }
+        if ($structure->mimetype == 'message/rfc822' && $structure->real_mimetype)
+            $mimetype = $structure->real_mimetype;
+        else
+            $mimetype = $structure->mimetype;
 
         // show message headers
         if ($recursive && is_array($structure->headers) && isset($structure->headers['subject'])) {
@@ -266,6 +260,17 @@
             $c->headers = &$structure->headers;
             $this->parts[] = $c;
         }
+
+        // Allow plugins to handle message parts
+        $plugin = $this->app->plugins->exec_hook('message_part_structure',
+            array('object' => $this, 'structure' => $structure,
+                'mimetype' => $mimetype, 'recursive' => $recursive));
+
+        if ($plugin['abort'])
+            return;
+
+        $structure = $plugin['structure'];
+        list($message_ctype_primary, $message_ctype_secondary) = explode('/', $plugin['mimetype']);
 
         // print body if message doesn't have multiple parts
         if ($message_ctype_primary == 'text' && !$recursive) {
@@ -278,8 +283,6 @@
                     $this->mime_parts[$uupart->mime_id] = $uupart;
                     $this->attachments[] = $uupart;
                 }
-
-            // @TODO: plugin hook?
         }
         // the same for pgp signed messages
         else if ($mimetype == 'application/pgp' && !$recursive) {
@@ -360,17 +363,6 @@
             $p->ctype_secondary = 'plain';
             $p->body            = rcube_label('encryptedmessage');
             $p->size            = strlen($p->body);
-      
-            // maybe some plugins are able to decode this encrypted message part
-            $data = $this->app->plugins->exec_hook('message_part_encrypted',
-                array('object' => $this, 'struct' => $structure, 'part' => $p));
-
-            if (is_array($data['parts'])) {
-                $this->parts = array_merge($this->parts, $data['parts']);
-            }
-            else if ($data['part']) {
-                $this->parts[] = $p;
-            }
         }
         // message contains multiple parts
         else if (is_array($structure->parts) && !empty($structure->parts)) {
@@ -401,6 +393,17 @@
                 else if ((($part_mimetype == 'text/plain' || $part_mimetype == 'text/html') && $mail_part->disposition != 'attachment') ||
                     $part_mimetype == 'message/delivery-status' || $part_mimetype == 'message/disposition-notification'
                 ) {
+                    // Allow plugins to handle also this part
+                    $plugin = $this->app->plugins->exec_hook('message_part_structure',
+                        array('object' => $this, 'structure' => $mail_part,
+                            'mimetype' => $part_mimetype, 'recursive' => true));
+
+                    if ($plugin['abort'])
+                        continue;
+
+                    $mail_part = $plugin['structure'];
+                    list($primary_type, $secondary_type) = explode('/', $plugin['mimetype']);
+
                     // add text part if it matches the prefs
                     if (!$this->parse_alternative ||
                         ($secondary_type == 'html' && $this->opt['prefer_html']) ||

--
Gitblit v1.9.1