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 | 54 +++++++++++++++++++++++++++++++----------------------- 1 files changed, 31 insertions(+), 23 deletions(-) diff --git a/program/include/rcube_message.php b/program/include/rcube_message.php index c3753b2..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']) || @@ -488,6 +491,11 @@ else if ($structure->filename) { $this->attachments[] = $structure; } + // message is a single part non-text (without filename) + else if (preg_match('/application\//i', $mimetype)) { + $structure->filename = 'Part '.$structure->mime_id; + $this->attachments[] = $structure; + } } -- Gitblit v1.9.1