Remove \Seen flag setting for imap trash. (see #3966) (see #3860)

Commit 323e3d6e5e4c has a side effect where spurious FETCH flag
updates after setting the \Seen flag can cause a sync to abort.

Remove manually setting \Seen for all trashed message before copying.

The next commit will change the imap trash function to use the same
code as the imap copy/save function for syncing the message before
server-side copying.

diff --git a/imap/imap.c b/imap/imap.c
index 7bc67c5225e311bfe6cc36dd265b9c32c5d91786..517741c4a8bddd91a508b5cf99674a35057dfdb6 100644 (file)
--- a/imap/imap.c
+++ b/imap/imap.c
@@ -2241,13 +2241,6 @@ int imap_fast_trash(struct Context *ctx, char *dest)
   /* loop in case of TRYCREATE */
   do
   {
-    rc = imap_exec_msgset(idata, "UID STORE", "+FLAGS.SILENT (\\Seen)", MUTT_TRASH, 0, 0);
-    if (rc < 0)
-    {
-      mutt_debug(1, "imap_fast_trash: Unable to mark messages as seen\n");
-      goto out;
-    }
-
     rc = imap_exec_msgset(idata, "UID COPY", mmbox, MUTT_TRASH, 0, 0);
     if (!rc)
     {