From c8dfed5b5f4ccb87003119f14e189566219365bb Mon Sep 17 00:00:00 2001 From: Kali Kaneko Date: Thu, 18 Jun 2015 23:20:45 -0400 Subject: [bug] fixes for display attachments and move between folders - Add errback handling to catch properly errors that were not allowing the complete insertion of the parts for a given message. Fixes blank attachments and moving of messages to different folders. - Force overwritting of mdoc when it is a copy. This was avoiding a message to be copied back to a folder from where it already had been copied to another (since the mdoc was already existing there, with the same doc_id, which was forbidding the creation of the new one). This case also needs special care in the indexer, since we have to delete the old hash entry first. Closes: #7178, #7158 --- src/leap/mail/imap/mailbox.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) (limited to 'src/leap/mail/imap') diff --git a/src/leap/mail/imap/mailbox.py b/src/leap/mail/imap/mailbox.py index 1412344..c4821ff 100644 --- a/src/leap/mail/imap/mailbox.py +++ b/src/leap/mail/imap/mailbox.py @@ -846,8 +846,9 @@ class IMAPMailbox(object): #deferLater(self.reactor, 0, self._do_copy, message, d) #return d - return self.collection.copy_msg(message.message, - self.collection.mbox_uuid) + d = self.collection.copy_msg(message.message, + self.collection.mbox_uuid) + return d # convenience fun -- cgit v1.2.3