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 6e0021a6bee9343289d1d378bfd70235e39037e2..d156e44019b8af35f2d42c8bf8469ff8d0a9c5fa 100644 (file)
--- a/imap/imap.c
+++ b/imap/imap.c
@@ -2151,14 +2151,6 @@ int imap_fast_trash (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)
-    {
-      dprint (1, (debugfile, "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)
     {