alecpl
2011-10-03 56ec81cb6fafde1a662aac0a8feeb87e45b80bff
- Don't print error to the log when trying to delete non-existing draft message


1 files modified
18 ■■■■■ changed files
program/steps/mail/sendmail.inc 18 ●●●●● patch | view | raw | blame | history
program/steps/mail/sendmail.inc
@@ -640,8 +640,7 @@
else    
  $store_target = isset($_POST['_store_target']) ? get_input_value('_store_target', RCUBE_INPUT_POST) : $CONFIG['sent_mbox'];
if ($store_target)
  {
if ($store_target) {
  // check if folder is subscribed
  if ($IMAP->mailbox_exists($store_target, true))
    $store_folder = true;
@@ -653,7 +652,6 @@
  // append message to sent box
  if ($store_folder) {
    // message body in file
    if ($mailbody_file || $MAIL_MIME->getParam('delay_file_io')) {
      $headers = $MAIL_MIME->txtHeaders();
@@ -700,11 +698,12 @@
      }
    }
  if ($olddraftmessageid)
    {
  if ($olddraftmessageid) {
    // delete previous saved draft
    $a_deleteid = $IMAP->search_once($CONFIG['drafts_mbox'],
        'HEADER Message-ID '.$olddraftmessageid, true);
    if (!empty($a_deleteid)) {
    $deleted = $IMAP->delete_message($a_deleteid, $CONFIG['drafts_mbox']);
    // raise error if deletion of old draft failed
@@ -714,14 +713,14 @@
                'message' => "Could not delete message from ".$CONFIG['drafts_mbox']), TRUE, FALSE);
    }
  }
}
// remove temp file
else if ($mailbody_file) {
  unlink($mailbody_file);
  }
if ($savedraft)
  {
if ($savedraft) {
  $msgid = strtr($message_id, array('>' => '', '<' => ''));
  
  // remember new draft-uid
@@ -740,8 +739,7 @@
  $OUTPUT->send('iframe');
  }
else
  {
else {
  rcmail_compose_cleanup($COMPOSE_ID);
  if ($store_folder && !$saved)
@@ -750,5 +748,3 @@
    $OUTPUT->command('sent_successfully', 'confirmation', rcube_label('messagesent'));
  $OUTPUT->send('iframe');
  }