From 58ee14bfb1c1c24d06fb0509e42e06e37a505479 Mon Sep 17 00:00:00 2001 From: Barry Warsaw Date: Mon, 11 Jan 2016 09:35:14 -0500 Subject: Use the more efficient find_member() API. --- src/mailman/rest/lists.py | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/src/mailman/rest/lists.py b/src/mailman/rest/lists.py index a4d2da56a..a2b9b4a9c 100644 --- a/src/mailman/rest/lists.py +++ b/src/mailman/rest/lists.py @@ -151,12 +151,11 @@ class AList(_ListBase): """Return a single member representation.""" if self._mlist is None: return NotFound(), [] - members = getUtility(ISubscriptionService).find_members( + member = getUtility(ISubscriptionService).find_member( email, self._mlist.list_id, role) - if len(members) == 0: + if member is None: return NotFound(), [] - assert len(members) == 1, 'Too many matches' - return AMember(request.context['api_version'], members[0].member_id) + return AMember(request.context['api_version'], member.member_id) @child(roster_matcher) def roster(self, request, segments, role): -- cgit v1.3.1