From 5a08c47353cefccaa021e0f488ab76f3806dd49c Mon Sep 17 00:00:00 2001 From: Barry Warsaw Date: Sat, 18 Jul 2015 23:29:07 -0400 Subject: Clean up based on review. --- src/mailman/rest/helpers.py | 1 + src/mailman/rest/lists.py | 3 +-- src/mailman/rest/wsgiapp.py | 1 + 3 files changed, 3 insertions(+), 2 deletions(-) (limited to 'src') diff --git a/src/mailman/rest/helpers.py b/src/mailman/rest/helpers.py index 84aa3663b..0bba86521 100644 --- a/src/mailman/rest/helpers.py +++ b/src/mailman/rest/helpers.py @@ -54,6 +54,7 @@ def path_to(resource, api_version): system base URI. :type resource: string :param api_version: API version to report. + :type api_version: string :return: The full path to the resource. :rtype: bytes """ diff --git a/src/mailman/rest/lists.py b/src/mailman/rest/lists.py index 3409a3a00..a17946738 100644 --- a/src/mailman/rest/lists.py +++ b/src/mailman/rest/lists.py @@ -235,8 +235,7 @@ class MembersOfList(MemberCollection): def _get_collection(self, request): """See `CollectionMixin`.""" # Overrides _MemberBase._get_collection() because we only want to - # return the members from the requested roster. Don't call super() - # but be sure to set the request object. + # return the members from the requested roster. roster = self._mlist.get_roster(self._role) address_of_member = attrgetter('address.email') return list(sorted(roster.members, key=address_of_member)) diff --git a/src/mailman/rest/wsgiapp.py b/src/mailman/rest/wsgiapp.py index 035125864..94f654217 100644 --- a/src/mailman/rest/wsgiapp.py +++ b/src/mailman/rest/wsgiapp.py @@ -52,6 +52,7 @@ class AdminWebServiceWSGIRequestHandler(WSGIRequestHandler): class SetAPIVersion: """Falcon middleware object that sets the api_version on resources.""" + def process_resource(self, request, response, resource): # Set this attribute on the resource right before it is dispatched # too. This can be used by the resource to provide different -- cgit v1.2.3-70-g09d2