Use GET parameter to specify a page instead of different url reversers

boards/templates/boards/all_threads.html

@@ -153,9 +153,9 b''
                     {% endifequal %}
                     href="
                 {% if tag %}
-                    {% url "tag" tag_name=tag.name page=page %}
+                {% url "tag" tag_name=tag.name %}?page={{ page }}
                 {% else %}
-                    {% url "index" page=page %}
+                    {% url "index" %}?page={{ page }}
                 {% endif %}
                 ">{{ page }}</a>
                 {% if not forloop.last %},{% endif %}

boards/templates/boards/feed.html

@@ -61,9 +61,7 b''
                     {% ifequal page current_page.number %}
                         class="current_page"
                     {% endifequal %}
-                    href="
-                {% url "feed" page=page %}
-                ">{{ page }}</a>
+                    href="{% url "feed" %}?page={{ page }}">{{ page }}</a>
                 {% if not forloop.last %},{% endif %}
             {% endfor %}
         {% endwith %}

boards/urls.py

@@ -19,16 +19,10 b' js_info_dict = {'
 urlpatterns = patterns('',
     # /boards/
     url(r'^$', all_threads.AllThreadsView.as_view(), name='index'),
-    # /boards/page/
-    url(r'^page/(?P<page>\w+)/$', all_threads.AllThreadsView.as_view(),
-        name='index'),
 
     # /boards/tag/tag_name/
     url(r'^tag/(?P<tag_name>\w+)/$', tag_threads.TagView.as_view(),
         name='tag'),
-    # /boards/tag/tag_id/page/
-    url(r'^tag/(?P<tag_name>\w+)/page/(?P<page>\w+)/$',
-        tag_threads.TagView.as_view(), name='tag'),
 
     # /boards/thread/
     url(r'^thread/(?P<post_id>\d+)/$', views.thread.NormalThreadView.as_view(),
@@ -39,8 +33,6 b" urlpatterns = patterns('',"
         name='thread_tree'),
     # /feed/
     url(r'^feed/$', views.feed.FeedView.as_view(), name='feed'),
-    url(r'^feed/page/(?P<page>\w+)/$', views.feed.FeedView.as_view(),
-        name='feed'),
 
     url(r'^settings/$', settings.SettingsView.as_view(), name='settings'),
     url(r'^tags/(?P<query>\w+)?/?$', all_tags.AllTagsView.as_view(), name='tags'),

boards/views/all_threads.py

@@ -43,7 +43,9 b' class AllThreadsView(PostMixin, BaseBoar'
         self.settings_manager = None
         super(AllThreadsView, self).__init__()
 
-    def get(self, request, page=DEFAULT_PAGE, form: ThreadForm=None):
+    def get(self, request, form: ThreadForm=None):
+        page = request.GET.get('page', DEFAULT_PAGE)
+
         params = self.get_context_data(request=request)
 
         if not form:
@@ -67,7 +69,7 b' class AllThreadsView(PostMixin, BaseBoar'
 
         return render(request, TEMPLATE, params)
 
-    def post(self, request, page=DEFAULT_PAGE):
+    def post(self, request):
         form = ThreadForm(request.POST, request.FILES,
                           error_class=PlainErrorList)
         form.session = request.session
@@ -78,7 +80,7 b' class AllThreadsView(PostMixin, BaseBoar'
             # Ban user because he is suspected to be a bot
             self._ban_current_user(request)
 
-        return self.get(request, page, form)
+        return self.get(request, form)
 
     def get_page_context(self, paginator, params, page):
         """
@@ -95,14 +97,12 b' class AllThreadsView(PostMixin, BaseBoar'
             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(),
-        })
+        return reverse('index') + '?page=' \
+                + str(current_page.previous_page_number())
 
     def get_next_page_link(self, current_page):
-        return reverse('index', kwargs={
-            'page': current_page.next_page_number(),
-        })
+        return reverse('index') + '?page=' \
+                + str(current_page.next_page_number())
 
     @staticmethod
     def parse_tags_string(tag_strings):

boards/views/feed.py

@@ -28,7 +28,9 b' DEFAULT_PAGE = 1'
 
 class FeedView(PostMixin, BaseBoardView):
 
-    def get(self, request, page=DEFAULT_PAGE):
+    def get(self, request):
+        page = request.GET.get('page', DEFAULT_PAGE)
+
         params = self.get_context_data(request=request)
 
         settings_manager = get_settings_manager(request)
@@ -61,11 +63,9 b' class FeedView(PostMixin, BaseBoardView)'
             params[PARAMETER_NEXT_LINK] = self.get_next_page_link(current_page)
 
     def get_previous_page_link(self, current_page):
-        return reverse('feed', kwargs={
-            'page': current_page.previous_page_number(),
-        })
+        return reverse('feed') + '?page={}'.format(
+                current_page.previous_page_number())
 
     def get_next_page_link(self, current_page):
-        return reverse('feed', kwargs={
-            'page': current_page.next_page_number(),
-        })
+        return reverse('feed') + '?page={}'.format(
+                current_page.next_page_number())

boards/views/tag_threads.py

@@ -52,22 +52,20 b' class TagView(AllThreadsView, Dispatcher'
     def get_previous_page_link(self, current_page):
         return reverse('tag', kwargs={
             'tag_name': self.tag_name,
-            'page': current_page.previous_page_number(),
-        })
+        }) + '?page=' + str(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(),
-        })
+        }) + '?page=' + str(current_page.next_page_number())
 
-    def get(self, request, tag_name, page=DEFAULT_PAGE, form=None):
+    def get(self, request, tag_name, form=None):
         self.tag_name = tag_name
 
-        return super(TagView, self).get(request, page, form)
+        return super(TagView, self).get(request, form)
 
 
-    def post(self, request, tag_name, page=DEFAULT_PAGE):
+    def post(self, request, tag_name):
         self.tag_name = tag_name
 
         if 'method' in request.POST: