diff options
| author | Ruben Pollan <meskio@sindominio.net> | 2014-08-22 11:26:57 -0500 |
|---|---|---|
| committer | Ruben Pollan <meskio@sindominio.net> | 2014-08-22 11:26:57 -0500 |
| commit | 726ee20f8e19fcc3bfe1d960e2b75cae53423533 (patch) | |
| tree | 5e22446e9dfd8604859703f6d72b6c34fc4862a7 | |
| parent | 597a0aa1ccd125ce83727128c0cddccc9ef29796 (diff) | |
| parent | 87665f1590f00e602c01fd52a14fd7a1b01a1010 (diff) | |
Merge branch 'bwgpro/fix_message_iterator' into develop
| -rw-r--r-- | changes/bug_messages-iterator-needs-collection | 2 | ||||
| -rw-r--r-- | src/leap/mail/imap/messages.py | 10 |
2 files changed, 7 insertions, 5 deletions
diff --git a/changes/bug_messages-iterator-needs-collection b/changes/bug_messages-iterator-needs-collection new file mode 100644 index 0000000..50c67d0 --- /dev/null +++ b/changes/bug_messages-iterator-needs-collection @@ -0,0 +1,2 @@ + o MessageCollection iterator now creates the LeapMessage with the collection + reference, so setFlags will work properly diff --git a/src/leap/mail/imap/messages.py b/src/leap/mail/imap/messages.py index b0b2f95..4e8e1a3 100644 --- a/src/leap/mail/imap/messages.py +++ b/src/leap/mail/imap/messages.py @@ -750,7 +750,7 @@ class MessageCollection(WithMsgFields, IndexedDB, MBoxParser): :return: a dict with the template :rtype: dict """ - if not _type in self.templates.keys(): + if _type not in self.templates.keys(): raise TypeError("Improper type passed to _get_empty_doc") return copy.deepcopy(self.templates[_type]) @@ -938,10 +938,10 @@ class MessageCollection(WithMsgFields, IndexedDB, MBoxParser): # We can say the observer that we're done at this point, but # before that we should make sure it has no serious consequences # if we're issued, for instance, a fetch command right after... - #self.reactor.callFromThread(observer.callback, uid) + # self.reactor.callFromThread(observer.callback, uid) # if we did the notify, we need to invalidate the deferred # so not to try to fire it twice. - #observer = None + # observer = None fd = self._populate_flags(flags, uid, chash, size, multi) hd = self._populate_headr(msg, chash, subject, date) @@ -1337,7 +1337,7 @@ class MessageCollection(WithMsgFields, IndexedDB, MBoxParser): :returns: a list of LeapMessages :rtype: list """ - return [LeapMessage(self._soledad, docid, self.mbox) + return [LeapMessage(self._soledad, docid, self.mbox, collection=self) for docid in self.unseen_iter()] # recent messages @@ -1370,7 +1370,7 @@ class MessageCollection(WithMsgFields, IndexedDB, MBoxParser): :returns: iterator of dicts with content for all messages. :rtype: iterable """ - return (LeapMessage(self._soledad, docuid, self.mbox) + return (LeapMessage(self._soledad, docuid, self.mbox, collection=self) for docuid in self.all_uid_iter()) def __repr__(self): |