From 46f7b7096450939fe03c95aa81ce06ae4bfca89d Mon Sep 17 00:00:00 2001 From: Aleksander Machniak <alec@alec.pl> Date: Mon, 28 Mar 2016 06:51:43 -0400 Subject: [PATCH] Enable reply/reply-all/forward buttons also in preview frame of message/rfc822 --- program/steps/mail/sendmail.inc | 32 ++++++++++++++++++++++---------- 1 files changed, 22 insertions(+), 10 deletions(-) diff --git a/program/steps/mail/sendmail.inc b/program/steps/mail/sendmail.inc index f797e2d..68c444e 100644 --- a/program/steps/mail/sendmail.inc +++ b/program/steps/mail/sendmail.inc @@ -189,11 +189,17 @@ } // remember reply/forward UIDs in special headers -if (!empty($COMPOSE['reply_uid']) && $savedraft) { - $headers['X-Draft-Info'] = array('type' => 'reply', 'uid' => $COMPOSE['reply_uid']); -} -else if (!empty($COMPOSE['forward_uid']) && $savedraft) { - $headers['X-Draft-Info'] = array('type' => 'forward', 'uid' => rcube_imap_generic::compressMessageSet($COMPOSE['forward_uid'])); +if ($savedraft) { + // Note: We ignore <UID>.<PART> forwards/replies here + if (($uid = $COMPOSE['reply_uid']) && !preg_match('/^\d+[0-9.]+$/', $uid)) { + $headers['X-Draft-Info'] = array('type' => 'reply', 'uid' => $uid); + } + else if (!empty($COMPOSE['forward_uid']) + && ($uid = rcube_imap_generic::compressMessageSet($COMPOSE['forward_uid'])) + && !preg_match('/^\d+[0-9.]+$/', $uid) + ) { + $headers['X-Draft-Info'] = array('type' => 'forward', 'uid' => $uid); + } } if (!empty($COMPOSE['reply_msgid'])) { @@ -234,7 +240,7 @@ // sending aborted by plugin if ($data['abort'] && !$savedraft) { - $OUTPUT->show_message($data['message'] ? $data['message'] : 'sendingfailed'); + $OUTPUT->show_message($data['message'] ?: 'sendingfailed'); $OUTPUT->send('iframe'); } else { @@ -455,7 +461,7 @@ } else { $ctype = str_replace('image/pjpeg', 'image/jpeg', $attachment['mimetype']); // #1484914 - $file = $attachment['data'] ? $attachment['data'] : $attachment['path']; + $file = $attachment['data'] ?: $attachment['path']; $folding = (int) $RCMAIL->config->get('mime_param_folding'); $MAIL_MIME->addAttachment($file, @@ -550,12 +556,18 @@ // set replied/forwarded flag if ($COMPOSE['reply_uid']) { foreach (rcmail::get_uids($COMPOSE['reply_uid'], $COMPOSE['mailbox']) as $mbox => $uids) { - $RCMAIL->storage->set_flag($uids, 'ANSWERED', $mbox); + // skip <UID>.<PART> replies + if (!preg_match('/^\d+[0-9.]+$/', implode(',', (array) $uids))) { + $RCMAIL->storage->set_flag($uids, 'ANSWERED', $mbox); + } } } else if ($COMPOSE['forward_uid']) { foreach (rcmail::get_uids($COMPOSE['forward_uid'], $COMPOSE['mailbox']) as $mbox => $uids) { - $RCMAIL->storage->set_flag($uids, 'FORWARDED', $mbox); + // skip <UID>.<PART> forwards + if (!preg_match('/^\d+[0-9.]+$/', implode(',', (array) $uids))) { + $RCMAIL->storage->set_flag($uids, 'FORWARDED', $mbox); + } } } } @@ -673,7 +685,7 @@ array('msgid' => $message_id, 'uid' => $saved, 'folder' => $store_target)); // display success - $OUTPUT->show_message($plugin['message'] ? $plugin['message'] : 'messagesaved', 'confirmation'); + $OUTPUT->show_message($plugin['message'] ?: 'messagesaved', 'confirmation'); // update "_draft_saveid" and the "cmp_hash" to prevent "Unsaved changes" warning $COMPOSE['param']['draft_uid'] = $plugin['uid']; -- Gitblit v1.9.1