X-Git-Url: http://git.madism.org/?a=blobdiff_plain;f=imap%2Fmessage.c;h=264e65be0a3182d73f9aee57d912bdea0b481c0d;hb=2f0ad939a786df928a8b5d489692fbe11cf8c9d3;hp=08a9d3f72a879180f4355c65fa886383a8632140;hpb=0f44dc85fc1280372ffab911d701e703d803fb4b;p=apps%2Fmadmutt.git diff --git a/imap/message.c b/imap/message.c index 08a9d3f..264e65b 100644 --- a/imap/message.c +++ b/imap/message.c @@ -30,6 +30,7 @@ #include "lib/mem.h" #include "lib/intl.h" #include "lib/str.h" +#include "lib/debug.h" #if HAVE_STDINT_H #include @@ -161,9 +162,7 @@ int imap_read_headers (IMAP_DATA * idata, int msgbegin, int msgend) mutt_hcache_restore ((unsigned char *) uid_validity, 0); ctx->hdrs[msgno]->index = h.sid - 1; if (h.sid != ctx->msgcount + 1) - dprint (1, - (debugfile, - "imap_read_headers: msgcount and sequence ID are inconsistent!")); + debug_print (1, ("msgcount and sequence ID are inconsistent!\n")); /* messages which have not been expunged are ACTIVE (borrowed from mh * folders) */ ctx->hdrs[msgno]->active = 1; @@ -254,9 +253,7 @@ int imap_read_headers (IMAP_DATA * idata, int msgbegin, int msgend) ctx->hdrs[msgno]->index = h.sid - 1; if (h.sid != ctx->msgcount + 1) - dprint (1, - (debugfile, - "imap_read_headers: msgcount and sequence ID are inconsistent!")); + debug_print (1, ("msgcount and sequence ID are inconsistent!\n")); /* messages which have not been expunged are ACTIVE (borrowed from mh * folders) */ ctx->hdrs[msgno]->active = 1; @@ -557,8 +554,7 @@ int imap_append_message (CONTEXT * ctx, MESSAGE * msg) if (rc != IMAP_CMD_RESPOND) { char *pc; - dprint (1, (debugfile, "imap_append_message(): command failed: %s\n", - idata->cmd.buf)); + debug_print (1, ("command failed: %s\n", idata->cmd.buf)); pc = idata->cmd.buf + SEQLEN; SKIPWS (pc); @@ -594,8 +590,7 @@ int imap_append_message (CONTEXT * ctx, MESSAGE * msg) if (!imap_code (idata->cmd.buf)) { char *pc; - dprint (1, (debugfile, "imap_append_message(): command failed: %s\n", - idata->cmd.buf)); + debug_print (1, ("command failed: %s\n", idata->cmd.buf)); pc = idata->cmd.buf + SEQLEN; SKIPWS (pc); pc = imap_next_word (pc); @@ -621,37 +616,36 @@ fail: int imap_copy_messages (CONTEXT * ctx, HEADER * h, char *dest, int delete) { IMAP_DATA *idata; - BUFFER cmd; + BUFFER cmd, sync_cmd; char uid[11]; char mbox[LONG_STRING]; char mmbox[LONG_STRING]; int rc; int n; IMAP_MBOX mx; + int err_continue = M_NO; idata = (IMAP_DATA *) ctx->data; if (imap_parse_path (dest, &mx)) { - dprint (1, (debugfile, "imap_copy_messages: bad destination %s\n", dest)); + debug_print (1, ("bad destination %s\n", dest)); return -1; } /* check that the save-to folder is in the same account */ if (!mutt_account_match (&(CTX_DATA->conn->account), &(mx.account))) { - dprint (3, (debugfile, "imap_copy_messages: %s not same server as %s\n", - dest, ctx->path)); + debug_print (3, ("%s not same server as %s\n", dest, ctx->path)); return 1; } if (h && h->attach_del) { - dprint (3, - (debugfile, - "imap_copy_messages: Message contains attachments to be deleted\n")); + debug_print (3, ("Message contains attachments to be deleted\n")); return 1; } imap_fix_path (idata, mx.mbox, mbox, sizeof (mbox)); + memset (&sync_cmd, 0, sizeof (sync_cmd)); memset (&cmd, 0, sizeof (cmd)); mutt_buffer_addstr (&cmd, "UID COPY "); @@ -662,16 +656,25 @@ int imap_copy_messages (CONTEXT * ctx, HEADER * h, char *dest, int delete) * remainder. */ for (n = 0; n < ctx->msgcount; n++) { if (ctx->hdrs[n]->tagged && ctx->hdrs[n]->attach_del) { - dprint (3, - (debugfile, - "imap_copy_messages: Message contains attachments to be deleted\n")); + debug_print (3, ("Message contains attachments to be deleted\n")); return 1; } + + if (ctx->hdrs[n]->tagged && ctx->hdrs[n]->active && + ctx->hdrs[n]->changed) + { + rc = imap_sync_message (idata, ctx->hdrs[n], &sync_cmd, &err_continue); + if (rc < 0) + { + debug_print (1, ("could not sync\n")); + goto fail; + } + } } rc = imap_make_msg_set (idata, &cmd, M_TAG, 0); if (!rc) { - dprint (1, (debugfile, "imap_copy_messages: No messages tagged\n")); + debug_print (1, ("No messages tagged\n")); goto fail; } mutt_message (_("Copying %d messages to %s..."), rc, mbox); @@ -680,6 +683,16 @@ int imap_copy_messages (CONTEXT * ctx, HEADER * h, char *dest, int delete) mutt_message (_("Copying message %d to %s..."), h->index + 1, mbox); snprintf (uid, sizeof (uid), "%u", HEADER_DATA (h)->uid); mutt_buffer_addstr (&cmd, uid); + + if (h->active && h->changed) + { + rc = imap_sync_message (idata, h, &sync_cmd, &err_continue); + if (rc < 0) + { + debug_print (1, ("could not sync\n")); + goto fail; + } + } } /* let's get it on */ @@ -695,8 +708,7 @@ int imap_copy_messages (CONTEXT * ctx, HEADER * h, char *dest, int delete) imap_error ("imap_copy_messages", idata->cmd.buf); goto fail; } - dprint (2, - (debugfile, "imap_copy_messages: server suggests TRYCREATE\n")); + debug_print (2, ("server suggests TRYCREATE\n")); snprintf (mmbox, sizeof (mmbox), _("Create %s?"), mbox); if (option (OPTCONFIRMCREATE) && mutt_yesorno (mmbox, 1) < 1) { mutt_clear_error (); @@ -734,12 +746,16 @@ int imap_copy_messages (CONTEXT * ctx, HEADER * h, char *dest, int delete) if (cmd.data) FREE (&cmd.data); + if (sync_cmd.data) + FREE (&sync_cmd.data); FREE (&mx.mbox); return 0; fail: if (cmd.data) FREE (&cmd.data); + if (sync_cmd.data) + FREE (&sync_cmd.data); FREE (&mx.mbox); return -1; } @@ -785,7 +801,7 @@ char *imap_set_flags (IMAP_DATA * idata, HEADER * h, char *s) memset (&newh, 0, sizeof (newh)); newh.data = safe_calloc (1, sizeof (IMAP_HEADER_DATA)); - dprint (2, (debugfile, "imap_fetch_message: parsing FLAGS\n")); + debug_print (2, ("parsing FLAGS\n")); if ((s = msg_parse_flags (&newh, s)) == NULL) { FREE (&newh.data); return NULL; @@ -974,9 +990,7 @@ static int msg_parse_fetch (IMAP_HEADER * h, char *s) s += 12; SKIPWS (s); if (*s != '\"') { - dprint (1, - (debugfile, - "msg_parse_fetch(): bogus INTERNALDATE entry: %s\n", s)); + debug_print (1, ("bogus INTERNALDATE entry: %s\n", s)); return -1; } s++; @@ -1022,13 +1036,13 @@ static char *msg_parse_flags (IMAP_HEADER * h, char *s) /* sanity-check string */ if (ascii_strncasecmp ("FLAGS", s, 5) != 0) { - dprint (1, (debugfile, "msg_parse_flags: not a FLAGS response: %s\n", s)); + debug_print (1, ("not a FLAGS response: %s\n", s)); return NULL; } s += 5; SKIPWS (s); if (*s != '(') { - dprint (1, (debugfile, "msg_parse_flags: bogus FLAGS response: %s\n", s)); + debug_print (1, ("bogus FLAGS response: %s\n", s)); return NULL; } s++; @@ -1081,8 +1095,7 @@ static char *msg_parse_flags (IMAP_HEADER * h, char *s) s++; } else { - dprint (1, (debugfile, - "msg_parse_flags: Unterminated FLAGS response: %s\n", s)); + debug_print (1, ("Unterminated FLAGS response: %s\n", s)); return NULL; }