Ticket #4053: paginator-check_int.diff

core/paginator.py

@@ -1 +1 @@
 class InvalidPage(Exception):
     pass
 
+class InvalidParameter(Exception):
+    pass
+
 class ObjectPaginator(object):
     """
     This class makes pagination easy. Feed it a QuerySet or list, plus the number
@@ -40 +43 @@
 
     def has_next_page(self, page_number):
         "Does page $page_number have a 'next' page?"
-        return page_number < self.pages - 1
+        try:
+            return int(page_number) < self.pages - 1
+        except ValueError:
+            raise InvalidParameter
 
     def has_previous_page(self, page_number):
-        return page_number > 0
+        try:
+            return int(page_number) > 0
+        except:
+            raise InvalidParameter
 
     def first_on_page(self, page_number):
         """