X-Git-Url: https://vcs.fsf.org/?a=blobdiff_plain;f=src%2Fcompose.php;h=a76e7e65abdc9ac9ad2650bd07880fcbc8bbeba6;hb=7e2ddf11abed8b8a93bc437fde95cfeba987a091;hp=abcdf9d8e89254bb3f573611c2e3bd88e58d0927;hpb=47ccfad452e8d345542d09e59112cac317cffed8;p=squirrelmail.git
diff --git a/src/compose.php b/src/compose.php
index abcdf9d8..a76e7e65 100644
--- a/src/compose.php
+++ b/src/compose.php
@@ -1,5 +1,4 @@
setDelayedErrors(true);
/** SESSION/POST/GET VARS */
sqgetGlobalVar('session',$session);
@@ -84,13 +78,14 @@ if ( sqgetGlobalVar('startMessage',$startMessage) ) {
/** 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';
}
@@ -294,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');
@@ -375,7 +376,7 @@ if ($draft) {
$draft_message = _("Draft Email Saved");
/* If this is a resumed draft, then delete the original */
if(isset($delete_draft)) {
- $imap_stream = sqimap_login($username, $key, $imapServerAddress, $imapPort, false);
+ $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
@@ -386,14 +387,17 @@ if ($draft) {
}
sqimap_logout($imap_stream);
}
+
+ $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");
} else {
- echo '
'
- . _("Return") . ' ';
+ . _("Return") . '';
}
exit();
} else {
@@ -401,10 +405,10 @@ if ($draft) {
Header("Location: $location/right_main.php?mailbox=" . urlencode($draft_folder) .
"&startMessage=1¬e=".urlencode($draft_message));
} else {
- echo ' '
- . _("Return") . ' ';
+ . _("Return") . '';
}
exit();
}
@@ -461,16 +465,17 @@ 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)) {
- $imap_stream = sqimap_login($username, $key, $imapServerAddress, $imapPort, false);
+ $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
@@ -481,14 +486,19 @@ if ($send) {
}
sqimap_logout($imap_stream);
}
+ /*
+ * Store the error array in the session because they will be lost on a redirect
+ */
+ $oErrorHandler->saveDelayedErrors();
session_write_close();
+
if ($compose_new_win == '1') {
if ( !isset($pageheader_sent) || !$pageheader_sent ) {
Header("Location: $location/compose.php?mail_sent=yes");
} else {
- echo ' '
- . _("Return") . ' ';
+ . _("Return") . '';
}
exit();
} else {
@@ -496,10 +506,10 @@ if ($send) {
Header("Location: $location/right_main.php?mailbox=$urlMailbox".
"&startMessage=$startMessage&mail_sent=yes");
} else {
- echo ' "
- . _("Return") . ' ';
+ . _("Return") . '';
}
exit();
}
@@ -549,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"));
}
}
/*
@@ -563,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)) {
@@ -670,8 +680,9 @@ function getforwardSubject($subject)
function newMail ($mailbox='', $passed_id='', $passed_ent_id='', $action='', $session='') {
global $editor_size, $default_use_priority, $body, $idents,
$use_signature, $data_dir, $username,
- $username, $key, $imapServerAddress, $imapPort, $compose_messages,
- $composeMessage, $body_quote;
+ $key, $imapServerAddress, $imapPort, $compose_messages,
+ $composeMessage, $body_quote, $request_mdn, $request_dr,
+ $default_use_mdn, $mdn_user_support;
global $languages, $squirrelmail_language, $default_charset;
/*
@@ -685,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);
@@ -726,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);
@@ -815,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;
@@ -1024,8 +1040,8 @@ function showInputForm ($session, $values=false) {
$from_htmladdr_search, $location_of_buttons, $attachment_dir,
$username, $data_dir, $identity, $idents, $delete_draft,
$mailprio, $compose_new_win, $saved_draft, $mail_sent, $sig_first,
- $username, $compose_messages, $composesession, $default_charset,
- $compose_onsubmit;
+ $compose_messages, $composesession, $default_charset,
+ $compose_onsubmit, $oTemplate, $oErrorHandler;
if (checkForJavascript()) {
$onfocus = ' onfocus="alreadyFocused=true;"';
@@ -1052,7 +1068,7 @@ function showInputForm ($session, $values=false) {
}
if ($use_javascript_addr_book) {
- echo "\n". '\n".
@@ -1387,19 +1313,22 @@ function showComposeButtonRow() {
_("Addresses").'" />'.
" \n";
} else {
- echo ' ' . "\n";
}
- if ($save_as_draft) {
- echo ' \n";
- }
-
- echo ' ' . "\n";
- do_hook('compose_button_row');
+ $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" .
- ' ' . "\n\n";
+ $oTemplate->display('compose_buttons.tpl');
}
function checkInput ($show) {
@@ -1409,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;
}
@@ -1550,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();
@@ -1588,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);
@@ -1609,12 +1544,20 @@ function deliverMessage($composeMessage, $draft=false) {
} elseif (!$draft) {
require_once(SM_PATH . 'class/deliver/Deliver_SendMail.class.php');
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');
@@ -1629,7 +1572,7 @@ function deliverMessage($composeMessage, $draft=false) {
return $length;
} else {
$msg = ' '.sprintf(_("Error: Draft folder %s does not exist."), htmlspecialchars($draft_folder));
- plain_error_message($msg, $color);
+ plain_error_message($msg);
return false;
}
}
@@ -1640,14 +1583,19 @@ function deliverMessage($composeMessage, $draft=false) {
}
if (!$success) {
// $deliver->dlv_server_msg is not always server's reply
- $msg = $deliver->dlv_msg . ' ' .
- _("Server replied:") . ' ' . $deliver->dlv_ret_nr . ' ' .
- $deliver->dlv_server_msg;
- plain_error_message($msg, $color);
+ $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)) {
@@ -1689,16 +1637,19 @@ function deliverMessage($composeMessage, $draft=false) {
$composeMessage->purgeAttachments();
if ($action == 'reply' || $action == 'reply_all') {
+ require(SM_PATH . 'functions/mailbox_display.php');
$aMailbox = sqm_api_mailbox_select($imap_stream, $iAccount, $mailbox,array('setindex' => $what, 'offset' => $startMessage),array());
- //sqimap_mailbox_select ($imap_stream, $mailbox);
- $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'];
+ // 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'];
+ }
}
}
/**
@@ -1711,5 +1662,4 @@ function deliverMessage($composeMessage, $draft=false) {
}
return $success;
}
-
-?>
+?>
\ No newline at end of file