X-Git-Url: https://vcs.fsf.org/?p=squirrelmail.git;a=blobdiff_plain;f=src%2Fcompose.php;h=8bc437c8e81e1a561774e9d49f76c95ef561c3cd;hp=509ca0471c4d9d787f47e707f8511fe3473566c0;hb=210b5bb5d7f46c4c5c4bda8d567bef2f3bbef400;hpb=8df4c9294fffc58cba49b68d1ee4954a3168d326 diff --git a/src/compose.php b/src/compose.php index 509ca047..8bc437c8 100644 --- a/src/compose.php +++ b/src/compose.php @@ -1,11 +1,7 @@ setDelayedErrors(true); /** SESSION/POST/GET VARS */ sqgetGlobalVar('session',$session); @@ -77,22 +70,34 @@ sqgetGlobalVar('draft_id',$draft_id); sqgetGlobalVar('ent_num',$ent_num); sqgetGlobalVar('saved_draft',$saved_draft); sqgetGlobalVar('delete_draft',$delete_draft); -sqgetGlobalVar('startMessage',$startMessage); +if ( sqgetGlobalVar('startMessage',$startMessage) ) { + $startMessage = (int)$startMessage; +} else { + $startMessage = 1; +} + /** POST VARS */ -sqgetGlobalVar('sigappend', $sigappend, SQ_POST); -sqgetGlobalVar('from_htmladdr_search', $from_htmladdr_search, SQ_POST); -sqgetGlobalVar('addr_search_done', $html_addr_search_done, SQ_POST); -sqgetGlobalVar('send_to_search', $send_to_search, SQ_POST); -sqgetGlobalVar('do_delete', $do_delete, SQ_POST); -sqgetGlobalVar('delete', $delete, SQ_POST); -sqgetGlobalVar('restoremessages', $restoremessages, SQ_POST); +sqgetGlobalVar('sigappend', $sigappend, SQ_POST); +sqgetGlobalVar('from_htmladdr_search', $from_htmladdr_search, SQ_POST); +sqgetGlobalVar('addr_search_done', $html_addr_search_done, SQ_POST); +sqgetGlobalVar('addr_search_cancel', $html_addr_search_cancel, SQ_POST); +sqgetGlobalVar('send_to_search', $send_to_search, SQ_POST); +sqgetGlobalVar('do_delete', $do_delete, SQ_POST); +sqgetGlobalVar('delete', $delete, SQ_POST); +sqgetGlobalVar('restoremessages', $restoremessages, SQ_POST); if ( sqgetGlobalVar('return', $temp, SQ_POST) ) { $html_addr_search_done = 'Use Addresses'; } /** GET VARS */ sqgetGlobalVar('attachedmessages', $attachedmessages, SQ_GET); +if ( sqgetGlobalVar('account', $temp, SQ_GET) ) { + $iAccount = (int) $temp; +} else { + $iAccount = 0; +} + /** get smaction */ if ( !sqgetGlobalVar('smaction',$action) ) @@ -118,7 +123,6 @@ function replyAllString($header) { /** * 1) Remove the addresses we'll be sending the message 'to' */ - $url_replytoall_avoid_addrs = ''; if (isset($header->replyto)) { $excl_ar = $header->getAddr_a('replyto'); } @@ -158,55 +162,89 @@ function replyAllString($header) { return $url_replytoallcc; } +/** + * creates top line in reply citations + * + * Line style depends on user preferences. + * $orig_date argument is available only from 1.4.3 and 1.5.1 version. + * @param object $orig_from From: header object. + * @param integer $orig_date email's timestamp + * @return string reply citation + */ function getReplyCitation($orig_from, $orig_date) { global $reply_citation_style, $reply_citation_start, $reply_citation_end; - $orig_from = decodeHeader($orig_from->getAddress(false),false,false,true); -// $from = decodeHeader($orig_header->getAddr_s('from',"\n$indent"),false,false); + + if (!is_object($orig_from)) { + $sOrig_from = ''; + } else { + $sOrig_from = decodeHeader($orig_from->getAddress(false),false,false,true); + } + /* First, return an empty string when no citation style selected. */ if (($reply_citation_style == '') || ($reply_citation_style == 'none')) { return ''; } /* Make sure our final value isn't an empty string. */ - if ($orig_from == '') { + if ($sOrig_from == '') { return ''; } /* Otherwise, try to select the desired citation style. */ switch ($reply_citation_style) { - case 'author_said': - $start = ''; - $end = ' ' . _("said") . ':'; - break; - case 'quote_who': - $start = '<' . _("quote") . ' ' . _("who") . '="'; - $end = '">'; - break; - case 'date_time_author': - $start = 'On ' . getLongDateString($orig_date) . ', '; - $end = ' ' . _("said") . ':'; - break; - case 'user-defined': - $start = $reply_citation_start . - ($reply_citation_start == '' ? '' : ' '); - $end = $reply_citation_end; - break; - default: - return ''; - } - - /* Build and return the citation string. */ - return ($start . $orig_from . $end . "\n"); + case 'author_said': + /** + * To translators: %s is for author's name + */ + $full_reply_citation = sprintf(_("%s wrote:"),$sOrig_from); + break; + case 'quote_who': + $start = ''; + $full_reply_citation = $start . $sOrig_from . $end; + break; + case 'date_time_author': + /** + * To translators: + * first %s is for date string, second %s is for author's name. Date uses + * formating from "D, F j, Y g:i a" and "D, F j, Y H:i" translations. + * Example string: + * "On Sat, December 24, 2004 23:59, Santa wrote:" + * If you have to put author's name in front of date string, check comments about + * argument swapping at http://www.php.net/sprintf + */ + $full_reply_citation = sprintf(_("On %s, %s wrote:"), getLongDateString($orig_date), $sOrig_from); + break; + case 'user-defined': + $start = $reply_citation_start . + ($reply_citation_start == '' ? '' : ' '); + $end = $reply_citation_end; + $full_reply_citation = $start . $sOrig_from . $end; + break; + default: + return ''; + } + + /* Add line feed and return the citation string. */ + return ($full_reply_citation . "\n"); } +/** + * Creates header fields in forwarded email body + * + * $default_charset global must be set correctly before you call this function. + * @param object $orig_header + * @return $string + */ function getforwardHeader($orig_header) { - global $editor_size; - - $display = array( _("Subject") => strlen(_("Subject")), - _("From") => strlen(_("From")), - _("Date") => strlen(_("Date")), - _("To") => strlen(_("To")), - _("Cc") => strlen(_("Cc")) ); + global $editor_size, $default_charset; + + // using own strlen function in order to detect correct string length + $display = array( _("Subject") => sq_strlen(_("Subject"),$default_charset), + _("From") => sq_strlen(_("From"),$default_charset), + _("Date") => sq_strlen(_("Date"),$default_charset), + _("To") => sq_strlen(_("To"),$default_charset), + _("Cc") => sq_strlen(_("Cc"),$default_charset) ); $maxsize = max($display); $indent = str_pad('',$maxsize+2); foreach($display as $key => $val) { @@ -218,7 +256,9 @@ function getforwardHeader($orig_header) { $to = str_replace(' ',' ',$to); $subject = decodeHeader($orig_header->subject,false,false,true); $subject = str_replace(' ',' ',$subject); - $bodyTop = str_pad(' '._("Original Message").' ',$editor_size -2,'-',STR_PAD_BOTH) . + + // using own str_pad function in order to create correct string pad + $bodyTop = sq_str_pad(' '._("Original Message").' ',$editor_size -2,'-',STR_PAD_BOTH,$default_charset) . "\n". $display[_("Subject")] . $subject . "\n" . $display[_("From")] . $from . "\n" . $display[_("Date")] . getLongDateString( $orig_header->date ). "\n" . @@ -249,13 +289,19 @@ if (sqsession_is_registered('session_expired_post')) { sqsession_unregister('session_expired_post'); session_write_close(); } else { - foreach ($session_expired_post as $postvar => $val) { - if (isset($val)) { - $$postvar = $val; - } else { - $$postvar = ''; + // these are the vars that we can set from the expired composed session + $compo_var_list = array ( 'send_to', 'send_to_cc','body','startMessage', + 'passed_body','use_signature','signature','attachments','subject','newmail', + 'send_to_bcc', 'passed_id', 'mailbox', 'from_htmladdr_search', 'identity', + 'draft_id', 'delete_draft', 'mailprio', 'edit_as_new', 'compose_messsages', + 'composesession', 'request_mdn', 'request_dr'); + + foreach ($compo_var_list as $var) { + if ( isset($session_expired_post[$var]) && !isset($$var) ) { + $$var = $session_expired_post[$var]; } } + $compose_messages = unserialize(urldecode($restoremessages)); sqsession_register($compose_messages,'compose_messages'); sqsession_register($composesession,'composesession'); @@ -273,7 +319,13 @@ if (sqsession_is_registered('session_expired_post')) { if ($compose_new_win == '1') { compose_Header($color, $mailbox); } else { - displayPageHeader($color, $mailbox); + $sHeaderJs = (isset($sHeaderJs)) ? $sHeaderJs : ''; + if (strpos($action, 'reply') !== false && $reply_focus) { + $sBodyTagJs = 'onload="checkForm(\''.$replyfocus.'\');"'; + } else { + $sBodyTagJs = 'onload="checkForm();"'; + } + displayPageHeader($color, $mailbox,$sHeaderJs,$sBodyTagJs); } showInputForm($session, false); exit(); @@ -292,13 +344,14 @@ if (!isset($session) || (isset($newmessage) && $newmessage)) { if (!isset($compose_messages)) { $compose_messages = array(); } + if (!isset($compose_messages[$session]) || ($compose_messages[$session] == NULL)) { - /* if (!array_key_exists($session, $compose_messages)) { /* We can only do this in PHP >= 4.1 */ $composeMessage = new Message(); $rfc822_header = new Rfc822Header(); $composeMessage->rfc822_header = $rfc822_header; $composeMessage->reply_rfc822_header = ''; $compose_messages[$session] = $composeMessage; + sqsession_register($compose_messages,'compose_messages'); } else { $composeMessage=$compose_messages[$session]; @@ -323,20 +376,41 @@ if ($draft) { $draft_message = _("Draft Email Saved"); /* If this is a resumed draft, then delete the original */ if(isset($delete_draft)) { - Header("Location: $location/delete_message.php?mailbox=" . urlencode($draft_folder) . - "&message=$delete_draft&startMessage=1&saved_draft=yes"); - exit(); + $imap_stream = sqimap_login($username, false, $imapServerAddress, $imapPort, false); + sqimap_mailbox_select($imap_stream, $draft_folder); + // force bypass_trash=true because message should be saved when deliverMessage() returns true. + // in current implementation of sqimap_msgs_list_flag() single message id can + // be submitted as string. docs state that it should be array. + sqimap_msgs_list_delete($imap_stream, $draft_folder, $delete_draft, true); + if ($auto_expunge) { + sqimap_mailbox_expunge($imap_stream, $draft_folder, true); + } + sqimap_logout($imap_stream); } - else { - if ($compose_new_win == '1') { + + $oErrorHandler->saveDelayedErrors(); + session_write_close(); + + if ($compose_new_win == '1') { + if ( !isset($pageheader_sent) || !$pageheader_sent ) { Header("Location: $location/compose.php?saved_draft=yes&session=$composesession"); - exit(); + } else { + echo '

