From bb90ed5446c3884c1c41e0d21c65786c8eb07f37 Mon Sep 17 00:00:00 2001 From: Jakub Cuth Date: Mon, 20 Sep 2021 14:36:20 +0000 Subject: [PATCH] GODT-966: Fixed comment. --- internal/imap/mailbox_append.go | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/internal/imap/mailbox_append.go b/internal/imap/mailbox_append.go index 1694752f..70034a35 100644 --- a/internal/imap/mailbox_append.go +++ b/internal/imap/mailbox_append.go @@ -164,7 +164,9 @@ func (im *imapMailbox) labelExistingMessage(msg storeMessageProvider) error { // storeMBox := im.storeMailbox // Outlook Uses APPEND instead of COPY. There is no need to copy to All Mail because messages are already there. - // if the message is copied from Spam or Trash, it should stay as is. Return error. + // If the message is copied from Spam or Trash, it must be moved otherwise we will have data loss. + // If the message is moved from any folder, the moment when expunge happens on source we will move message trash unless we move it to archive. + // If the message is already in Archive we should not call API at all. // Otherwise the message is already in All mail, Return OK. if pmapi.AllMailLabel == storeMBox.LabelID() { foundArchive := false