Set previous and next page links in the thread list view instead of template

boards/templates/boards/posting_general.html

@@ -14,23 +14,11 b''
         <title>{{ site_name }}</title>
     {% endif %}
 
-    {% if current_page.has_previous %}
-        <link rel="prev" href="
-            {% if tag %}
-                {% url "tag" tag_name=tag.name page=current_page.previous_page_number %}
-            {% else %}
-                {% url "index" page=current_page.previous_page_number %}
-            {% endif %}
-        " />
+    {% if prev_page_link %}
+        <link rel="prev" href="{{ prev_page_link }}" />
     {% endif %}
-    {% if current_page.has_next %}
-        <link rel="next" href="
-            {% if tag %}
-                {% url "tag" tag_name=tag.name page=current_page.next_page_number %}
-            {% else %}
-                {% url "index" page=current_page.next_page_number %}
-            {% endif %}
-        " />
+    {% if next_page_link %}
+        <link rel="next" href="{{ next_page_link }}" />
     {% endif %}
 
 {% endblock %}
@@ -69,15 +57,9 b''
 {% endif %}
 
     {% if threads %}
-        {% if current_page.has_previous %}
+        {% if prev_page_link %}
             <div class="page_link">
-                <a href="
-            {% if tag %}
-                {% url "tag" tag_name=tag.name page=current_page.previous_page_number %}
-            {% else %}
-                {% url "index" page=current_page.previous_page_number %}
-            {% endif %}
-                ">{% trans "Previous page" %}</a>
+                <a href="{{ prev_page_link }}">{% trans "Previous page" %}</a>
             </div>
         {% endif %}
 
@@ -107,15 +89,9 b''
             </div>
         {% endfor %}
 
-        {% if current_page.has_next %}
+        {% if next_page_link %}
             <div class="page_link">
-                <a href="
-            {% if tag %}
-                {% url "tag" tag_name=tag.name page=current_page.next_page_number %}
-            {% else %}
-                {% url "index" page=current_page.next_page_number %}
-            {% endif %}
-                ">{% trans "Next page" %}</a>
+                <a href="{{ next_page_link }}">{% trans "Next page" %}</a>
             </div>
         {% endif %}
     {% else %}

boards/views/all_threads.py

@@ -1,3 +1,4 b''
+from django.core.urlresolvers import reverse
 from django.core.files import File
 from django.core.files.temp import NamedTemporaryFile
 from django.core.paginator import EmptyPage
@@ -28,6 +29,9 b" PARAMETER_CURRENT_PAGE = 'current_page'"
 PARAMETER_PAGINATOR = 'paginator'
 PARAMETER_THREADS = 'threads'
 
+PARAMETER_PREV_LINK = 'prev_page_link'
+PARAMETER_NEXT_LINK = 'next_page_link'
+
 TEMPLATE = 'boards/posting_general.html'
 DEFAULT_PAGE = 1
 
@@ -57,7 +61,7 b' class AllThreadsView(PostMixin, BaseBoar'
         params[PARAMETER_THREADS] = threads
         params[CONTEXT_FORM] = form
 
-        self._get_page_context(paginator, params, page)
+        self.get_page_context(paginator, params, page)
 
         return render(request, TEMPLATE, params)
 
@@ -74,13 +78,29 b' class AllThreadsView(PostMixin, BaseBoar'
 
         return self.get(request, page, form)
 
-    def _get_page_context(self, paginator, params, page):
+    def get_page_context(self, paginator, params, page):
         """
         Get pagination context variables
         """
 
         params[PARAMETER_PAGINATOR] = paginator
-        params[PARAMETER_CURRENT_PAGE] = paginator.page(int(page))
+        current_page = paginator.page(int(page))
+        params[PARAMETER_CURRENT_PAGE] = current_page
+        if current_page.has_previous():
+            params[PARAMETER_PREV_LINK] = self.get_previous_page_link(
+                    current_page)
+        if current_page.has_next():
+            params[PARAMETER_NEXT_LINK] = self.get_next_page_link(current_page)
+
+    def get_previous_page_link(self, current_page):
+        return reverse('index', kwargs={
+            'page': current_page.previous_page_number(),
+        })
+
+    def get_next_page_link(self, current_page):
+        return reverse('index', kwargs={
+            'page': current_page.next_page_number(),
+        })
 
     @staticmethod
     def parse_tags_string(tag_strings):

boards/views/tag_threads.py

@@ -1,4 +1,5 b''
 from django.shortcuts import get_object_or_404, redirect
+from django.core.urlresolvers import reverse
 
 from boards.abstracts.settingsmanager import get_settings_manager, \
         SETTING_FAVORITE_TAGS, SETTING_HIDDEN_TAGS
@@ -48,6 +49,18 b' class TagView(AllThreadsView, Dispatcher'
 
         return params
 
+    def get_previous_page_link(self, current_page):
+        return reverse('tag', kwargs={
+            'tag_name': self.tag_name,
+            'page': current_page.previous_page_number(),
+        })
+
+    def get_next_page_link(self, current_page):
+        return reverse('tag', kwargs={
+            'tag_name': self.tag_name,
+            'page': current_page.next_page_number(),
+        })
+
     def get(self, request, tag_name, page=DEFAULT_PAGE, form=None):
         self.tag_name = tag_name