Skip to content

Commit

Permalink
Merge remote branch 'carljm/hashtag'
Browse files Browse the repository at this point in the history
Conflicts:
	pagination/templatetags/pagination_tags.py
  • Loading branch information
brosner committed Apr 16, 2010
2 parents cbdc3bd + 1d3f1e0 commit 947647e
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
4 changes: 2 additions & 2 deletions pagination/templates/pagination/pagination.html
Original file line number Diff line number Diff line change
Expand Up @@ -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">&lsaquo;&lsaquo; {% trans "previous" %}</a>
<a href="?page={{ page_obj.previous_page_number }}{{ getvars }}{{ hashtag }}" class="prev">&lsaquo;&lsaquo; {% trans "previous" %}</a>
{% else %}
<span class="disabled prev">&lsaquo;&lsaquo; {% trans "previous" %}</span>
{% endif %}
Expand All @@ -18,7 +18,7 @@
{% endif %}
{% endfor %}
{% if page_obj.has_next %}
<a href="?page={{ page_obj.next_page_number }}{{ getvars }}" class="next">{% trans "next" %} &rsaquo;&rsaquo;</a>
<a href="?page={{ page_obj.next_page_number }}{{ getvars }}{{ hashtag }}" class="next">{% trans "next" %} &rsaquo;&rsaquo;</a>
{% else %}
<span class="disabled next">{% trans "next" %} &rsaquo;&rsaquo;</span>
{% endif %}
Expand Down
3 changes: 2 additions & 1 deletion pagination/templatetags/pagination_tags.py
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,8 @@ def render(self, context):
context['page_obj'] = page_obj
return u''

def paginate(context, window=DEFAULT_WINDOW, hashtag=None):

def paginate(context, window=DEFAULT_WINDOW, hashtag=''):
"""
Renders the ``pagination/pagination.html`` template, resulting in a
Digg-like display of the available pages, given the current page. If there
Expand Down

1 comment on commit 947647e

@ericflo
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Man, I was trying to merge this and it took me a good half hour to figure out that you had already done so brosner :) Thanks for doing so, though. Fixes a pretty major bug in the latest release.

Please sign in to comment.