Commit 919a208345280d19aa0922f76501b862e2c4a18a
1 parent
aa8a55a4
Exists in
master
and in
39 other branches
Adding verification on paginator
Showing
1 changed file
with
5 additions
and
2 deletions
Show diff stats
src/super_archives/views.py
@@ -11,7 +11,7 @@ from django.conf import settings | @@ -11,7 +11,7 @@ from django.conf import settings | ||
11 | from django.contrib import messages | 11 | from django.contrib import messages |
12 | from django.db import IntegrityError | 12 | from django.db import IntegrityError |
13 | from django.views.generic import View | 13 | from django.views.generic import View |
14 | -from django.core.paginator import Paginator | 14 | +from django.core.paginator import Paginator, EmptyPage |
15 | from django.utils.translation import ugettext as _ | 15 | from django.utils.translation import ugettext as _ |
16 | from django.core.exceptions import ObjectDoesNotExist | 16 | from django.core.exceptions import ObjectDoesNotExist |
17 | from django.utils.decorators import method_decorator | 17 | from django.utils.decorators import method_decorator |
@@ -128,7 +128,10 @@ def list_messages(request): | @@ -128,7 +128,10 @@ def list_messages(request): | ||
128 | page = int(request.GET.get('p', '1')) | 128 | page = int(request.GET.get('p', '1')) |
129 | except ValueError: | 129 | except ValueError: |
130 | page = 1 | 130 | page = 1 |
131 | - threads = paginator.page(page) | 131 | + try: |
132 | + threads = paginator.page(page) | ||
133 | + except EmptyPage: | ||
134 | + raise http.Http404 | ||
132 | 135 | ||
133 | lists = MailingList.objects.all() | 136 | lists = MailingList.objects.all() |
134 | 137 |