' + . _("Return") . '
'; } - else { + exit(); + } else { + if ( !isset($pageheader_sent) || !$pageheader_sent ) { Header("Location: $location/right_main.php?mailbox=" . urlencode($draft_folder) . - "&startMessage=1¬e=".urlencode($draft_message)); - exit(); + "&startMessage=1¬e=".urlencode($draft_message)); + } else { + echo '

' + . _("Return") . '
'; } + exit(); } } } @@ -350,7 +424,7 @@ if ($send) { if (checkInput(false) && !isset($AttachFailure)) { if ($mailbox == "All Folders") { /* We entered compose via the search results page */ - $mailbox="INBOX"; /* Send 'em to INBOX, that's safe enough */ + $mailbox = 'INBOX'; /* Send 'em to INBOX, that's safe enough */ } $urlMailbox = urlencode (trim($mailbox)); if (! isset($passed_id)) { @@ -377,10 +451,10 @@ if ($send) { if( $line <> '-- ' ) { $line = rtrim($line); } - if (strlen($line) <= $editor_size + 1) { + if (sq_strlen($line,$default_charset) <= $editor_size + 1) { $newBody .= $line . "\n"; } else { - sqWordWrap($line, $editor_size); + sqWordWrap($line, $editor_size,$default_charset); $newBody .= $line . "\n"; } @@ -391,23 +465,53 @@ if ($send) { $composeMessage=$compose_messages[$session]; $Result = deliverMessage($composeMessage); + + do_hook('compose_send_after', $Result, $composeMessage); if (! $Result) { showInputForm($session); exit(); } unset($compose_messages[$session]); + + /* if it is resumed draft, delete draft message */ if ( isset($delete_draft)) { - Header("Location: $location/delete_message.php?mailbox=" . urlencode( $draft_folder ). - "&message=$delete_draft&startMessage=1&mail_sent=yes"); - exit(); + $imap_stream = sqimap_login($username, false, $imapServerAddress, $imapPort, false); + sqimap_mailbox_select($imap_stream, $draft_folder); + // bypass_trash=true because message should be saved when deliverMessage() returns true. + // in current implementation of sqimap_msgs_list_flag() single message id can + // be submitted as string. docs state that it should be array. + sqimap_msgs_list_delete($imap_stream, $draft_folder, $delete_draft, true); + if ($auto_expunge) { + sqimap_mailbox_expunge($imap_stream, $draft_folder, true); + } + sqimap_logout($imap_stream); } - if ($compose_new_win == '1') { + /* + * Store the error array in the session because they will be lost on a redirect + */ + $oErrorHandler->saveDelayedErrors(); + session_write_close(); - Header("Location: $location/compose.php?mail_sent=yes"); - } - else { - Header("Location: $location/right_main.php?mailbox=$urlMailbox". + if ($compose_new_win == '1') { + if ( !isset($pageheader_sent) || !$pageheader_sent ) { + Header("Location: $location/compose.php?mail_sent=yes"); + } else { + echo '

' + . _("Return") . '
'; + } + exit(); + } else { + if ( !isset($pageheader_sent) || !$pageheader_sent ) { + Header("Location: $location/right_main.php?mailbox=$urlMailbox". "&startMessage=$startMessage&mail_sent=yes"); + } else { + echo '

" + . _("Return") . '
'; + } + exit(); } } else { if ($compose_new_win == '1') { @@ -455,12 +559,12 @@ if ($send) { } } showInputForm($session); -} elseif (isset($html_addr_search)) { +} elseif (isset($html_addr_search) && !isset($html_addr_search_cancel)) { if (isset($_FILES['attachfile']) && $_FILES['attachfile']['tmp_name'] && $_FILES['attachfile']['tmp_name'] != 'none') { if(saveAttachedFiles($session)) { - plain_error_message(_("Could not move/copy file. File not attached"), $color); + plain_error_message(_("Could not move/copy file. File not attached")); } } /* @@ -469,14 +573,14 @@ if ($send) { */ include_once('./addrbook_search_html.php'); } elseif (isset($attach)) { - if (saveAttachedFiles($session)) { - plain_error_message(_("Could not move/copy file. File not attached"), $color); - } if ($compose_new_win == '1') { compose_Header($color, $mailbox); } else { displayPageHeader($color, $mailbox); } + if (saveAttachedFiles($session)) { + plain_error_message(_("Could not move/copy file. File not attached")); + } showInputForm($session); } elseif (isset($sigappend)) { @@ -499,9 +603,10 @@ elseif (isset($sigappend)) { if (isset($delete) && is_array($delete)) { $composeMessage = $compose_messages[$session]; foreach($delete as $index) { - $attached_file = $composeMessage->entities[$index]->att_local_name; - unlink ($attached_file); - unset ($composeMessage->entities[$index]); + if (!empty($composeMessage->entities) && isset($composeMessage->entities[$index])) { + $composeMessage->entities[$index]->purgeAttachments(); + unset ($composeMessage->entities[$index]); + } } $new_entities = array(); foreach ($composeMessage->entities as $entity) { @@ -574,9 +679,10 @@ function getforwardSubject($subject) /* This function is used when not sending or adding attachments */ function newMail ($mailbox='', $passed_id='', $passed_ent_id='', $action='', $session='') { global $editor_size, $default_use_priority, $body, $idents, - $use_signature, $composesession, $data_dir, $username, - $username, $key, $imapServerAddress, $imapPort, $compose_messages, - $composeMessage, $body_quote; + $use_signature, $data_dir, $username, + $key, $imapServerAddress, $imapPort, $compose_messages, + $composeMessage, $body_quote, $request_mdn, $request_dr, + $default_use_mdn, $mdn_user_support; global $languages, $squirrelmail_language, $default_charset; /* @@ -590,7 +696,7 @@ function newMail ($mailbox='', $passed_id='', $passed_ent_id='', $action='', $se $mailprio = 3; if ($passed_id) { - $imapConnection = sqimap_login($username, $key, $imapServerAddress, + $imapConnection = sqimap_login($username, false, $imapServerAddress, $imapPort, 0); sqimap_mailbox_select($imapConnection, $mailbox); @@ -623,7 +729,6 @@ function newMail ($mailbox='', $passed_id='', $passed_ent_id='', $action='', $se $orig_header = $message->rfc822_header; } - $encoding = $message->header->encoding; $type0 = $message->type0; $type1 = $message->type1; foreach ($entities as $ent) { @@ -632,7 +737,7 @@ function newMail ($mailbox='', $passed_id='', $passed_ent_id='', $action='', $se $type1 = $msg->type1; $unencoded_bodypart = mime_fetch_body($imapConnection, $passed_id, $ent); $body_part_entity = $message->getEntity($ent); - $bodypart = decodeBody($unencoded_bodypart, + $bodypart = decodeBody($unencoded_bodypart, $body_part_entity->header->encoding); if ($type1 == 'html') { $bodypart = str_replace("\n", ' ', $bodypart); @@ -647,6 +752,7 @@ function newMail ($mailbox='', $passed_id='', $passed_ent_id='', $action='', $se } } + // charset encoding in compose form stuff if (isset($body_part_entity->header->parameters['charset'])) { $actual = $body_part_entity->header->parameters['charset']; } else { @@ -656,6 +762,7 @@ function newMail ($mailbox='', $passed_id='', $passed_ent_id='', $action='', $se if ( $actual && is_conversion_safe($actual) && $actual != $default_charset){ $bodypart = charset_convert($actual,$bodypart,$default_charset,false); } + // end of charset encoding in compose $body .= $bodypart; } @@ -667,7 +774,6 @@ function newMail ($mailbox='', $passed_id='', $passed_ent_id='', $action='', $se } else { $mailprio = ''; } - //ClearAttachments($session); $identity = ''; $from_o = $orig_header->from; @@ -720,6 +826,11 @@ function newMail ($mailbox='', $passed_id='', $passed_ent_id='', $action='', $se } } $subject = decodeHeader($orig_header->subject,false,false,true); + + // Remember the receipt settings + $request_mdn = $mdn_user_support && !empty($orig_header->dnt) ? '1' : '0'; + $request_dr = $mdn_user_support && !empty($orig_header->drnt) ? '1' : '0'; + /* remember the references and in-reply-to headers in case of an reply */ $composeMessage->rfc822_header->more_headers['References'] = $orig_header->references; $composeMessage->rfc822_header->more_headers['In-Reply-To'] = $orig_header->in_reply_to; @@ -746,6 +857,7 @@ function newMail ($mailbox='', $passed_id='', $passed_ent_id='', $action='', $se // forwarded message text should be as undisturbed as possible, so commenting out this call // sqUnWordWrap($body); $composeMessage = getAttachments($message, $composeMessage, $passed_id, $entities, $imapConnection); + //add a blank line after the forward headers $body = "\n" . $body; break; @@ -823,6 +935,16 @@ function newMail ($mailbox='', $passed_id='', $passed_ent_id='', $action='', $se return ($ret); } /* function newMail() */ +/** + * downloads attachments from original message, stores them in attachment directory and adds + * them to composed message. + * @param object $message + * @param object $composeMessage + * @param integer $passed_id + * @param mixed $entities + * @param mixed $imapConnection + * @return object + */ function getAttachments($message, &$composeMessage, $passed_id, $entities, $imapConnection) { global $attachment_dir, $username, $data_dir, $squirrelmail_language, $languages; $hashed_attachment_dir = getHashedDir($username, $attachment_dir); @@ -866,9 +988,7 @@ function getAttachments($message, &$composeMessage, $passed_id, $entities, $imap /* Write Attachment to file */ $fp = fopen ("$hashed_attachment_dir/$localfilename", 'wb'); - fputs($fp, decodeBody(mime_fetch_body($imapConnection, - $passed_id, $message->entity_id), - $message->header->encoding)); + mime_print_body_lines ($imapConnection, $passed_id, $message->entity_id, $message->header->encoding, $fp); fclose ($fp); } } else { @@ -881,7 +1001,7 @@ function getAttachments($message, &$composeMessage, $passed_id, $entities, $imap function getMessage_RFC822_Attachment($message, $composeMessage, $passed_id, $passed_ent_id='', $imapConnection) { - global $attachments, $attachment_dir, $username, $data_dir; + global $attachment_dir, $username, $data_dir; $hashed_attachment_dir = getHashedDir($username, $attachment_dir); if (!$passed_ent_id) { $body_a = sqimap_run_command($imapConnection, @@ -913,15 +1033,24 @@ function getMessage_RFC822_Attachment($message, $composeMessage, $passed_id, } function showInputForm ($session, $values=false) { - global $send_to, $send_to_cc, $body, $startMessage, - $passed_body, $color, $use_signature, $signature, $prefix_sig, - $editor_size, $editor_height, $attachments, $subject, $newmail, + global $send_to, $send_to_cc, $body, $startMessage, $action, + $color, $use_signature, $signature, $prefix_sig, + $editor_size, $editor_height, $subject, $newmail, $use_javascript_addr_book, $send_to_bcc, $passed_id, $mailbox, $from_htmladdr_search, $location_of_buttons, $attachment_dir, - $username, $data_dir, $identity, $idents, $draft_id, $delete_draft, - $mailprio, $default_use_mdn, $mdn_user_support, $compose_new_win, - $saved_draft, $mail_sent, $sig_first, $edit_as_new, $action, - $username, $compose_messages, $composesession, $default_charset; + $username, $data_dir, $identity, $idents, $delete_draft, + $mailprio, $compose_new_win, $saved_draft, $mail_sent, $sig_first, + $compose_messages, $composesession, $default_charset, + $compose_onsubmit, $oTemplate, $oErrorHandler; + + if (checkForJavascript()) { + $onfocus = ' onfocus="alreadyFocused=true;"'; + $onfocus_array = array('onfocus' => 'alreadyFocused=true;'); + } + else { + $onfocus = ''; + $onfocus_array = array(); + } $composeMessage = $compose_messages[$session]; if ($values) { @@ -939,7 +1068,7 @@ function showInputForm ($session, $values=false) { } if ($use_javascript_addr_book) { - echo "\n". '\n"; } else { - echo ' ' . "\n"; } - if ($save_as_draft) { - echo ' \n"; - } + $oTemplate->assign('allow_priority', $default_use_priority==1); + $oTemplate->assign('priority_list', $priorities); + $oTemplate->assign('current_priority', $priority); + + $oTemplate->assign('notifications_enabled', $mdn_user_support==1); + $oTemplate->assign('read_receipt', $request_mdn=='1'); + $oTemplate->assign('delivery_receipt', $request_dr=='1'); + + $oTemplate->assign('drafts_enabled', $save_as_draft); + $oTemplate->assign('address_book_button', $addr_book); - echo ' ' . "\n"; - do_hook('compose_button_row'); - - echo ' ' . "\n" . - ' ' . "\n\n"; + $oTemplate->display('compose_buttons.tpl'); } function checkInput ($show) { @@ -1260,11 +1338,11 @@ function checkInput ($show) { * using $show=false, and then when i'm ready to display the error * message, show=true */ - global $body, $send_to, $send_to_bcc, $subject, $color; + global $send_to, $send_to_bcc; if ($send_to == '' && $send_to_bcc == '') { if ($show) { - plain_error_message(_("You have not filled in the \"To:\" field."), $color); + plain_error_message(_("You have not filled in the \"To:\" field.")); } return false; } @@ -1274,7 +1352,7 @@ function checkInput ($show) { /* True if FAILURE */ function saveAttachedFiles($session) { - global $_FILES, $attachment_dir, $attachments, $username, + global $_FILES, $attachment_dir, $username, $data_dir, $compose_messages; /* get out of here if no file was attached at all */ @@ -1290,10 +1368,10 @@ function saveAttachedFiles($session) { $full_localfilename = "$hashed_attachment_dir/$localfilename"; } - // FIXME: we SHOULD prefer move_uploaded_file over rename because - // m_u_f works better with restricted PHP installes (safe_mode, open_basedir) - if (!@rename($_FILES['attachfile']['tmp_name'], $full_localfilename)) { - if (!@move_uploaded_file($_FILES['attachfile']['tmp_name'],$full_localfilename)) { + // m_u_f works better with restricted PHP installs (safe_mode, open_basedir), + // if that doesn't work, try a simple rename. + if (!@move_uploaded_file($_FILES['attachfile']['tmp_name'],$full_localfilename)) { + if (!@rename($_FILES['attachfile']['tmp_name'], $full_localfilename)) { return true; } } @@ -1305,18 +1383,6 @@ function saveAttachedFiles($session) { sqsession_register($compose_messages , 'compose_messages'); } -function ClearAttachments($composeMessage) { - if ($composeMessage->att_local_name) { - $attached_file = $composeMessage->att_local_name; - if (file_exists($attached_file)) { - unlink($attached_file); - } - } - for ($i=0, $entCount=count($composeMessage->entities);$i< $entCount; ++$i) { - ClearAttachments($composeMessage->entities[$i]); - } -} - /* parse values like 8M and 2k into bytes */ function getByteSize($ini_size) { @@ -1348,12 +1414,12 @@ function getByteSize($ini_size) { } -/* temporary function to make use of the deliver class. - In the future the responsable backend should be automaticly loaded - and conf.pl should show a list of available backends. - The message also should be constructed by the message class. +/** + * temporary function to make use of the deliver class. + * In the future the responsible backend should be automaticly loaded + * and conf.pl should show a list of available backends. + * The message also should be constructed by the message class. */ - function deliverMessage($composeMessage, $draft=false) { global $send_to, $send_to_cc, $send_to_bcc, $mailprio, $subject, $body, $username, $popuser, $usernamedata, $identity, $idents, $data_dir, @@ -1413,11 +1479,17 @@ function deliverMessage($composeMessage, $draft=false) { /* Receipt: On Read */ if (isset($request_mdn) && $request_mdn) { $rfc822_header->dnt = $rfc822_header->parseAddress($from_mail,true); + } elseif (isset($rfc822_header->dnt)) { + unset($rfc822_header->dnt); } + /* Receipt: On Delivery */ if (isset($request_dr) && $request_dr) { $rfc822_header->more_headers['Return-Receipt-To'] = $from_mail; + } elseif (isset($rfc822_header->more_headers['Return-Receipt-To'])) { + unset($rfc822_header->more_headers['Return-Receipt-To']); } + /* multipart messages */ if (count($composeMessage->entities)) { $message_body = new Message(); @@ -1451,7 +1523,7 @@ function deliverMessage($composeMessage, $draft=false) { $rfc822_header->content_type = $content_type; $composeMessage->rfc822_header = $rfc822_header; - + /* Here you can modify the message structure just before we hand it over to deliver */ $hookReturn = do_hook('compose_send', $composeMessage); @@ -1463,7 +1535,7 @@ function deliverMessage($composeMessage, $draft=false) { if (!$useSendmail && !$draft) { require_once(SM_PATH . 'class/deliver/Deliver_SMTP.class.php'); $deliver = new Deliver_SMTP(); - global $smtpServerAddress, $smtpPort, $pop_before_smtp, $smtp_auth_mech; + global $smtpServerAddress, $smtpPort, $pop_before_smtp; $authPop = (isset($pop_before_smtp) && $pop_before_smtp) ? true : false; get_smtp_user($user, $pass); @@ -1471,13 +1543,21 @@ function deliverMessage($composeMessage, $draft=false) { $smtpServerAddress, $smtpPort, $user, $pass, $authPop); } elseif (!$draft) { require_once(SM_PATH . 'class/deliver/Deliver_SendMail.class.php'); - global $sendmail_path; - $deliver = new Deliver_SendMail(); + global $sendmail_path, $sendmail_args; + // Check for outdated configuration + if (!isset($sendmail_args)) { + if ($sendmail_path=='/var/qmail/bin/qmail-inject') { + $sendmail_args = ''; + } else { + $sendmail_args = '-i -t'; + } + } + $deliver = new Deliver_SendMail(array('sendmail_args'=>$sendmail_args)); $stream = $deliver->initStream($composeMessage,$sendmail_path); } elseif ($draft) { global $draft_folder; require_once(SM_PATH . 'class/deliver/Deliver_IMAP.class.php'); - $imap_stream = sqimap_login($username, $key, $imapServerAddress, + $imap_stream = sqimap_login($username, false, $imapServerAddress, $imapPort, 0); if (sqimap_mailbox_exists ($imap_stream, $draft_folder)) { require_once(SM_PATH . 'class/deliver/Deliver_IMAP.class.php'); @@ -1488,27 +1568,34 @@ function deliverMessage($composeMessage, $draft=false) { sqimap_append_done ($imap_stream, $draft_folder); sqimap_logout($imap_stream); unset ($imap_deliver); + $composeMessage->purgeAttachments(); return $length; } else { - $msg = '
'.sprintf(_("Error: Draft folder %s does not exist."), $draft_folder); - plain_error_message($msg, $color); + $msg = '
'.sprintf(_("Error: Draft folder %s does not exist."), htmlspecialchars($draft_folder)); + plain_error_message($msg); return false; } } - $succes = false; + $success = false; if ($stream) { $length = $deliver->mail($composeMessage, $stream); - $succes = $deliver->finalizeStream($stream); - } - if (!$succes) { - $msg = $deliver->dlv_msg . '
' . - _("Server replied: ") . $deliver->dlv_ret_nr . ' '. - $deliver->dlv_server_msg; - plain_error_message($msg, $color); + $success = $deliver->finalizeStream($stream); + } + if (!$success) { + // $deliver->dlv_server_msg is not always server's reply + $msg = $deliver->dlv_msg; + if (!empty($deliver->dlv_server_msg)) { + // add 'server replied' part only when it is not empty. + // Delivery error can be generated by delivery class itself + $msg.='
' . + _("Server replied:") . ' ' . $deliver->dlv_ret_nr . ' ' . + nl2br(htmlspecialchars($deliver->dlv_server_msg)); + } + plain_error_message($msg); } else { unset ($deliver); $move_to_sent = getPref($data_dir,$username,'move_to_sent'); - $imap_stream = sqimap_login($username, $key, $imapServerAddress, $imapPort, 0); + $imap_stream = sqimap_login($username, false, $imapServerAddress, $imapPort, 0); /* Move to sent code */ if (isset($default_move_to_sent) && ($default_move_to_sent != 0)) { @@ -1545,16 +1632,57 @@ function deliverMessage($composeMessage, $draft=false) { sqimap_append_done ($imap_stream, $sent_folder); unset ($imap_deliver); } - global $passed_id, $mailbox, $action; - ClearAttachments($composeMessage); - if ($action == 'reply' || $action == 'reply_all') { - sqimap_mailbox_select ($imap_stream, $mailbox); - sqimap_messages_flag ($imap_stream, $passed_id, $passed_id, 'Answered', false); + + global $passed_id, $mailbox, $action, $what, $iAccount,$startMessage; + + $composeMessage->purgeAttachments(); + if ($action=='reply' || $action=='reply_all' || $action=='forward' || $action=='forward_as_attachment') { + require(SM_PATH . 'functions/mailbox_display.php'); + $aMailbox = sqm_api_mailbox_select($imap_stream, $iAccount, $mailbox,array('setindex' => $what, 'offset' => $startMessage),array()); + switch($action) { + case 'reply': + case 'reply_all': + // check if we are allowed to set the \\Answered flag + if (in_array('\\answered',$aMailbox['PERMANENTFLAGS'], true)) { + $aUpdatedMsgs = sqimap_toggle_flag($imap_stream, array($passed_id), '\\Answered', true, false); + if (isset($aUpdatedMsgs[$passed_id]['FLAGS'])) { + /** + * Only update the cached headers if the header is + * cached. + */ + if (isset($aMailbox['MSG_HEADERS'][$passed_id])) { + $aMailbox['MSG_HEADERS'][$passed_id]['FLAGS'] = $aMsg['FLAGS']; + } + } + } + break; + case 'forward': + case 'forward_as_attachment': + // check if we are allowed to set the $Forwarded flag (RFC 4550 paragraph 2.8) + if (in_array('$forwarded',$aMailbox['PERMANENTFLAGS'], true) || + in_array('\\*',$aMailbox['PERMANENTFLAGS'])) { + + $aUpdatedMsgs = sqimap_toggle_flag($imap_stream, array($passed_id), '$Forwarded', true, false); + if (isset($aUpdatedMsgs[$passed_id]['FLAGS'])) { + if (isset($aMailbox['MSG_HEADERS'][$passed_id])) { + $aMailbox['MSG_HEADERS'][$passed_id]['FLAGS'] = $aMsg['FLAGS']; + } + } + } + break; + } + + /** + * Write mailbox with updated seen flag information back to cache. + */ + if(isset($aUpdatedMsgs[$passed_id])) { + $mailbox_cache[$iAccount.'_'.$aMailbox['NAME']] = $aMailbox; + sqsession_register($mailbox_cache,'mailbox_cache'); + } + + sqimap_logout($imap_stream); } - sqimap_logout($imap_stream); } - return $succes; + return $success; } - -// vim: et ts=4 ?> \ No newline at end of file