Skip to content

D
django-rest-framework
Commit ae059c78 by Tom Drummond
Options

Now throws a 404 if page isn't an int

parent 1eb2dfcc
Showing with 1 additions and 1 deletions
djangorestframework/mixins.py
...@@ -713,7 +713,7 @@ class PaginatorMixin(object): ...@@ -713,7 +713,7 @@ class PaginatorMixin(object):
try: try:
page_num = int(self.request.GET.get('page', '1')) page_num = int(self.request.GET.get('page', '1'))
except ValueError: except ValueError:
page_num = 1 raise ErrorResponse(status.HTTP_404_NOT_FOUND, {'detail': 'That page contains no results'})
if page_num not in paginator.page_range: if page_num not in paginator.page_range:
raise ErrorResponse(status.HTTP_404_NOT_FOUND, {'detail': 'That page contains no results'}) raise ErrorResponse(status.HTTP_404_NOT_FOUND, {'detail': 'That page contains no results'})
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment