Use get_absolute_url instead of get_url for post, tag and thread

boards/mdx_neboard.py

@@ -130,7 +130,7 b' def render_reflink(tag_name, value, opti'
         try:
             post = boards.models.Post.objects.get(id=post_id)
 
-            result = '<a href="%s">&gt;&gt;%s</a>' % (post.get_url(), post_id)
+            result = '<a href="%s">&gt;&gt;%s</a>' % (post.get_absolute_url(), post_id)
         except ObjectDoesNotExist:
             pass
 

boards/models/post/__init__.py

@@ -199,7 +199,7 b' class Post(models.Model, Viewable):'
         the server from recalculating the map on every post show.
         """
 
-        post_urls = [REFMAP_STR.format(refpost.get_url(), refpost.id)
+        post_urls = [REFMAP_STR.format(refpost.get_absolute_url(), refpost.id)
                      for refpost in self.referenced_posts.all()]
 
         self.refmap = ', '.join(post_urls)
@@ -214,10 +214,6 b' class Post(models.Model, Viewable):'
 
         return self.get_thread().get_opening_post_id() == self.id
 
-    # TODO Remove this and use get_absolute_url method
-    def get_url(self):
-        return self.url
-
     def get_absolute_url(self):
         return self.url
 

boards/models/tag.py

@@ -56,13 +56,9 b' class Tag(models.Model, Viewable):'
     def get_thread_count(self) -> int:
         return self.get_threads().count()
 
-    # TODO Remove this and use get_absolute_url
-    def get_url(self):
+    def get_absolute_url(self):
         return reverse('tag', kwargs={'tag_name': self.name})
 
-    def get_absolute_url(self):
-        return self.get_url()
-
     def get_threads(self):
         return self.thread_set.order_by('-bump_time')
 

boards/templates/boards/post.html

@@ -5,7 +5,7 b''
 
 <div class="{{ css_class }}" id="{{ post.id }}" data-uid="{{ post.uid }}">
     <div class="post-info">
-        <a class="post_id" href="{{ post.get_url }}">({{ post.get_absolute_id }})</a>
+        <a class="post_id" href="{{ post.get_absolute_url }}">({{ post.get_absolute_id }})</a>
         <span class="title">{{ post.title }}</span>
         <span class="pub_time"><time datetime="{{ post.pub_time|date:'c' }}">{{ post.pub_time }}</time></span>
         {% comment %}

boards/templates/boards/posting_general.html

@@ -32,7 +32,7 b''
         <div class="post">
             <div class="title">{{ banner.title }}</div>
             <div>{{ banner.text }}</div>
-            <div>{% trans 'Related message' %}: <a href="{{ banner.post.get_url }}">>>{{ banner.post.id }}</a></div>
+            <div>{% trans 'Related message' %}: <a href="{{ banner.post.get_absolute_url }}">>>{{ banner.post.id }}</a></div>
         </div>
     {% endfor %}
 

boards/templates/boards/thread_gallery.html

@@ -32,7 +32,7 b''
                         {{ image.width }}x{{ image.height }}
                         {% image_actions image.image.url request.get_host %}
                         <br />
-                        <a href="{{ post.get_url }}">>>{{ post.id }}</a>
+                        <a href="{{ post.get_absolute_url }}">>>{{ post.id }}</a>
                     </div>
                 {% endwith %}
                 </div>

boards/templatetags/board.py

@@ -26,7 +26,7 b' def post_url(*args, **kwargs):'
 
     post = get_object_or_404('Post', id=post_id)
 
-    return post.get_url()
+    return post.get_absolute_url()
 
 
 @register.simple_tag(name='image_actions')

boards/views/all_threads.py

@@ -158,7 +158,7 b' class AllThreadsView(PostMixin, BaseBoar'
         post.notify_clients()
 
         if html_response:
-            return redirect(post.get_url())
+            return redirect(post.get_absolute_url())
 
     def get_threads(self):
         """

boards/views/thread/thread.py

@@ -38,7 +38,8 b' class ThreadView(BaseBoardView, PostMixi'
 
         # If this is not OP, don't show it as it is
         if not opening_post.is_opening():
-            return redirect(opening_post.get_thread().get_opening_post().get_url())
+            return redirect(opening_post.get_thread().get_opening_post()
+                .get_absolute_url())
 
         if not form:
             form = PostForm(error_class=PlainErrorList)
@@ -111,7 +112,7 b' class ThreadView(BaseBoardView, PostMixi'
 
         if html_response:
             if opening_post:
-                return redirect(post.get_url())
+                return redirect(post.get_absolute_url())
         else:
             return post