From 13ffa2810eaef30a0f69fc3af6ecbb5bc9dd0bdc Mon Sep 17 00:00:00 2001 From: alecpl <alec@alec.pl> Date: Thu, 30 Sep 2010 03:10:03 -0400 Subject: [PATCH] - Added line number for error message --- program/steps/mail/compose.inc | 428 ++++++++++++++++++++++++++++++++++++---------------- 1 files changed, 293 insertions(+), 135 deletions(-) diff --git a/program/steps/mail/compose.inc b/program/steps/mail/compose.inc index 6fcdf43..5a66af1 100644 --- a/program/steps/mail/compose.inc +++ b/program/steps/mail/compose.inc @@ -4,8 +4,8 @@ +-----------------------------------------------------------------------+ | program/steps/mail/compose.inc | | | - | This file is part of the RoundCube Webmail client | - | Copyright (C) 2005-2009, RoundCube Dev. - Switzerland | + | This file is part of the Roundcube Webmail client | + | Copyright (C) 2005-2009, Roundcube Dev. - Switzerland | | Licensed under the GNU GPL | | | | PURPOSE: | @@ -44,6 +44,8 @@ // process values like "mailto:foo@bar.com?subject=new+message&cc=another" if ($_SESSION['compose']['param']['to']) { + // #1486037: remove "mailto:" prefix + $_SESSION['compose']['param']['to'] = preg_replace('/^mailto:/i', '', $_SESSION['compose']['param']['to']); $mailto = explode('?', $_SESSION['compose']['param']['to']); if (count($mailto) > 1) { $_SESSION['compose']['param']['to'] = $mailto[0]; @@ -53,9 +55,12 @@ } } + // select folder where to save the sent message + $_SESSION['compose']['param']['sent_mbox'] = $RCMAIL->config->get('sent_mbox'); + // pipe compose parameters thru plugins $plugin = $RCMAIL->plugins->exec_hook('message_compose', $_SESSION['compose']); - $_SESSION['compose']['param'] = array_merge($_SESSION['compose']['param'], $plugin['param']); + $_SESSION['compose']['param'] = array_merge($_SESSION['compose']['param'], $plugin['param']); // add attachments listed by message_compose hook if (is_array($plugin['attachments'])) { @@ -76,7 +81,7 @@ // save attachment if valid if (($attachment['data'] && $attachment['name']) || ($attachment['path'] && file_exists($attachment['path']))) { - $attachment = rcmail::get_instance()->plugins->exec_hook('save_attachment', $attachment); + $attachment = rcmail::get_instance()->plugins->exec_hook('attachment_save', $attachment); } if ($attachment['status'] && !$attachment['abort']) { @@ -84,6 +89,15 @@ $_SESSION['compose']['attachments'][$attachment['id']] = $attachment; } } + } + + // check if folder for saving sent messages exists and is subscribed (#1486802) + if (($sent_folder = $_SESSION['compose']['param']['sent_mbox']) && !$IMAP->mailbox_exists($sent_folder, true)) { + // folder may exist but isn't subscribed (#1485241) + if (!$IMAP->mailbox_exists($sent_folder)) + $IMAP->create_mailbox($sent_folder, true); + else + $IMAP->subscribe($sent_folder); } // redirect to a unique URL with all parameters stored in session @@ -128,13 +142,20 @@ else $OUTPUT->set_env('show_sig', false); +// set line length for body wrapping +$LINE_LENGTH = $RCMAIL->config->get('line_length', 72); + if (!empty($msg_uid)) { // similar as in program/steps/mail/show.inc // re-set 'prefer_html' to have possibility to use html part for compose - $CONFIG['prefer_html'] = $CONFIG['htmleditor'] || $compose_mode == RCUBE_COMPOSE_DRAFT || $compose_mode == RCUBE_COMPOSE_EDIT; + $CONFIG['prefer_html'] = $CONFIG['prefer_html'] || $CONFIG['htmleditor'] || $compose_mode == RCUBE_COMPOSE_DRAFT || $compose_mode == RCUBE_COMPOSE_EDIT; $MESSAGE = new rcube_message($msg_uid); + // make sure message is marked as read + if ($MESSAGE && $MESSAGE->headers && !$MESSAGE->headers->seen) + $IMAP->set_flag($msg_uid, 'SEEN'); + if (!empty($MESSAGE->headers->charset)) $IMAP->set_charset($MESSAGE->headers->charset); @@ -146,26 +167,41 @@ if (!empty($_SESSION['compose']['param']['all'])) $MESSAGE->reply_all = 1; + + $OUTPUT->set_env('compose_mode', 'reply'); } else if ($compose_mode == RCUBE_COMPOSE_DRAFT) { - if($MESSAGE->headers->in_reply_to) + if ($MESSAGE->headers->others['x-draft-info']) { - // TODO: how to get reply_uid/forward_uid value, maybe we must set X-Reply-UID/X-Forward-UID - // $_SESSION['compose']['reply_uid'] = ? - // $_SESSION['compose']['forward_uid'] = ? - $_SESSION['compose']['reply_msgid'] = '<'.$MESSAGE->headers->in_reply_to.'>'; + // get reply_uid/forward_uid to flag the original message when sending + $info = rcmail_draftinfo_decode($MESSAGE->headers->others['x-draft-info']); + + if ($info['type'] == 'reply') + $_SESSION['compose']['reply_uid'] = $info['uid']; + else if ($info['type'] == 'forward') + $_SESSION['compose']['forward_uid'] = $info['uid']; + + $_SESSION['compose']['mailbox'] = $info['folder']; } + + if ($MESSAGE->headers->in_reply_to) + $_SESSION['compose']['reply_msgid'] = '<'.$MESSAGE->headers->in_reply_to.'>'; + $_SESSION['compose']['references'] = $MESSAGE->headers->references; } else if ($compose_mode == RCUBE_COMPOSE_FORWARD) { $_SESSION['compose']['forward_uid'] = $msg_uid; + $OUTPUT->set_env('compose_mode', 'forward'); } } -/****** compose mode functions ********/ +// process $MESSAGE body/attachments, set $MESSAGE_BODY/$HTML_MODE vars and some session data +$MESSAGE_BODY = rcmail_prepare_message_body(); + +/****** compose mode functions ********/ function rcmail_compose_headers($attrib) { @@ -173,36 +209,34 @@ static $sa_recipients = array(); list($form_start, $form_end) = get_form_tags($attrib); - + $out = ''; $part = strtolower($attrib['part']); - + switch ($part) { case 'from': - return rcmail_compose_header_from($attrib); + return $form_start . rcmail_compose_header_from($attrib); case 'to': $fname = '_to'; $header = $param = 'to'; - + // we have a set of recipients stored is session if (($mailto_id = $_SESSION['compose']['param']['mailto']) && $_SESSION['mailto'][$mailto_id]) $fvalue = urldecode($_SESSION['mailto'][$mailto_id]); - + case 'cc': - if (!$fname) - { + if (!$fname) { $fname = '_cc'; $header = $param = 'cc'; } case 'bcc': - if (!$fname) - { + if (!$fname) { $fname = '_bcc'; $header = $param = 'bcc'; } - + $allow_attrib = array('id', 'class', 'style', 'cols', 'rows', 'tabindex'); $field_type = 'html_textarea'; break; @@ -215,7 +249,7 @@ $field_type = 'html_inputfield'; break; } - + if ($fname && !empty($_POST[$fname])) { $fvalue = get_input_value($fname, RCUBE_INPUT_POST, TRUE); } @@ -226,13 +260,10 @@ // get recipent address(es) out of the message headers if ($header=='to' && !empty($MESSAGE->headers->replyto)) $fvalue = $MESSAGE->headers->replyto; - else if ($header=='to' && !empty($MESSAGE->headers->from)) $fvalue = $MESSAGE->headers->from; - // add recipent of original message if reply to all - else if ($header=='cc' && !empty($MESSAGE->reply_all)) - { + else if ($header=='cc' && !empty($MESSAGE->reply_all)) { if ($v = $MESSAGE->headers->to) $fvalue .= $v; @@ -241,39 +272,58 @@ } // split recipients and put them back together in a unique way - if (!empty($fvalue)) - { + if (!empty($fvalue)) { $to_addresses = $IMAP->decode_address_list($fvalue); $fvalue = ''; - foreach ($to_addresses as $addr_part) - { - if (!empty($addr_part['mailto']) - && !in_array($addr_part['mailto'], $sa_recipients) + foreach ($to_addresses as $addr_part) { + if (empty($addr_part['mailto'])) + continue; + + $mailto = idn_to_utf8($addr_part['mailto']); + + if (!in_array($mailto, $sa_recipients) && (!$MESSAGE->compose_from - || !in_array_nocase($addr_part['mailto'], $MESSAGE->compose_from) - || (count($to_addresses)==1 && $header=='to'))) // allow reply to yourself - { - $fvalue .= (strlen($fvalue) ? ', ':'').$addr_part['string']; + || !in_array_nocase($mailto, $MESSAGE->compose_from) + || (count($to_addresses)==1 && $header=='to')) // allow reply to yourself + ) { + if ($addr_part['name'] && $addr_part['mailto'] != $addr_part['name']) + $string = format_email_recipient($mailto, $addr_part['name']); + else + $string = $mailto; + $fvalue .= (strlen($fvalue) ? ', ':'') . $string; $sa_recipients[] = $addr_part['mailto']; } } } } - else if ($header && in_array($compose_mode, array(RCUBE_COMPOSE_DRAFT, RCUBE_COMPOSE_EDIT))) - { + else if ($header && in_array($compose_mode, array(RCUBE_COMPOSE_DRAFT, RCUBE_COMPOSE_EDIT))) { // get drafted headers if ($header=='to' && !empty($MESSAGE->headers->to)) $fvalue = $MESSAGE->get_header('to'); - - if ($header=='cc' && !empty($MESSAGE->headers->cc)) + else if ($header=='cc' && !empty($MESSAGE->headers->cc)) $fvalue = $MESSAGE->get_header('cc'); - - if ($header=='bcc' && !empty($MESSAGE->headers->bcc)) + else if ($header=='bcc' && !empty($MESSAGE->headers->bcc)) $fvalue = $MESSAGE->get_header('bcc'); + + $addresses = $IMAP->decode_address_list($fvalue); + $fvalue = ''; + + foreach ($addresses as $addr_part) { + if (empty($addr_part['mailto'])) + continue; + + $mailto = idn_to_utf8($addr_part['mailto']); + + if ($addr_part['name'] && $addr_part['mailto'] != $addr_part['name']) + $string = format_email_recipient($mailto, $addr_part['name']); + else + $string = $mailto; + $fvalue .= (strlen($fvalue) ? ', ':'') . $string; + } } - + if ($fname && $field_type) { // pass the following attributes to the form class @@ -290,9 +340,8 @@ if ($form_start) $out = $form_start.$out; - return $out; + return $out; } - function rcmail_compose_header_from($attrib) @@ -315,7 +364,7 @@ foreach ($a_to as $addr) { if (!empty($addr['mailto'])) - $a_recipients[] = mb_strtolower($addr['mailto']); + $a_recipients[] = mb_strtolower(idn_to_utf8($addr['mailto'])); } if (!empty($MESSAGE->headers->cc)) @@ -324,7 +373,7 @@ foreach ($a_cc as $addr) { if (!empty($addr['mailto'])) - $a_recipients[] = mb_strtolower($addr['mailto']); + $a_recipients[] = mb_strtolower(idn_to_utf8($addr['mailto'])); } } } @@ -342,6 +391,7 @@ foreach ($user_identities as $sql_arr) { + $sql_arr['email'] = mb_strtolower(idn_to_utf8($sql_arr['email'])); $identity_id = $sql_arr['identity_id']; $select_from->add(format_email_recipient($sql_arr['email'], $sql_arr['name']), $identity_id); @@ -360,13 +410,13 @@ if ($compose_mode == RCUBE_COMPOSE_REPLY && is_array($MESSAGE->compose_from)) $MESSAGE->compose_from[] = $sql_arr['email']; - if (empty($_POST['_from'])) + if (empty($_POST['_from']) && empty($_SESSION['compose']['param']['from'])) { // set draft's identity if ($compose_mode == RCUBE_COMPOSE_DRAFT && strstr($MESSAGE->headers->from, $sql_arr['email'])) $from_id = $sql_arr['identity_id']; // set identity if it's one of the reply-message recipients (with prio for default identity) - else if (in_array(mb_strtolower($sql_arr['email']), $a_recipients) && (empty($from_id) || $sql_arr['standard'])) + else if (in_array($sql_arr['email'], $a_recipients) && (empty($from_id) || $sql_arr['standard'])) $from_id = $sql_arr['identity_id']; } } @@ -374,6 +424,8 @@ // overwrite identity selection with post parameter if (!empty($_POST['_from'])) $from_id = get_input_value('_from', RCUBE_INPUT_POST); + else if (!empty($_SESSION['compose']['param']['from'])) + $from_id = $_SESSION['compose']['param']['from']; $out = $select_from->show($from_id); @@ -385,25 +437,14 @@ $input_from = new html_inputfield($field_attrib); $out = $input_from->show($_POST['_from']); } - - if ($form_start) - $out = $form_start.$out; return $out; } -function rcmail_compose_body($attrib) +function rcmail_prepare_message_body() { - global $RCMAIL, $CONFIG, $OUTPUT, $MESSAGE, $compose_mode; - - list($form_start, $form_end) = get_form_tags($attrib); - unset($attrib['form']); - - if (empty($attrib['id'])) - $attrib['id'] = 'rcmComposeBody'; - - $attrib['name'] = '_message'; + global $RCMAIL, $CONFIG, $MESSAGE, $compose_mode, $LINE_LENGTH, $HTML_MODE; if ($CONFIG['htmleditor'] || (($compose_mode == RCUBE_COMPOSE_DRAFT || $compose_mode == RCUBE_COMPOSE_EDIT) && $MESSAGE->has_html_part())) $isHtml = true; @@ -424,10 +465,20 @@ } else if ($compose_mode) { - if (($isHtml || $compose_mode == RCUBE_COMPOSE_DRAFT) && $MESSAGE->has_html_part()) + $has_html_part = $MESSAGE->has_html_part(); + if (($isHtml || $compose_mode == RCUBE_COMPOSE_DRAFT) && $has_html_part) { $body = $MESSAGE->first_html_part(); $isHtml = true; + } + else if ($has_html_part) + { + // use html part if it has been used for message (pre)viewing + // decrease line length for quoting + $len = $compose_mode == RCUBE_COMPOSE_REPLY ? $LINE_LENGTH-2 : $LINE_LENGTH; + $txt = new html2text($MESSAGE->first_html_part(), false, true, $len); + $body = $txt->get_text(); + $isHtml = false; } else { @@ -448,8 +499,38 @@ $plugin = $RCMAIL->plugins->exec_hook('message_compose_body', array('body' => $body, 'html' => $isHtml, 'mode' => $compose_mode)); - $body = $plugin['body']; + $body = $plugin['body']; + unset($plugin); + // add blocked.gif attachment (#1486516) + if ($isHtml && preg_match('#<img src="\./program/blocked\.gif"#', $body)) { + if ($attachment = rcmail_save_image('program/blocked.gif', 'image/gif')) { + $_SESSION['compose']['attachments'][$attachment['id']] = $attachment; + $body = preg_replace('#\./program/blocked\.gif#', + $RCMAIL->comm_path.'&_action=display-attachment&_file=rcmfile'.$attachment['id'], + $body); + } + } + + $HTML_MODE = $isHtml; + + return $body; +} + +function rcmail_compose_body($attrib) +{ + global $RCMAIL, $CONFIG, $OUTPUT, $MESSAGE, $compose_mode, $LINE_LENGTH, $HTML_MODE, $MESSAGE_BODY; + + list($form_start, $form_end) = get_form_tags($attrib); + unset($attrib['form']); + + if (empty($attrib['id'])) + $attrib['id'] = 'rcmComposeBody'; + + $attrib['name'] = '_message'; + + $isHtml = $HTML_MODE; + $out = $form_start ? "$form_start\n" : ''; $saveid = new html_hiddenfield(array('name' => '_draft_saveid', 'value' => $compose_mode==RCUBE_COMPOSE_DRAFT ? str_replace(array('<','>'), "", $MESSAGE->headers->messageID) : '')); @@ -464,7 +545,7 @@ // If desired, set this textarea to be editable by TinyMCE if ($isHtml) $attrib['class'] = 'mce_editor'; $textarea = new html_textarea($attrib); - $out .= $textarea->show($body); + $out .= $textarea->show($MESSAGE_BODY); $out .= $form_end ? "\n$form_end" : ''; $OUTPUT->set_env('composebody', $attrib['id']); @@ -475,20 +556,40 @@ // include GoogieSpell if (!empty($CONFIG['enable_spellcheck'])) { - $lang = strtolower(substr($_SESSION['language'], 0, 2)); - - $spellcheck_langs = (array)$RCMAIL->config->get('spellcheck_languages', array('da'=>'Dansk', 'de'=>'Deutsch', 'en' => 'English', 'es'=>'Español', 'fr'=>'Français', 'it'=>'Italiano', 'nl'=>'Nederlands', 'pl'=>'Polski', 'pt'=>'Português', 'fi'=>'Suomi', 'sv'=>'Svenska')); + $engine = $RCMAIL->config->get('spellcheck_engine','googie'); + $spellcheck_langs = (array) $RCMAIL->config->get('spellcheck_languages', + array('da'=>'Dansk', 'de'=>'Deutsch', 'en' => 'English', 'es'=>'Español', + 'fr'=>'Français', 'it'=>'Italiano', 'nl'=>'Nederlands', 'pl'=>'Polski', + 'pt'=>'Português', 'fi'=>'Suomi', 'sv'=>'Svenska')); + + // googie works only with two-letter codes + if ($engine == 'googie') { + $lang = strtolower(substr($_SESSION['language'], 0, 2)); + + $spellcheck_langs_googie = array(); + foreach ($spellcheck_langs as $key => $name) + $spellcheck_langs_googie[strtolower(substr($key,0,2))] = $name; + $spellcheck_langs = $spellcheck_langs_googie; + } + else { + $lang = $_SESSION['language']; + + // if not found in the list, try with two-letter code + if (!$spellcheck_langs[$lang]) + $lang = strtolower(substr($lang, 0, 2)); + } + if (!$spellcheck_langs[$lang]) $lang = 'en'; - + $editor_lang_set = array(); foreach ($spellcheck_langs as $key => $name) { $editor_lang_set[] = ($key == $lang ? '+' : '') . JQ($name).'='.JQ($key); - } + } $OUTPUT->include_script('googiespell.js'); $OUTPUT->add_script(sprintf( - "var googie = new GoogieSpell('\$__skin_path/images/googiespell/','%s&_action=spell&lang=');\n". + "var googie = new GoogieSpell('\$__skin_path/images/googiespell/','?_task=utils&_action=spell&lang=');\n". "googie.lang_chck_spell = \"%s\";\n". "googie.lang_rsm_edt = \"%s\";\n". "googie.lang_close = \"%s\";\n". @@ -499,7 +600,6 @@ "googie.setSpellContainer('spellcheck-control');\n". "googie.decorateTextarea('%s');\n". "%s.set_env('spellcheck', googie);", - $RCMAIL->comm_path, JQ(Q(rcube_label('checkspelling'))), JQ(Q(rcube_label('resumeediting'))), JQ(Q(rcube_label('close'))), @@ -522,39 +622,32 @@ function rcmail_create_reply_body($body, $bodyIsHtml) { - global $RCMAIL, $MESSAGE; + global $RCMAIL, $MESSAGE, $LINE_LENGTH; if (!$bodyIsHtml) { + $body = preg_replace('/\r?\n/', "\n", $body); + // try to remove the signature - if ($RCMAIL->config->get('strip_existing_sig', true) && ($sp = strrpos($body, '-- ')) !== false && ($sp == 0 || $body{$sp-1} == "\n")) { - if ($body{$sp+3}==' ' || $body{$sp+3}=="\n" || $body{$sp+3}=="\r") - $body = substr($body, 0, max(0, $sp-1)); - } - - // soft-wrap message first - $body = rcmail_wrap_quoted($body, 75); - - $body = rtrim($body, "\r\n"); - - if ($body) { - // split body into single lines - $a_lines = preg_split('/\r?\n/', $body); - - // add > to each line - for ($n=0; $n<sizeof($a_lines); $n++) { - if (strpos($a_lines[$n], '>')===0) - $a_lines[$n] = '>'.$a_lines[$n]; - else - $a_lines[$n] = '> '.$a_lines[$n]; + if ($RCMAIL->config->get('strip_existing_sig', true)) { + $len = strlen($body); + while (($sp = strrpos($body, "-- \n", $sp ? -$len+$sp-1 : 0)) !== false) { + if ($sp == 0 || $body[$sp-1] == "\n") { + // do not touch blocks with more that X lines + if (substr_count($body, "\n", $sp) < $RCMAIL->config->get('sig_max_lines', 15)) { + $body = substr($body, 0, max(0, $sp-1)); + } + break; + } } - - $body = join("\n", $a_lines); } + + // soft-wrap and quote message text + $body = rcmail_wrap_and_quote(rtrim($body, "\n"), $LINE_LENGTH); // add title line(s) - $prefix = rc_wordwrap(sprintf("On %s, %s wrote:\n", + $prefix = sprintf("On %s, %s wrote:\n", $MESSAGE->headers->date, - $MESSAGE->get_header('from')), 76); + $MESSAGE->get_header('from')); $suffix = ''; @@ -570,17 +663,17 @@ $body = rcmail_wash_html($body, array('safe' => $MESSAGE->is_safe), $cid_map); // build reply (quote content) - $prefix = sprintf("On %s, %s wrote:<br />\n", + $prefix = sprintf("<p>On %s, %s wrote:</p>\n", $MESSAGE->headers->date, htmlspecialchars(Q($MESSAGE->get_header('from'), 'replace'), ENT_COMPAT, $RCMAIL->output->get_charset())); - $prefix .= '<blockquote type="cite" style="padding-left:5px; border-left:#1010ff 2px solid; margin-left:5px; width:100%">'; - + $prefix .= '<blockquote>'; + if ($RCMAIL->config->get('top_posting')) { - $prefix = "<p></p>" . $prefix; - $suffix = "</blockquote>"; + $prefix = '<br>' . $prefix; + $suffix = '</blockquote>'; } else { - $suffix = "</blockquote><p></p>"; + $suffix = '</blockquote><p></p>'; } } @@ -603,8 +696,12 @@ $prefix .= 'Date: ' . $MESSAGE->headers->date . "\n"; $prefix .= 'From: ' . $MESSAGE->get_header('from') . "\n"; $prefix .= 'To: ' . $MESSAGE->get_header('to') . "\n"; + + if ($MESSAGE->headers->cc) + $prefix .= 'Cc: ' . $MESSAGE->get_header('cc') . "\n"; if ($MESSAGE->headers->replyto && $MESSAGE->headers->replyto != $MESSAGE->headers->from) $prefix .= 'Reply-To: ' . $MESSAGE->get_header('replyto') . "\n"; + $prefix .= "\n"; } else @@ -615,7 +712,7 @@ $body = rcmail_wash_html($body, array('safe' => $MESSAGE->is_safe), $cid_map); $prefix = sprintf( - "<p></p><p>-------- Original Message --------</p>" . + "<br /><p>-------- Original Message --------</p>" . "<table border=\"0\" cellpadding=\"0\" cellspacing=\"0\"><tbody>" . "<tr><th align=\"right\" nowrap=\"nowrap\" valign=\"baseline\">Subject: </th><td>%s</td></tr>" . "<tr><th align=\"right\" nowrap=\"nowrap\" valign=\"baseline\">Date: </th><td>%s</td></tr>" . @@ -625,6 +722,10 @@ Q($MESSAGE->headers->date), htmlspecialchars(Q($MESSAGE->get_header('from'), 'replace'), ENT_COMPAT, $OUTPUT->get_charset()), htmlspecialchars(Q($MESSAGE->get_header('to'), 'replace'), ENT_COMPAT, $OUTPUT->get_charset())); + + if ($MESSAGE->headers->cc) + $prefix .= sprintf("<tr><th align=\"right\" nowrap=\"nowrap\" valign=\"baseline\">Cc: </th><td>%s</td></tr>", + htmlspecialchars(Q($MESSAGE->get_header('cc'), 'replace'), ENT_COMPAT, $OUTPUT->get_charset())); if ($MESSAGE->headers->replyto && $MESSAGE->headers->replyto != $MESSAGE->headers->from) $prefix .= sprintf("<tr><th align=\"right\" nowrap=\"nowrap\" valign=\"baseline\">Reply-To: </th><td>%s</td></tr>", @@ -664,16 +765,33 @@ { global $OUTPUT; - $cid_map = array(); + $cid_map = $messages = array(); foreach ((array)$message->mime_parts as $pid => $part) { if (($part->ctype_primary != 'message' || !$bodyIsHtml) && $part->ctype_primary != 'multipart' && - ($part->disposition == 'attachment' || ($part->disposition == 'inline' && $bodyIsHtml) || $part->filename)) - { - if ($attachment = rcmail_save_attachment($message, $pid)) { + ($part->disposition == 'attachment' || ($part->disposition == 'inline' && $bodyIsHtml) || $part->filename) + && $part->mimetype != 'application/ms-tnef' + ) { + $skip = false; + if ($part->mimetype == 'message/rfc822') { + $messages[] = $part->mime_id; + } else if ($messages) { + // skip attachments included in message/rfc822 attachment (#1486487) + foreach ($messages as $mimeid) + if (strpos($part->mime_id, $mimeid.'.') === 0) { + $skip = true; + break; + } + } + + if (!$skip && ($attachment = rcmail_save_attachment($message, $pid))) { $_SESSION['compose']['attachments'][$attachment['id']] = $attachment; - if ($bodyIsHtml && $part->content_id) { - $cid_map['cid:'.$part->content_id] = $OUTPUT->app->comm_path.'&_action=display-attachment&_file=rcmfile'.$attachment['id']; + if ($bodyIsHtml && ($part->content_id || $part->content_location)) { + $url = $OUTPUT->app->comm_path.'&_action=display-attachment&_file=rcmfile'.$attachment['id']; + if ($part->content_id) + $cid_map['cid:'.$part->content_id] = $url; + else + $cid_map[$part->content_location] = $url; } } } @@ -691,14 +809,18 @@ $cid_map = array(); foreach ((array)$message->mime_parts as $pid => $part) { - if ($part->content_id && $part->filename) { + if (($part->content_id || $part->content_location) && $part->filename) { if ($attachment = rcmail_save_attachment($message, $pid)) { $_SESSION['compose']['attachments'][$attachment['id']] = $attachment; - $cid_map['cid:'.$part->content_id] = $OUTPUT->app->comm_path.'&_action=display-attachment&_file=rcmfile'.$attachment['id']; + $url = $OUTPUT->app->comm_path.'&_action=display-attachment&_file=rcmfile'.$attachment['id']; + if ($part->content_id) + $cid_map['cid:'.$part->content_id] = $url; + else + $cid_map[$part->content_location] = $url; } } } - + return $cid_map; } @@ -728,10 +850,11 @@ 'mimetype' => $part->ctype_primary . '/' . $part->ctype_secondary, 'content_id' => $part->content_id, 'data' => $data, - 'path' => $path + 'path' => $path, + 'size' => $path ? filesize($path) : strlen($data), ); - - $attachment = rcmail::get_instance()->plugins->exec_hook('save_attachment', $attachment); + + $attachment = rcmail::get_instance()->plugins->exec_hook('attachment_save', $attachment); if ($attachment['status']) { unset($attachment['data'], $attachment['status'], $attachment['content_id'], $attachment['abort']); @@ -743,6 +866,37 @@ return false; } +function rcmail_save_image($path, $mimetype='') +{ + // handle attachments in memory + $data = file_get_contents($path); + + $attachment = array( + 'name' => rcmail_basename($path), + 'mimetype' => $mimetype ? $mimetype : rc_mime_content_type($path, $name), + 'data' => $data, + 'size' => strlen($data), + ); + + $attachment = rcmail::get_instance()->plugins->exec_hook('attachment_save', $attachment); + + if ($attachment['status']) { + unset($attachment['data'], $attachment['status'], $attachment['content_id'], $attachment['abort']); + return $attachment; + } + + return false; +} + +function rcmail_basename($filename) +{ + // basename() is not unicode safe and locale dependent + if (stristr(PHP_OS, 'win') || stristr(PHP_OS, 'netware')) { + return preg_replace('/^.*[\\\\\\/]/', '', $filename); + } else { + return preg_replace('/^.*[\/]/', '', $filename); + } +} function rcmail_compose_subject($attrib) { @@ -786,7 +940,7 @@ $out = $form_start ? "$form_start\n" : ''; $out .= $textfield->show($subject); $out .= $form_end ? "\n$form_end" : ''; - + return $out; } @@ -801,7 +955,7 @@ $out = "\n"; $jslist = array(); - + if (is_array($_SESSION['compose']['attachments'])) { if ($attrib['deleteicon']) { @@ -861,11 +1015,11 @@ $button = new html_inputfield(array('type' => 'button')); $out = html::div($attrib, - $OUTPUT->form_tag(array('name' => 'form', 'method' => 'post', 'enctype' => 'multipart/form-data'), + $OUTPUT->form_tag(array('name' => 'uploadform', 'method' => 'post', 'enctype' => 'multipart/form-data'), html::div(null, rcmail_compose_attachment_field(array('size' => $attrib[attachmentfieldsize]))) . html::div('hint', rcube_label(array('name' => 'maxuploadsize', 'vars' => array('size' => $max_filesize)))) . html::div('buttons', - $button->show(rcube_label('close'), array('class' => 'button', 'onclick' => "document.getElementById('$attrib[id]').style.visibility='hidden'")) . ' ' . + $button->show(rcube_label('close'), array('class' => 'button', 'onclick' => "$('#$attrib[id]').hide()")) . ' ' . $button->show(rcube_label('upload'), array('class' => 'button mainaction', 'onclick' => JS_OBJECT_NAME . ".command('send-attachment', this.form)")) ) ) @@ -891,7 +1045,7 @@ list($form_start, $form_end) = get_form_tags($attrib); unset($attrib['form']); - + $attrib['name'] = '_priority'; $selector = new html_select($attrib); @@ -901,7 +1055,7 @@ rcube_label('high'), rcube_label('highest')), array(5, 4, 0, 2, 1)); - + if (isset($_POST['_priority'])) $sel = $_POST['_priority']; else if (intval($MESSAGE->headers->priority) != 3) @@ -912,18 +1066,18 @@ $out = $form_start ? "$form_start\n" : ''; $out .= $selector->show($sel); $out .= $form_end ? "\n$form_end" : ''; - + return $out; } function rcmail_receipt_checkbox($attrib) { - global $MESSAGE, $compose_mode; - + global $RCMAIL, $MESSAGE, $compose_mode; + list($form_start, $form_end) = get_form_tags($attrib); unset($attrib['form']); - + if (!isset($attrib['id'])) $attrib['id'] = 'receipt'; @@ -931,9 +1085,13 @@ $attrib['value'] = '1'; $checkbox = new html_checkbox($attrib); + if ($MESSAGE && in_array($compose_mode, array(RCUBE_COMPOSE_DRAFT, RCUBE_COMPOSE_EDIT))) + $mdn_default = (bool) $MESSAGE->headers->mdn_to; + else + $mdn_default = $RCMAIL->config->get('mdn_default'); + $out = $form_start ? "$form_start\n" : ''; - $out .= $checkbox->show(in_array($compose_mode, array(RCUBE_COMPOSE_DRAFT, RCUBE_COMPOSE_EDIT)) - && $MESSAGE->headers->mdn_to ? 1 : 0); + $out .= $checkbox->show($mdn_default); $out .= $form_end ? "\n$form_end" : ''; return $out; @@ -956,8 +1114,8 @@ if (empty($attrib['name'])) $attrib['name'] = 'editorSelect'; - - $attrib['onchange'] = "return rcmail_toggle_editor(this.value=='html', '".$attrib['editorid']."', '_is_html')"; + + $attrib['onchange'] = "return rcmail_toggle_editor(this, '".$attrib['editorid']."', '_is_html')"; $select = new html_select($attrib); @@ -981,7 +1139,7 @@ { $attrib['name'] = '_store_target'; $select = rcmail_mailbox_select(array_merge($attrib, array('noselection' => '- '.rcube_label('dontsave').' -'))); - return $select->show(rcmail::get_instance()->config->get('sent_mbox'), $attrib); + return $select->show($_SESSION['compose']['param']['sent_mbox'], $attrib); } @@ -990,7 +1148,7 @@ global $RCMAIL, $MESSAGE_FORM; $form_start = ''; - if (!strlen($MESSAGE_FORM)) + if (!$MESSAGE_FORM) { $hiddenfields = new html_hiddenfield(array('name' => '_task', 'value' => $RCMAIL->task)); $hiddenfields->add(array('name' => '_action', 'value' => 'send')); @@ -999,10 +1157,10 @@ $form_start .= $hiddenfields->show(); } - $form_end = (strlen($MESSAGE_FORM) && !strlen($attrib['form'])) ? '</form>' : ''; + $form_end = ($MESSAGE_FORM && !strlen($attrib['form'])) ? '</form>' : ''; $form_name = !empty($attrib['form']) ? $attrib['form'] : 'form'; - if (!strlen($MESSAGE_FORM)) + if (!$MESSAGE_FORM) $RCMAIL->output->add_gui_object('messageform', $form_name); $MESSAGE_FORM = $form_name; @@ -1027,4 +1185,4 @@ $OUTPUT->send('compose'); -?> + -- Gitblit v1.9.1