diff options
author | Patrick Maia <pmaia@thoughtworks.com> | 2015-02-20 11:14:58 -0300 |
---|---|---|
committer | Patrick Maia <pmaia@thoughtworks.com> | 2015-02-20 11:17:01 -0300 |
commit | 2bd46a719b947b0839f47fd9e4bf5517f396fd29 (patch) | |
tree | 3f9acb994c118163fea8eeaafedf030a20022b15 /service/pixelated/adapter/search/contacts.py | |
parent | c5f29f136f293a51d2fd42fcfd8fc18a39f09cfa (diff) |
Issue #174 - rejects bounced addresses on contacts suggestions
Diffstat (limited to 'service/pixelated/adapter/search/contacts.py')
-rw-r--r-- | service/pixelated/adapter/search/contacts.py | 13 |
1 files changed, 10 insertions, 3 deletions
diff --git a/service/pixelated/adapter/search/contacts.py b/service/pixelated/adapter/search/contacts.py index afef6ad0..0dfeb15b 100644 --- a/service/pixelated/adapter/search/contacts.py +++ b/service/pixelated/adapter/search/contacts.py @@ -31,14 +31,21 @@ def address_duplication_filter(contacts): return contacts_by_mail.values() +def bounced_addresses_filter(searcher, contacts): + query = QueryParser('bounced', searcher.schema).parse('*') + bounced_addresses = searcher.search(query, + limit=None, + groupedby=sorting.FieldFacet('bounced', + allow_overlap=True)).groups() + return set(contacts) - set(flatten([bounced_addresses])) + + def extract_mail_address(text): return parseaddr(text)[1] def contacts_suggestions(query, searcher): - if query: - return address_duplication_filter(search_addresses(searcher, query)) - return [] + return address_duplication_filter(bounced_addresses_filter(searcher, search_addresses(searcher, query))) if query else [] def search_addresses(searcher, query): |