diff options
| author | Patrick Maia and Victor Shyba <pixelated-team+pmaia+vshyba@thoughtworks.com> | 2014-09-06 23:49:53 +0000 |
|---|---|---|
| committer | Patrick Maia <pmaia@thoughtworks.com> | 2014-09-06 23:49:53 +0000 |
| commit | 2628ad607bec6f147a4a73786976585c74b483c7 (patch) | |
| tree | 7a7a6d28d5895cd6cd056a9cff441a48873c5a95 /service/pixelated/adapter | |
| parent | 0b7fd19efba22ebf4270a3be0677e3f83d4a8aaf (diff) | |
#51 - does not use Tag objects when strings are enough
Diffstat (limited to 'service/pixelated/adapter')
| -rw-r--r-- | service/pixelated/adapter/pixelated_mailbox.py | 3 | ||||
| -rw-r--r-- | service/pixelated/adapter/pixelated_mailboxes.py | 5 |
2 files changed, 2 insertions, 6 deletions
diff --git a/service/pixelated/adapter/pixelated_mailbox.py b/service/pixelated/adapter/pixelated_mailbox.py index ecd83d6e..739bc215 100644 --- a/service/pixelated/adapter/pixelated_mailbox.py +++ b/service/pixelated/adapter/pixelated_mailbox.py @@ -1,4 +1,3 @@ - # # Copyright (c) 2014 ThoughtWorks, Inc. # @@ -43,7 +42,7 @@ class PixelatedMailbox: return mails def mails_by_tags(self, tags): - if 'all' in map(str, tags): + if 'all' in tags: return self.mails() return [mail for mail in self.mails() if len(mail.tags.intersection(tags)) > 0] diff --git a/service/pixelated/adapter/pixelated_mailboxes.py b/service/pixelated/adapter/pixelated_mailboxes.py index 1125c137..652080ca 100644 --- a/service/pixelated/adapter/pixelated_mailboxes.py +++ b/service/pixelated/adapter/pixelated_mailboxes.py @@ -1,4 +1,3 @@ -from pixelated.adapter.tag import Tag from pixelated.adapter.pixelated_mailbox import PixelatedMailbox @@ -16,11 +15,9 @@ class PixelatedMailBoxes(): self.account.mailboxes] def mails_by_tag(self, query_tags): - tags = map(Tag, query_tags) - mails = [] for mailbox in self.mailboxes: - mails.extend(mailbox.mails_by_tags(tags)) + mails.extend(mailbox.mails_by_tags(query_tags)) return mails |