Merge remote branch 'refs/remotes/elrond/idea/new_get_page_url'

This commit is contained in:
Christopher Allan Webber 2011-05-22 16:55:01 -05:00
commit 57c6da19cc
2 changed files with 7 additions and 12 deletions

View File

@ -21,15 +21,13 @@
<div class=pagination>
{% if pagination.has_prev %}
<a href={{ pagination.get_page_url(request.path_info,
pagination.page-1, request.GET) }}>&laquo; Prev</>
<a href={{ pagination.get_page_url(request, pagination.page-1) }}>&laquo; Prev</>
{% endif %}
{%- for page in pagination.iter_pages() %}
{% if page %}
{% if page != pagination.page %}
<a href={{ pagination.get_page_url(request.path_info,
page, request.GET) }}>{{ page }}</a>
<a href={{ pagination.get_page_url(request, page) }}>{{ page }}</a>
{% else %}
<strong>{{ page }}</strong>
{% endif %}
@ -39,8 +37,7 @@
{%- endfor %}
{% if pagination.has_next %}
<a href={{ pagination.get_page_url(request.path_info,
pagination.page+1, request.GET) }}>Next &raquo;</a>
<a href={{ pagination.get_page_url(request, pagination.page+1) }}>Next &raquo;</a>
{% endif %}
</div>
{% endif %}

View File

@ -29,8 +29,6 @@ from mediagoblin import globals as mgoblin_globals
import urllib
from math import ceil
import copy
import decorators
from webob import exc
TESTS_ENABLED = False
def _activate_testing():
@ -353,13 +351,13 @@ class Pagination(object):
yield num
last = num
def get_page_url(self, path_info, page_no, get_params=None):
def get_page_url(self, request, page_no):
"""
Get a new page based of the path_info, the new page number,
and existing get parameters.
Get a new page url based of the request, and the new page number.
"""
path_info = request.path_info
get_params = request.GET
new_get_params = copy.copy(get_params or {})
new_get_params['page'] = page_no
return "%s?%s" % (
path_info, urllib.urlencode(new_get_params))