Author: gwilson
Date: 2007-11-19 00:10:23 -0600 (Mon, 19 Nov 2007)
New Revision: 6702
Modified:
django/trunk/django/core/paginator.py
django/trunk/tests/modeltests/pagination/models.py
Log:
Fixed #5932 -- Use `self.pages` and not `self._pages` in `_get_page_range` so
that an exception is not raised if `self.page_range` is accessed before
`self.pages`.
Modified: django/trunk/django/core/paginator.py
===================================================================
--- django/trunk/django/core/paginator.py 2007-11-19 05:59:58 UTC (rev
6701)
+++ django/trunk/django/core/paginator.py 2007-11-19 06:10:23 UTC (rev
6702)
@@ -91,7 +91,7 @@
a template for loop.
"""
if self._page_range is None:
- self._page_range = range(1, self._pages + 1)
+ self._page_range = range(1, self.pages + 1)
return self._page_range
hits = property(_get_hits)
Modified: django/trunk/tests/modeltests/pagination/models.py
===================================================================
--- django/trunk/tests/modeltests/pagination/models.py 2007-11-19 05:59:58 UTC
(rev 6701)
+++ django/trunk/tests/modeltests/pagination/models.py 2007-11-19 06:10:23 UTC
(rev 6702)
@@ -78,7 +78,8 @@
>>> paginator.pages
2
-# The paginator can provide a list of all available pages
+# The paginator can provide a list of all available pages.
+>>> paginator = ObjectPaginator(Article.objects.all(), 10)
>>> paginator.page_range
[1, 2]
"""}
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en
-~----------~----~----~----~------~----~------~--~---
