From ab0b51a1fef87bcc643c3aaf2e635c811b28ccd8 Mon Sep 17 00:00:00 2001
From: alecpl <alec@alec.pl>
Date: Tue, 15 Feb 2011 06:10:59 -0500
Subject: [PATCH] - Use only one from IMAP authentication methods to prevent login delays (1487784)
---
program/steps/mail/compose.inc | 512 ++++++++++++++++++++++++++++++++------------------------
1 files changed, 295 insertions(+), 217 deletions(-)
diff --git a/program/steps/mail/compose.inc b/program/steps/mail/compose.inc
index 406033c..41f7b14 100644
--- a/program/steps/mail/compose.inc
+++ b/program/steps/mail/compose.inc
@@ -5,7 +5,7 @@
| program/steps/mail/compose.inc |
| |
| This file is part of the Roundcube Webmail client |
- | Copyright (C) 2005-2009, Roundcube Dev. - Switzerland |
+ | Copyright (C) 2005-2009, The Roundcube Dev Team |
| Licensed under the GNU GPL |
| |
| PURPOSE: |
@@ -33,9 +33,17 @@
// Since there are many ways to leave the compose page improperly, it seems necessary to clean-up an old
// compose when a "new/forward/reply/draft" is called - otherwise the old session attachments will appear
-if (!is_array($_SESSION['compose']) || $_SESSION['compose']['id'] != get_input_value('_id', RCUBE_INPUT_GET))
+$MESSAGE_ID = get_input_value('_id', RCUBE_INPUT_GET);
+if (!is_array($_SESSION['compose']) || $_SESSION['compose']['id'] != $MESSAGE_ID)
{
rcmail_compose_cleanup();
+
+ // Infinite redirect prevention in case of broken session (#1487028)
+ if ($MESSAGE_ID)
+ raise_error(array('code' => 500, 'type' => 'php',
+ 'file' => __FILE__, 'line' => __LINE__,
+ 'message' => "Invalid session"), true, true);
+
$_SESSION['compose'] = array(
'id' => uniqid(mt_rand()),
'param' => request2param(RCUBE_INPUT_GET),
@@ -92,12 +100,8 @@
}
// 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);
+ if ($sent_folder = $_SESSION['compose']['param']['sent_mbox']) {
+ rcmail_check_sent_folder($sent_folder, true);
}
// redirect to a unique URL with all parameters stored in session
@@ -108,7 +112,8 @@
// add some labels to client
$OUTPUT->add_label('nosubject', 'nosenderwarning', 'norecipientwarning', 'nosubjectwarning', 'cancel',
'nobodywarning', 'notsentwarning', 'notuploadedwarning', 'savingmessage', 'sendingmessage',
- 'messagesaved', 'converting', 'editorwarning', 'searching', 'uploading', 'fileuploaderror');
+ 'messagesaved', 'converting', 'editorwarning', 'searching', 'uploading', 'fileuploaderror',
+ 'autocompletechars');
// add config parameters to client script
if (!empty($CONFIG['drafts_mbox'])) {
@@ -119,6 +124,7 @@
$OUTPUT->set_env('mailbox', $IMAP->get_mailbox_name());
$OUTPUT->set_env('sig_above', $CONFIG['sig_above']);
$OUTPUT->set_env('top_posting', $CONFIG['top_posting']);
+$OUTPUT->set_env('autocomplete_min_length', $CONFIG['autocomplete_min_length']);
// get reference message and set compose mode
if ($msg_uid = $_SESSION['compose']['param']['reply_uid'])
@@ -169,6 +175,13 @@
$MESSAGE->reply_all = $_SESSION['compose']['param']['all'];
$OUTPUT->set_env('compose_mode', 'reply');
+
+ // Save the sent message in the same folder of the message being replied to
+ if ($RCMAIL->config->get('reply_same_folder') && ($sent_folder = $_SESSION['compose']['mailbox'])
+ && rcmail_check_sent_folder($sent_folder, false)
+ ) {
+ $_SESSION['compose']['param']['sent_mbox'] = $sent_folder;
+ }
}
else if ($compose_mode == RCUBE_COMPOSE_DRAFT)
{
@@ -183,8 +196,15 @@
$_SESSION['compose']['forward_uid'] = $info['uid'];
$_SESSION['compose']['mailbox'] = $info['folder'];
+
+ // Save the sent message in the same folder of the message being replied to
+ if ($RCMAIL->config->get('reply_same_folder') && ($sent_folder = $info['folder'])
+ && rcmail_check_sent_folder($sent_folder, false)
+ ) {
+ $_SESSION['compose']['param']['sent_mbox'] = $sent_folder;
+ }
}
-
+
if ($MESSAGE->headers->in_reply_to)
$_SESSION['compose']['reply_msgid'] = '<'.$MESSAGE->headers->in_reply_to.'>';
@@ -197,90 +217,123 @@
}
}
-// process $MESSAGE body/attachments, set $MESSAGE_BODY/$HTML_MODE vars and some session data
-$MESSAGE_BODY = rcmail_prepare_message_body();
+$MESSAGE->compose = array();
-
-/****** compose mode functions ********/
-
-function rcmail_compose_headers($attrib)
+// get user's identities
+$MESSAGE->identities = $USER->list_identities();
+if (count($MESSAGE->identities))
{
- global $IMAP, $MESSAGE, $DB, $compose_mode;
- static $sa_recipients = array();
+ foreach ($MESSAGE->identities as $idx => $sql_arr) {
+ $email = mb_strtolower(rcube_idn_to_utf8($sql_arr['email']));
+ $MESSAGE->identities[$idx]['email_ascii'] = $sql_arr['email'];
+ $MESSAGE->identities[$idx]['email'] = $email;
+ }
+}
- list($form_start, $form_end) = get_form_tags($attrib);
-
- $out = '';
- $part = strtolower($attrib['part']);
-
- switch ($part)
+// Set From field value
+if (!empty($_POST['_from'])) {
+ $MESSAGE->compose['from'] = get_input_value('_from', RCUBE_INPUT_POST);
+}
+else if (!empty($_SESSION['compose']['param']['from'])) {
+ $MESSAGE->compose['from'] = $_SESSION['compose']['param']['from'];
+}
+else if (count($MESSAGE->identities)) {
+ // extract all recipients of the reply-message
+ $a_recipients = array();
+ if ($compose_mode == RCUBE_COMPOSE_REPLY && is_object($MESSAGE->headers))
{
- case 'from':
- return $form_start . rcmail_compose_header_from($attrib);
+ $a_to = $IMAP->decode_address_list($MESSAGE->headers->to);
+ foreach ($a_to as $addr) {
+ if (!empty($addr['mailto']))
+ $a_recipients[] = strtolower($addr['mailto']);
+ }
- 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) {
- $fname = '_cc';
- $header = $param = 'cc';
+ if (!empty($MESSAGE->headers->cc)) {
+ $a_cc = $IMAP->decode_address_list($MESSAGE->headers->cc);
+ foreach ($a_cc as $addr) {
+ if (!empty($addr['mailto']))
+ $a_recipients[] = strtolower($addr['mailto']);
}
- case 'bcc':
- if (!$fname) {
- $fname = '_bcc';
- $header = $param = 'bcc';
- }
+ }
+ }
- $allow_attrib = array('id', 'class', 'style', 'cols', 'rows', 'tabindex');
- $field_type = 'html_textarea';
+ $from_idx = null;
+ $default_identity = 0;
+ $return_path = $MESSAGE->headers->others['return-path'];
+
+ // Select identity
+ foreach ($MESSAGE->identities as $idx => $sql_arr) {
+ // save default identity ID
+ if ($sql_arr['standard']) {
+ $default_identity = $idx;
+ }
+ // we need ascii here
+ $email = $sql_arr['email_ascii'];
+ $ident = format_email_recipient($email, $sql_arr['name']);
+
+ // select identity
+ if (in_array($compose_mode, array(RCUBE_COMPOSE_DRAFT, RCUBE_COMPOSE_EDIT))) {
+ if ($MESSAGE->headers->from == $ident) {
+ $from_idx = $idx;
+ break;
+ }
+ }
+ // reply to self, force To header value
+ else if ($compose_mode == RCUBE_COMPOSE_REPLY && $MESSAGE->headers->from == $ident) {
+ $from_idx = $idx;
+ $MESSAGE->compose['to'] = $MESSAGE->headers->to;
break;
-
- case 'replyto':
- case 'reply-to':
- $fname = '_replyto';
- $param = 'replyto';
- $header = 'reply-to';
-
- case 'mailreplyto':
- case 'mailreply-to':
- if (!$fname) {
- $fname = '_mailreplyto';
- $param = 'mailreplyto';
- $header = 'mailreply-to';
- }
-
- case 'mailfollowupto':
- case 'mailfollowup-to':
- if (!$fname) {
- $fname = '_mailfollowupto';
- $param = 'mailfollowupto';
- $header = 'mailfollowup-to';
- }
-
- $allow_attrib = array('id', 'class', 'style', 'size', 'tabindex');
- $field_type = 'html_inputfield';
- break;
+ }
+ // set identity if it's one of the reply-message recipients
+ else if (in_array($email, $a_recipients) && ($from_idx === null || $sql_arr['standard'])) {
+ $from_idx = $idx;
+ }
+ // set identity when replying to mailing list
+ else if (strpos($return_path, str_replace('@', '=', $email).'@') !== false) {
+ $from_idx = $idx;
+ }
}
- if ($fname && !empty($_POST[$fname])) {
- $fvalue = get_input_value($fname, RCUBE_INPUT_POST, TRUE);
+ // Still no ID, use first identity
+ if ($from_idx === null) {
+ $from_idx = $default_identity;
}
- else if ($fname && !$fvalue && !empty($_SESSION['compose']['param'][$param])) {
- $fvalue = $_SESSION['compose']['param'][$param];
+
+ $ident = $MESSAGE->identities[$from_idx];
+ $from_id = $ident['identity_id'];
+
+ $MESSAGE->compose['from_email'] = $ident['email'];
+ $MESSAGE->compose['from'] = $from_id;
+}
+
+// Set other headers
+$a_recipients = array();
+$parts = array('to', 'cc', 'bcc', 'replyto', 'followupto');
+
+foreach ($parts as $header) {
+ $fvalue = '';
+
+ // we have a set of recipients stored is session
+ if ($header == 'to' && ($mailto_id = $_SESSION['compose']['param']['mailto'])
+ && $_SESSION['mailto'][$mailto_id]
+ ) {
+ $fvalue = urldecode($_SESSION['mailto'][$mailto_id]);
}
- else if ($header && $compose_mode == RCUBE_COMPOSE_REPLY) {
+ else if (!empty($_POST['_'.$header])) {
+ $fvalue = get_input_value('_'.$header, RCUBE_INPUT_POST, TRUE);
+ }
+ else if (!empty($_SESSION['compose']['param'][$header])) {
+ $fvalue = $_SESSION['compose']['param'][$header];
+ }
+ else if ($compose_mode == RCUBE_COMPOSE_REPLY) {
// get recipent address(es) out of the message headers
if ($header == 'to') {
$mailfollowup = $MESSAGE->headers->others['mail-followup-to'];
$mailreplyto = $MESSAGE->headers->others['mail-reply-to'];
- if ($MESSAGE->reply_all == 'list' && $mailfollowup)
+ if ($MESSAGE->compose['to'])
+ $fvalue = $MESSAGE->compose['to'];
+ else if ($MESSAGE->reply_all == 'list' && $mailfollowup)
$fvalue = $mailfollowup;
else if ($MESSAGE->reply_all == 'list'
&& preg_match('/<mailto:([^>]+)>/i', $MESSAGE->headers->others['list-post'], $m))
@@ -292,41 +345,15 @@
else if (!empty($MESSAGE->headers->from))
$fvalue = $MESSAGE->headers->from;
}
- // add recipent of original message if reply to all
+ // add recipient of original message if reply to all
else if ($header == 'cc' && !empty($MESSAGE->reply_all) && $MESSAGE->reply_all != 'list') {
if ($v = $MESSAGE->headers->to)
$fvalue .= $v;
if ($v = $MESSAGE->headers->cc)
$fvalue .= (!empty($fvalue) ? ', ' : '') . $v;
}
-
- // split recipients and put them back together in a unique way
- if (!empty($fvalue)) {
- $to_addresses = $IMAP->decode_address_list($fvalue);
- $fvalue = '';
-
- 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($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 (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');
@@ -334,28 +361,92 @@
$fvalue = $MESSAGE->get_header('cc');
else if ($header=='bcc' && !empty($MESSAGE->headers->bcc))
$fvalue = $MESSAGE->get_header('bcc');
- else if ($header=='reply-to' && !empty($MESSAGE->headers->replyto))
+ else if ($header=='replyto' && !empty($MESSAGE->headers->others['mail-reply-to']))
+ $fvalue = $MESSAGE->get_header('mail-reply-to');
+ else if ($header=='replyto' && !empty($MESSAGE->headers->replyto))
$fvalue = $MESSAGE->get_header('reply-to');
- else if ($header=='mail-reply-to' && !empty($MESSAGE->headers->others['mail-reply-to']))
- $fvalue = $MESSAGE->get_header('followup-to');
- else if ($header=='mail-followup-to' && !empty($MESSAGE->headers->others['mail-followup-to']))
+ else if ($header=='followupto' && !empty($MESSAGE->headers->others['mail-followup-to']))
$fvalue = $MESSAGE->get_header('mail-followup-to');
+ }
- $addresses = $IMAP->decode_address_list($fvalue);
- $fvalue = '';
+ // split recipients and put them back together in a unique way
+ if (!empty($fvalue) && in_array($header, array('to', 'cc', 'bcc'))) {
+ $to_addresses = $IMAP->decode_address_list($fvalue);
+ $fvalue = array();
- foreach ($addresses as $addr_part) {
+ foreach ($to_addresses as $addr_part) {
if (empty($addr_part['mailto']))
continue;
- $mailto = idn_to_utf8($addr_part['mailto']);
+ $mailto = mb_strtolower(rcube_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 (!in_array($mailto, $a_recipients)
+ && (empty($MESSAGE->compose['from_email']) || $mailto != $MESSAGE->compose['from_email'])
+ ) {
+ if ($addr_part['name'] && $addr_part['mailto'] != $addr_part['name'])
+ $string = format_email_recipient($mailto, $addr_part['name']);
+ else
+ $string = $mailto;
+
+ $fvalue[] = $string;
+ $a_recipients[] = $addr_part['mailto'];
+ }
}
+
+ $fvalue = implode(', ', $fvalue);
+ }
+
+ $MESSAGE->compose[$header] = $fvalue;
+}
+unset($a_recipients);
+
+// 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)
+{
+ global $MESSAGE;
+
+ list($form_start, $form_end) = get_form_tags($attrib);
+
+ $out = '';
+ $part = strtolower($attrib['part']);
+
+ switch ($part)
+ {
+ case 'from':
+ return $form_start . rcmail_compose_header_from($attrib);
+
+ case 'to':
+ case 'cc':
+ case 'bcc':
+ $fname = '_' . $part;
+ $header = $param = $part;
+
+ $allow_attrib = array('id', 'class', 'style', 'cols', 'rows', 'tabindex');
+ $field_type = 'html_textarea';
+ break;
+
+ case 'replyto':
+ case 'reply-to':
+ $fname = '_replyto';
+ $param = 'replyto';
+ $header = 'reply-to';
+
+ case 'followupto':
+ case 'followup-to':
+ if (!$fname) {
+ $fname = '_followupto';
+ $param = 'followupto';
+ $header = 'mail-followup-to';
+ }
+
+ $allow_attrib = array('id', 'class', 'style', 'size', 'tabindex');
+ $field_type = 'html_inputfield';
+ break;
}
if ($fname && $field_type)
@@ -368,7 +459,7 @@
// create teaxtarea object
$input = new $field_type($field_attrib);
- $out = $input->show($fvalue);
+ $out = $input->show($MESSAGE->compose[$param]);
}
if ($form_start)
@@ -380,42 +471,15 @@
function rcmail_compose_header_from($attrib)
{
- global $IMAP, $MESSAGE, $DB, $USER, $OUTPUT, $compose_mode;
-
+ global $MESSAGE, $OUTPUT;
+
// pass the following attributes to the form class
$field_attrib = array('name' => '_from');
foreach ($attrib as $attr => $value)
if (in_array($attr, array('id', 'class', 'style', 'size', 'tabindex')))
$field_attrib[$attr] = $value;
- // extract all recipients of the reply-message
- $a_recipients = array();
- if ($compose_mode == RCUBE_COMPOSE_REPLY && is_object($MESSAGE->headers))
- {
- $MESSAGE->compose_from = array();
-
- $a_to = $IMAP->decode_address_list($MESSAGE->headers->to);
- foreach ($a_to as $addr)
- {
- if (!empty($addr['mailto']))
- $a_recipients[] = strtolower($addr['mailto']);
- }
-
- if (!empty($MESSAGE->headers->cc))
- {
- $a_cc = $IMAP->decode_address_list($MESSAGE->headers->cc);
- foreach ($a_cc as $addr)
- {
- if (!empty($addr['mailto']))
- $a_recipients[] = strtolower($addr['mailto']);
- }
- }
- }
-
- // get this user's identities
- $user_identities = $USER->list_identities();
-
- if (count($user_identities))
+ if (count($MESSAGE->identities))
{
$a_signatures = array();
@@ -423,11 +487,10 @@
$select_from = new html_select($field_attrib);
// create SELECT element
- foreach ($user_identities as $sql_arr)
+ foreach ($MESSAGE->identities as $sql_arr)
{
- $email = mb_strtolower(idn_to_utf8($sql_arr['email']));
$identity_id = $sql_arr['identity_id'];
- $select_from->add(format_email_recipient($email, $sql_arr['name']), $identity_id);
+ $select_from->add(format_email_recipient($sql_arr['email'], $sql_arr['name']), $identity_id);
// add signature to array
if (!empty($sql_arr['signature']) && empty($_SESSION['compose']['param']['nosig']))
@@ -440,50 +503,18 @@
$a_signatures[$identity_id]['plain_text'] = trim($h2t->get_text());
}
}
-
- if ($compose_mode == RCUBE_COMPOSE_REPLY && is_array($MESSAGE->compose_from))
- $MESSAGE->compose_from[] = $email;
}
- $from_id = 0;
-
- // 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'];
- else {
- $return_path = $MESSAGE->headers->others['return-path'];
-
- // Set identity
- foreach ($user_identities as $sql_arr) {
- // set draft's identity
- if ($compose_mode == RCUBE_COMPOSE_DRAFT) {
- if (strstr($MESSAGE->headers->from, $sql_arr['email']))
- $from_id = $sql_arr['identity_id'];
- break;
- }
- // set identity if it's one of the reply-message recipients (with prio for default identity)
- else if (in_array($sql_arr['email'], $a_recipients) && (empty($from_id) || $sql_arr['standard']))
- $from_id = $sql_arr['identity_id'];
- // set identity when replying to mailing list
- else if (strpos($return_path, str_replace('@', '=', $sql_arr['email']).'@') !== false)
- $from_id = $sql_arr['identity_id'];
-
- if ($from_id)
- break;
- }
- }
-
- $out = $select_from->show($from_id);
+ $out = $select_from->show($MESSAGE->compose['from']);
// add signatures to client
$OUTPUT->set_env('signatures', $a_signatures);
}
- else
- {
+ // no identities, display text input field
+ else {
+ $field_attrib['class'] = 'from_address';
$input_from = new html_inputfield($field_attrib);
- $out = $input_from->show($_POST['_from']);
+ $out = $input_from->show($MESSAGE->compose['from']);
}
return $out;
@@ -537,7 +568,12 @@
$body = $MESSAGE->first_html_part();
}
else {
- $body = rcmail_plain_body($MESSAGE->first_text_part());
+ $body = $MESSAGE->first_text_part();
+ // try to remove the signature
+ if ($RCMAIL->config->get('strip_existing_sig', true))
+ $body = rcmail_remove_signature($body);
+ // add HTML formatting
+ $body = rcmail_plain_body($body);
if ($body)
$body = '<pre>' . $body . '</pre>';
}
@@ -620,9 +656,20 @@
$out .= $msgtype->show();
// If desired, set this textarea to be editable by TinyMCE
- if ($isHtml) $attrib['class'] = 'mce_editor';
- $textarea = new html_textarea($attrib);
- $out .= $textarea->show($MESSAGE_BODY);
+ if ($isHtml) {
+ $attrib['class'] = 'mce_editor';
+ $textarea = new html_textarea($attrib);
+ $out .= $textarea->show($MESSAGE_BODY);
+ }
+ else {
+ $textarea = new html_textarea($attrib);
+ $out .= $textarea->show('');
+ // quote plain text, inject into textarea
+ $table = get_html_translation_table(HTML_SPECIALCHARS);
+ $MESSAGE_BODY = strtr($MESSAGE_BODY, $table);
+ $out = substr($out, 0, -11) . $MESSAGE_BODY . '</textarea>';
+ }
+
$out .= $form_end ? "\n$form_end" : '';
$OUTPUT->set_env('composebody', $attrib['id']);
@@ -702,26 +749,16 @@
global $RCMAIL, $MESSAGE, $LINE_LENGTH;
// build reply prefix
- $from = array_pop($RCMAIL->imap->decode_address_list($MESSAGE->get_header('from')));
+ $from = array_pop($RCMAIL->imap->decode_address_list($MESSAGE->get_header('from'), 1, false));
$prefix = sprintf("On %s, %s wrote:",
- $MESSAGE->headers->date, $from['name'] ? $from['name'] : idn_to_utf8($from['email']));
+ $MESSAGE->headers->date, $from['name'] ? $from['name'] : rcube_idn_to_utf8($from['mailto']));
if (!$bodyIsHtml) {
$body = preg_replace('/\r?\n/', "\n", $body);
// try to remove the signature
- 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;
- }
- }
- }
+ if ($RCMAIL->config->get('strip_existing_sig', true))
+ $body = rcmail_remove_signature($body);
// soft-wrap and quote message text
$body = rcmail_wrap_and_quote(rtrim($body, "\n"), $LINE_LENGTH);
@@ -835,11 +872,32 @@
return $body;
}
-
-
+
+
+function rcmail_remove_signature($body)
+{
+ global $RCMAIL;
+
+ $len = strlen($body);
+ $sig_max_lines = $RCMAIL->config->get('sig_max_lines', 15);
+
+ 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) < $sig_max_lines) {
+ $body = substr($body, 0, max(0, $sp-1));
+ }
+ break;
+ }
+ }
+
+ return $body;
+}
+
+
function rcmail_write_compose_attachments(&$message, $bodyIsHtml)
{
- global $OUTPUT;
+ global $RCMAIL;
$cid_map = $messages = array();
foreach ((array)$message->mime_parts as $pid => $part)
@@ -863,7 +921,7 @@
if (!$skip && ($attachment = rcmail_save_attachment($message, $pid))) {
$_SESSION['compose']['attachments'][$attachment['id']] = $attachment;
if ($bodyIsHtml && ($part->content_id || $part->content_location)) {
- $url = $OUTPUT->app->comm_path.'&_action=display-attachment&_file=rcmfile'.$attachment['id'];
+ $url = $RCMAIL->comm_path.'&_action=display-attachment&_file=rcmfile'.$attachment['id'];
if ($part->content_id)
$cid_map['cid:'.$part->content_id] = $url;
else
@@ -881,14 +939,14 @@
function rcmail_write_inline_attachments(&$message)
{
- global $OUTPUT;
+ global $RCMAIL;
$cid_map = array();
foreach ((array)$message->mime_parts as $pid => $part) {
if (($part->content_id || $part->content_location) && $part->filename) {
if ($attachment = rcmail_save_attachment($message, $pid)) {
$_SESSION['compose']['attachments'][$attachment['id']] = $attachment;
- $url = $OUTPUT->app->comm_path.'&_action=display-attachment&_file=rcmfile'.$attachment['id'];
+ $url = $RCMAIL->comm_path.'&_action=display-attachment&_file=rcmfile'.$attachment['id'];
if ($part->content_id)
$cid_map['cid:'.$part->content_id] = $url;
else
@@ -1092,7 +1150,7 @@
$out = html::div($attrib,
$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(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' => "$('#$attrib[id]').hide()")) . ' ' .
@@ -1236,6 +1294,26 @@
}
+function rcmail_check_sent_folder($folder, $create=false)
+{
+ global $IMAP;
+
+ if ($IMAP->mailbox_exists($folder, true)) {
+ return true;
+ }
+
+ // folder may exist but isn't subscribed (#1485241)
+ if ($create) {
+ if (!$IMAP->mailbox_exists($folder))
+ return $IMAP->create_mailbox($folder, true);
+ else
+ return $IMAP->subscribe($folder);
+ }
+
+ return false;
+}
+
+
function get_form_tags($attrib)
{
global $RCMAIL, $MESSAGE_FORM;
@@ -1249,13 +1327,13 @@
$form_start = empty($attrib['form']) ? $RCMAIL->output->form_tag(array('name' => "form", 'method' => "post")) : '';
$form_start .= $hiddenfields->show();
}
-
+
$form_end = ($MESSAGE_FORM && !strlen($attrib['form'])) ? '</form>' : '';
$form_name = !empty($attrib['form']) ? $attrib['form'] : 'form';
-
+
if (!$MESSAGE_FORM)
$RCMAIL->output->add_gui_object('messageform', $form_name);
-
+
$MESSAGE_FORM = $form_name;
return array($form_start, $form_end);
--
Gitblit v1.9.1