From 8f4c8a69d6f80b8dbad8518bbe4c3da1c492ec8a 2015-03-04 23:04:13
From: Bradley M. Kuhn <bkuhn@ebb.org>
Date: 2015-03-04 23:04:13
Subject: [PATCH] Rework blog's custom_index for new pagination.

The pagination support changed, and as was previously done a few commits
ago for news, I'm trying a similar solution for blogs.

In this case, I'm trying to use the existing custom_index() method we
have and adapt it to properly support pagination in the way we want.

I'm not completely sure this will work, but I think it's at least close.

---

diff --git a/www/conservancy/apps/blog/views.py b/www/conservancy/apps/blog/views.py
index 7a95819ac8024373f6e9934d8241a52808e3b056..829c0233ea07d476e3efea858d3ad5094f2454ba 100644
--- a/www/conservancy/apps/blog/views.py
+++ b/www/conservancy/apps/blog/views.py
@@ -64,6 +64,24 @@ def custom_index(request, queryset, *args, **kwargs):
     # return object_list(request, queryset, *args, **kwargs)
     kwargs['queryset'] = queryset
     kwargs['extra_context'] = extra_context
+
+    paginator = Paginator(queryset, paginate_by)
+
+    page = request.GET.get('page')
+
+    try:
+        blog_entries = paginator.page(page)
+    except PageNotAnInteger:
+        # If page is not an integer, deliver first page.
+        blog_entries = paginator.page(1)
+    except EmptyPage:
+        # If page is out of range (e.g. 9999), deliver last page of results.
+        blog_entires = paginator.page(paginator.num_pages)
+
+    return render_to_response('blog/entry_list.html',
+                              {"blog_entries": blog_entries, "date_list" : date_list,
+                               "authors" : authors, "tags" : tags })
+
     callable = BlogListView.as_view(**kwargs)
     return callable(request)
 
diff --git a/www/conservancy/templates/blog/entry_list.html b/www/conservancy/templates/blog/entry_list.html
index 29f1adb3c256ef9f2247f09d3ee1edab56a1bb8b..e7c21317ac1e7bd0fa86d332755258d678c893dc 100644
--- a/www/conservancy/templates/blog/entry_list.html
+++ b/www/conservancy/templates/blog/entry_list.html
@@ -12,19 +12,21 @@
 </p>
 {% endif %}
 
-{% for object in object_list %}
+{% for entry in blog_entries %}
     <div class="shaded">
-    <p class="date">{{ object.pub_date|date:"F j, Y" }} by {{ object.author.formal_name }}</p>
-    <h3><a href="{{ object.get_absolute_url }}">{{ object.headline|safe }}</a></h3>
-        {{ object.body|safe }}
-    <p class="date small">Posted by <strong>{{ object.author.formal_name}}</strong> on {{ object.pub_date|date:"F j, Y" }}</p>
-    {% if object.tags.all %}<p class="blog-tags small">Tags: {% for tag in object.tags.all %}<a href="{{ tag.get_absolute_url }}">{{ tag.label }}</a>{% if not forloop.last %}, {% endif %}{% endfor %}</p>{% endif %}
+    <p class="date">{{ entry.pub_date|date:"F j, Y" }} by {{ entry.author.formal_name }}</p>
+    <h3><a href="{{ entry.get_absolute_url }}">{{ entry.headline|safe }}</a></h3>
+        {{ entry.body|safe }}
+    <p class="date small">Posted by <strong>{{ entry.author.formal_name}}</strong> on {{ entry.pub_date|date:"F j, Y" }}</p>
+    {% if entry.tags.all %}<p class="blog-tags small">Tags: {% for tag in entry.tags.all %}<a href="{{ tag.get_absolute_url }}">{{ tag.label }}</a>{% if not forloop.last %}, {% endif %}{% endfor %}</p>{% endif %}
     </div>
 {% endfor %}
 
 <p>
-{% if has_next %}<a class="next_page_button" href="?page={{ next }}{% if query_string %}&amp;{{ query_string|escape }}{% endif %}">Next page (older) &raquo;</a>{% endif %}
-{% if has_previous %}<a href="?page={{ previous }}{% if query_string %}&amp;{{ query_string|escape }}{% endif %}">&laquo; Previous page (newer)</a>{% endif %}
+{% if blog_entries.has_next %}<a class="next_page_button" href="?page={{ blog_entries.next_page_number }}{% if query_string %}&amp;{{ query_string|escape }}{% endif %}">Next page (older) &raquo;</a>{% endif %}
+{% if blog_entries.has_previous %}<a href="?page={{ blog_entires.previous_page_number }}{% if query_string %}&amp;{{ query_string|escape }}{% endif %}">&laquo; Previous page (newer)</a>{% endif %}
+</p>
+<p>{% for pagenum in blog_entries.paginator.page_range %}{% ifequal pagenum blog_entries.number %}[{{ pagenum }}]{% else %}<a href="?page={{ pagenum }}">{{ pagenum }}</a>{% endifequal %} {% endfor %}
 </p>
 <div class="clear"></div>