summaryrefslogtreecommitdiff
path: root/src/mailman/rest
diff options
context:
space:
mode:
authorAurélien Bompard2016-10-26 10:30:58 +0200
committerBarry Warsaw2016-10-26 08:39:07 -0400
commit88176f08e67a8f6ad706f371167167e75d4a3e04 (patch)
tree29e4a10e478d6acc8b5342aa5976613b762c4bbc /src/mailman/rest
parente75d653f700dcf9a0a31550384d6511c47d024c1 (diff)
downloadmailman-88176f08e67a8f6ad706f371167167e75d4a3e04.tar.gz
mailman-88176f08e67a8f6ad706f371167167e75d4a3e04.tar.zst
mailman-88176f08e67a8f6ad706f371167167e75d4a3e04.zip
Implement the review suggestions
See !161
Diffstat (limited to 'src/mailman/rest')
-rw-r--r--src/mailman/rest/docs/lists.rst6
-rw-r--r--src/mailman/rest/lists.py4
2 files changed, 5 insertions, 5 deletions
diff --git a/src/mailman/rest/docs/lists.rst b/src/mailman/rest/docs/lists.rst
index eb8314704..aa9c01362 100644
--- a/src/mailman/rest/docs/lists.rst
+++ b/src/mailman/rest/docs/lists.rst
@@ -49,14 +49,14 @@ You can also query for lists from a particular domain.
start: 0
total_size: 1
-Advertised lists can be filtered using the ``only_advertised`` query parameter.
+Advertised lists can be filtered using the ``advertised`` query parameter.
::
>>> mlist = create_list('bird@example.com')
>>> mlist.advertised = False
>>> transaction.commit()
- >>> dump_json('http://localhost:9001/3.0/lists?only_advertised=true')
+ >>> dump_json('http://localhost:9001/3.0/lists?advertised=true')
entry 0:
...
list_id: ant.example.com
@@ -67,7 +67,7 @@ Advertised lists can be filtered using the ``only_advertised`` query parameter.
The same applies to lists from a particular domain.
- >>> dump_json('http://localhost:9001/3.0/domains/example.com/lists?only_advertised=true')
+ >>> dump_json('http://localhost:9001/3.0/domains/example.com/lists?advertised=true')
entry 0:
...
list_id: ant.example.com
diff --git a/src/mailman/rest/lists.py b/src/mailman/rest/lists.py
index dc1d75203..8952f1156 100644
--- a/src/mailman/rest/lists.py
+++ b/src/mailman/rest/lists.py
@@ -111,8 +111,8 @@ class _ListBase(CollectionMixin):
def _filter_lists(self, request, **kw):
"""Filter a collection using query parameters."""
- only_advertised = request.get_param_as_bool('only_advertised')
- if only_advertised:
+ advertised = request.get_param_as_bool('advertised')
+ if advertised:
kw['advertised'] = True
return getUtility(IListManager).find(**kw)