From: Florian Schlachter <flosch@florian-schlachters-imac.local>
Date: Wed, 22 Apr 2009 13:25:23 +0000 (+0200)
Subject: Added support for multiple paginations on the same template.
X-Git-Tag: release-2.0~32^2~1^2^2
X-Git-Url: https://git.mdrn.pl/django-pagination.git/commitdiff_plain/92a1a772e0ea5c7d0e370634bae94eb155ed8195?page=2
Added support for multiple paginations on the same template.
---
diff --git a/pagination/middleware.py b/pagination/middleware.py
index f8a2a6f..676f909 100644
--- a/pagination/middleware.py
+++ b/pagination/middleware.py
@@ -1,10 +1,10 @@
-def get_page(self):
+def get_page(self, suffix):
"""
A function which will be monkeypatched onto the request to get the current
integer representing the current page.
"""
try:
- return int(self.REQUEST['page'])
+ return int(self.REQUEST['page%s' % suffix])
except (KeyError, ValueError, TypeError):
return 1
@@ -14,4 +14,4 @@ class PaginationMiddleware(object):
it exists in either **GET** or **POST** portions of the request.
"""
def process_request(self, request):
- request.__class__.page = property(get_page)
\ No newline at end of file
+ request.__class__.page = get_page
\ No newline at end of file
diff --git a/pagination/templates/pagination/pagination.html b/pagination/templates/pagination/pagination.html
index 37cef40..c2fb72d 100644
--- a/pagination/templates/pagination/pagination.html
+++ b/pagination/templates/pagination/pagination.html
@@ -2,7 +2,7 @@
{% load i18n %}
<div class="pagination">
{% if page_obj.has_previous %}
- <a href="?page={{ page_obj.previous_page_number }}{{ getvars }}" class="prev">‹‹ {% trans "previous" %}</a>
+ <a href="?page{{ page_suffix }}={{ page_obj.previous_page_number }}{{ getvars }}" class="prev">‹‹ {% trans "previous" %}</a>
{% else %}
<span class="disabled prev">‹‹ {% trans "previous" %}</span>
{% endif %}
@@ -11,14 +11,14 @@
{% ifequal page page_obj.number %}
<span class="current page">{{ page }}</span>
{% else %}
- <a href="?page={{ page }}{{ getvars }}" class="page">{{ page }}</a>
+ <a href="?page{{ page_suffix }}={{ page }}{{ getvars }}" class="page">{{ page }}</a>
{% endifequal %}
{% else %}
...
{% endif %}
{% endfor %}
{% if page_obj.has_next %}
- <a href="?page={{ page_obj.next_page_number }}{{ getvars }}" class="next">{% trans "next" %} ››</a>
+ <a href="?page{{ page_suffix }}={{ page_obj.next_page_number }}{{ getvars }}" class="next">{% trans "next" %} ››</a>
{% else %}
<span class="disabled next">{% trans "next" %} ››</span>
{% endif %}
diff --git a/pagination/templatetags/pagination_tags.py b/pagination/templatetags/pagination_tags.py
index 90d26de..c115720 100644
--- a/pagination/templatetags/pagination_tags.py
+++ b/pagination/templatetags/pagination_tags.py
@@ -4,6 +4,7 @@ except NameError:
from sets import Set as set
from django import template
+from django.template import TOKEN_BLOCK
from django.http import Http404
from django.core.paginator import Paginator, InvalidPage
from django.conf import settings
@@ -20,6 +21,12 @@ def do_autopaginate(parser, token):
"""
Splits the arguments to the autopaginate tag and formats them correctly.
"""
+
+ # Check whether there are any other autopaginations are later in this template
+ expr = lambda obj: (obj.token_type == TOKEN_BLOCK and \
+ len(obj.split_contents()) > 0 and obj.split_contents()[0] == "autopaginate")
+ multiple_paginations = len(filter(expr, parser.tokens)) > 0
+
split = token.split_contents()
as_index = None
context_var = None
@@ -36,10 +43,10 @@ def do_autopaginate(parser, token):
"context_var_name %%}" % split[0])
del split[as_index:as_index + 2]
if len(split) == 2:
- return AutoPaginateNode(split[1])
+ return AutoPaginateNode(split[1], multiple_paginations=multiple_paginations)
elif len(split) == 3:
return AutoPaginateNode(split[1], paginate_by=split[2],
- context_var=context_var)
+ context_var=context_var, multiple_paginations=multiple_paginations)
elif len(split) == 4:
try:
orphans = int(split[3])
@@ -47,7 +54,7 @@ def do_autopaginate(parser, token):
raise template.TemplateSyntaxError(u'Got %s, but expected integer.'
% split[3])
return AutoPaginateNode(split[1], paginate_by=split[2], orphans=orphans,
- context_var=context_var)
+ context_var=context_var, multiple_paginations=multiple_paginations)
else:
raise template.TemplateSyntaxError('%r tag takes one required ' +
'argument and one optional argument' % split[0])
@@ -69,7 +76,7 @@ class AutoPaginateNode(template.Node):
tag. If you choose not to use *{% paginate %}*, make sure to display the
list of available pages, or else the application may seem to be buggy.
"""
- def __init__(self, queryset_var, paginate_by=DEFAULT_PAGINATION,
+ def __init__(self, queryset_var, multiple_paginations, paginate_by=DEFAULT_PAGINATION,
orphans=DEFAULT_ORPHANS, context_var=None):
self.queryset_var = template.Variable(queryset_var)
if isinstance(paginate_by, int):
@@ -78,8 +85,14 @@ class AutoPaginateNode(template.Node):
self.paginate_by = template.Variable(paginate_by)
self.orphans = orphans
self.context_var = context_var
+ self.multiple_paginations = multiple_paginations
def render(self, context):
+ if self.multiple_paginations or context.has_key('paginator'):
+ page_suffix = '_%s' % self.queryset_var
+ else:
+ page_suffix = ''
+
key = self.queryset_var.var
value = self.queryset_var.resolve(context)
if isinstance(self.paginate_by, int):
@@ -88,7 +101,7 @@ class AutoPaginateNode(template.Node):
paginate_by = self.paginate_by.resolve(context)
paginator = Paginator(value, paginate_by, self.orphans)
try:
- page_obj = paginator.page(context['request'].page)
+ page_obj = paginator.page(context['request'].page(page_suffix))
except InvalidPage:
if INVALID_PAGE_RAISES_404:
raise Http404('Invalid page requested. If DEBUG were set to ' +
@@ -102,6 +115,7 @@ class AutoPaginateNode(template.Node):
context[key] = page_obj.object_list
context['paginator'] = paginator
context['page_obj'] = page_obj
+ context['page_suffix'] = page_suffix
return u''
def paginate(context, window=DEFAULT_WINDOW):
@@ -132,6 +146,7 @@ def paginate(context, window=DEFAULT_WINDOW):
try:
paginator = context['paginator']
page_obj = context['page_obj']
+ page_suffix = context.get('page_suffix', '')
page_range = paginator.page_range
# First and last are simply the first *n* pages and the last *n* pages,
# where *n* is the current window size.
@@ -203,11 +218,12 @@ def paginate(context, window=DEFAULT_WINDOW):
'page_obj': page_obj,
'paginator': paginator,
'is_paginated': paginator.count > paginator.per_page,
+ 'page_suffix': page_suffix,
}
if 'request' in context:
getvars = context['request'].GET.copy()
- if 'page' in getvars:
- del getvars['page']
+ if 'page%s' % page_suffix in getvars:
+ del getvars['page%s' % page_suffix]
if len(getvars.keys()) > 0:
to_return['getvars'] = "&%s" % getvars.urlencode()
else:
diff --git a/pagination/tests.py b/pagination/tests.py
index a55ef49..454e19d 100644
--- a/pagination/tests.py
+++ b/pagination/tests.py
@@ -28,11 +28,15 @@
>>> paginate({'paginator': p, 'page_obj': p.page(1)})['pages']
[1, 2, 3, 4, None, 7, 8, 9, 10]
+>>> p = Paginator(range(21), 2, 1)
+>>> paginate({'paginator': p, 'page_obj': p.page(1)})['pages']
+[1, 2, 3, 4, None, 7, 8, 9, 10]
+
>>> t = Template("{% load pagination_tags %}{% autopaginate var 2 %}{% paginate %}")
>>> from django.http import HttpRequest as DjangoHttpRequest
>>> class HttpRequest(DjangoHttpRequest):
-... page = 1
+... page = lambda self, suffix: 1
>>> t.render(Context({'var': range(21), 'request': HttpRequest()}))
u'\\n\\n<div class="pagination">...
@@ -45,11 +49,15 @@ u'\\n\\n<div class="pagination">...
u'\\n\\n<div class="pagination">...
>>> t = Template("{% load pagination_tags %}{% autopaginate var by %}{% paginate %}")
>>> t.render(Context({'var': range(21), 'by': 20, 'request': HttpRequest()}))
-u'\\n\\n<div class="pagination">...
+u'\\n\\n<div class="pagination">...<a href="?page=2"...
>>> t = Template("{% load pagination_tags %}{% autopaginate var by as foo %}{{ foo }}")
>>> t.render(Context({'var': range(21), 'by': 20, 'request': HttpRequest()}))
u'[0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19]'
>>>
+>>> t = Template("{% load pagination_tags %}{% autopaginate var2 by as foo2 %}{% paginate %}{% autopaginate var by as foo %}{% paginate %}")
+>>> t.render(Context({'var': range(21), 'var2': range(50, 121), 'by': 20, 'request': HttpRequest()}))
+u'\\n\\n<div class="pagination">...<a href="?page_var2=2"...<a href="?page_var=2"...
+>>>
# Testing InfinitePaginator