From cb2bc809ef29f349d38c89e202d821e67bb4c947 Mon Sep 17 00:00:00 2001 From: thomascube <thomas@roundcube.net> Date: Tue, 21 Sep 2010 14:47:55 -0400 Subject: [PATCH] Fix db_mode check in insert_id() --- program/include/rcube_message.php | 139 ++++++++++++++++++++++++++++++++------------- 1 files changed, 98 insertions(+), 41 deletions(-) diff --git a/program/include/rcube_message.php b/program/include/rcube_message.php index c3753b2..40df819 100644 --- a/program/include/rcube_message.php +++ b/program/include/rcube_message.php @@ -214,23 +214,22 @@ if ($mimetype == 'text/plain') { $out = $this->imap->get_message_part($this->uid, $mime_id, $part); - + // re-format format=flowed content if ($part->ctype_secondary == 'plain' && $part->ctype_parameters['format'] == 'flowed') $out = self::unfold_flowed($out); break; } else if ($mimetype == 'text/html') { - $html_part = $this->imap->get_message_part($this->uid, $mime_id, $part); + $out = $this->imap->get_message_part($this->uid, $mime_id, $part); // remove special chars encoding $trans = array_flip(get_html_translation_table(HTML_ENTITIES)); - $html_part = strtr($html_part, $trans); + $out = strtr($out, $trans); // create instance of html2text class - $txt = new html2text($html_part); + $txt = new html2text($out); $out = $txt->get_text(); - break; } } @@ -247,17 +246,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 +259,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 +282,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 +362,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 +392,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']) || @@ -486,6 +488,11 @@ } // message is a single part non-text 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; } } @@ -596,10 +603,52 @@ */ public static function unfold_flowed($text) { - return preg_replace( - array('/-- (\r?\n)/', '/^ /m', '/(.) \r?\n/', '/--%SIGEND%(\r?\n)/'), - array('--%SIGEND%\\1', '', '\\1 ', '-- \\1'), - $text); + $text = preg_split('/\r?\n/', $text); + $last = -1; + $q_level = 0; + + foreach ($text as $idx => $line) { + if ($line[0] == '>' && preg_match('/^(>+\s*)/', $line, $regs)) { + $q = strlen(str_replace(' ', '', $regs[0])); + $line = substr($line, strlen($regs[0])); + + if ($q == $q_level && $line + && isset($text[$last]) + && $text[$last][strlen($text[$last])-1] == ' ' + ) { + $text[$last] .= $line; + unset($text[$idx]); + } + else { + $last = $idx; + } + } + else { + $q = 0; + if ($line == '-- ') { + $last = $idx; + } + else { + // remove space-stuffing + $line = preg_replace('/^\s/', '', $line); + + if (isset($text[$last]) && $line + && $text[$last] != '-- ' + && $text[$last][strlen($text[$last])-1] == ' ' + ) { + $text[$last] .= $line; + unset($text[$idx]); + } + else { + $text[$idx] = $line; + $last = $idx; + } + } + } + $q_level = $q; + } + + return implode("\r\n", $text); } @@ -608,17 +657,25 @@ */ public static function format_flowed($text, $length = 72) { - $out = ''; - - foreach (preg_split('/\r?\n/', trim($text)) as $line) { - // don't wrap quoted lines (to avoid wrapping problems) - if ($line[0] != '>') - $line = rc_wordwrap(rtrim($line), $length - 1, " \r\n"); + $text = preg_split('/\r?\n/', $text); - $out .= $line . "\r\n"; + foreach ($text as $idx => $line) { + if ($line != '-- ') { + if ($line[0] == '>' && preg_match('/^(>+)/', $line, $regs)) { + $prefix = $regs[0]; + $level = strlen($prefix); + $line = rtrim(substr($line, $level)); + $line = $prefix . rc_wordwrap($line, $length - $level - 2, " \r\n$prefix "); + } + else { + $line = ' ' . rc_wordwrap(rtrim($line), $length - 2, " \r\n "); + } + + $text[$idx] = $line; + } } - - return $out; + + return implode("\r\n", $text); } } -- Gitblit v1.9.1