From 9b4177b6ea38de6e86b0fe723834b6ef36af15b3 Mon Sep 17 00:00:00 2001
From: Craig Blaszczyk
Date: Wed, 7 Jan 2015 11:41:06 +0000
Subject: switch to using format strings in error messages; raise NotFound when
 pagination fails to provide a more useful error message

---
 rest_framework/generics.py | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

(limited to 'rest_framework/generics.py')

diff --git a/rest_framework/generics.py b/rest_framework/generics.py
index 680992d7..fe92355d 100644
--- a/rest_framework/generics.py
+++ b/rest_framework/generics.py
@@ -10,6 +10,7 @@ from django.shortcuts import get_object_or_404 as _get_object_or_404
 from django.utils import six
 from django.utils.translation import ugettext as _
 from rest_framework import views, mixins
+from rest_framework.exceptions import NotFound
 from rest_framework.settings import api_settings
 
 
@@ -119,15 +120,16 @@ class GenericAPIView(views.APIView):
             if page == 'last':
                 page_number = paginator.num_pages
             else:
-                raise Http404(_("Choose a valid page number. Page numbers must be a whole number, or must be the string 'last'."))
+                raise NotFound(_("Choose a valid page number. Page numbers must be a whole number, or must be the string 'last'."))
+
+        page_number = -1
         try:
             page = paginator.page(page_number)
         except InvalidPage as exc:
-            error_format = _('Invalid page (%(page_number)s): %(message)s')
-            raise Http404(error_format % {
-                'page_number': page_number,
-                'message': six.text_type(exc)
-            })
+            error_format = _('Invalid page ({page_number}): {message}')
+            raise NotFound(error_format.format(
+                page_number=page_number, message=six.text_type(exc)
+            ))
 
         return page
 
-- 
cgit v1.2.3