Reflinks to OPs are bold now. Refactored reflinks to build using the same code. Refactored autoescaping

boards/migrations/0025_auto_20150825_2049.py

@@ -0,0 +1,21 b''
+# -*- coding: utf-8 -*-
+from __future__ import unicode_literals
+
+from django.db import migrations
+from boards.models import Post
+
+
+class Migration(migrations.Migration):
+
+    def refuild_refmap(apps, schema_editor):
+        for post in Post.objects.all():
+            post.build_refmap()
+            post.save(update_fields=['refmap'])
+
+    dependencies = [
+        ('boards', '0024_post_tripcode'),
+    ]
+
+    operations = [
+        migrations.RunPython(refuild_refmap),
+    ]

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_absolute_url(), post_id)
+            result = post.get_link_view()
         except ObjectDoesNotExist:
             pass
 

boards/models/post/__init__.py

@@ -61,8 +61,6 b' POST_VIEW_PARAMS = ('
     'mode_tree',
 )
 
-REFMAP_STR = '<a href="{}">&gt;&gt;{}</a>'
-
 IMAGE_TYPES = (
     'jpeg',
     'jpg',
@@ -230,7 +228,7 b' class Post(models.Model, Viewable):'
         the server from recalculating the map on every post show.
         """
 
-        post_urls = [REFMAP_STR.format(refpost.get_absolute_url(), refpost.id)
+        post_urls = [refpost.get_link_view()
                      for refpost in self.referenced_posts.all()]
 
         self.refmap = ', '.join(post_urls)
@@ -444,3 +442,15 b' class Post(models.Model, Viewable):'
     def get_tripcode(self):
         if self.tripcode:
             return Tripcode(self.tripcode)
+
+    def get_link_view(self):
+        """
+        Gets view of a reflink to the post.
+        """
+
+        result = '<a href="{}">&gt;&gt;{}</a>'.format(self.get_absolute_url(),
+                self.id)
+        if self.is_opening():
+            result = '<b>{}</b>'.format(result)
+
+        return result

boards/templates/boards/all_threads.html

@@ -64,25 +64,19 b''
                         <button name="method" value="hide" class="not_fav">H</button>
                     {% endif %}
                 </form>
-                {% autoescape off %}
-                    {{ tag.get_view }}
-                {% endautoescape %}
+                {{ tag.get_view|safe }}
                 {% if moderator %}
                 <span class="moderator_info">| <a href="{% url 'admin:boards_tag_change' tag.id %}">{% trans 'Edit tag' %}</a></span>
                 {% endif %}
             </h2>
             {% if tag.get_description %}
-                {% autoescape off %}
-                    <p>{{ tag.get_description }}</p>
-                {% endautoescape %}
+                <p>{{ tag.get_description|safe }}</p>
             {% endif %}
             <p>{% blocktrans with active_thread_count=tag.get_active_thread_count thread_count=tag.get_thread_count post_count=tag.get_post_count %}This tag has {{ thread_count }} threads ({{ active_thread_count}} active) and {{ post_count }} posts.{% endblocktrans %}</p>
             {% if related_tags %}
                 <p>{% trans 'Related tags:' %}
                 {% for rel_tag in related_tags %}
-                    {% autoescape off %}
-                    {{ rel_tag.get_view }}{% if not forloop.last %}, {% else %}.{% endif %}
-                    {% endautoescape %}
+                    {{ rel_tag.get_view|safe }}{% if not forloop.last %}, {% else %}.{% endif %}
                 {% endfor %}
                 </p>
             {% endif %}

boards/templates/boards/post.html

@@ -36,7 +36,7 b''
         {% else %}
             {% if need_op_data %}
                 {% with thread.get_opening_post as op %}
-                    {% trans " in " %}<a href="{{ op.get_absolute_url }}">&gt;&gt;{{ op.id }}</a> <span class="title">{{ op.get_title|striptags|truncatewords:5 }}</span>
+                    {% trans " in " %}{{ op.get_link_view|safe }} <span class="title">{{ op.get_title|striptags|truncatewords:5 }}</span>
                 {% endwith %}
             {% endif %}
         {% endif %}
@@ -59,16 +59,12 b''
     {%  endcomment %}
     {% if post.images.exists %}
         {% with post.images.first as image %}
-            {% autoescape off %}
-                {{ image.get_view }}
-            {% endautoescape %}
+            {{ image.get_view|safe }}
         {% endwith %}
     {% endif %}
     {% if post.attachments.exists %}
         {% with post.attachments.first as file %}
-            {% autoescape off %}
-                {{ file.get_view }}
-            {% endautoescape %}
+            {{ file.get_view|safe }}
         {% endwith %}
     {% endif %}
     {% comment %}
@@ -92,9 +88,7 b''
             </div>
         {% else %}
             <div class="refmap">
-                {% autoescape off %}
-                    {% trans "Replies" %}: {{ post.refmap }}
-                {% endautoescape %}
+                {% trans "Replies" %}: {{ post.refmap|safe }}
             </div>
         {% endif %}
     {% endif %}
@@ -108,9 +102,7 b''
                 {{ thread.get_images_count }} {% trans 'images' %}.
             {% endif %}
             <span class="tags">
-                {% autoescape off %}
-                    {{ thread.get_tag_url_list }}
-                {% endautoescape %}
+                {{ thread.get_tag_url_list|safe }}
             </span>
         </div>
     {% endif %}

boards/templates/boards/settings.html

@@ -18,9 +18,7 b''
         {% if hidden_tags %}
             <p>{% trans 'Hidden tags:' %}
                 {% for tag in hidden_tags %}
-                    {% autoescape off %}
-                        {{ tag.get_view }}
-                    {% endautoescape %}
+                    {{ tag.get_view|safe }}
                 {% endfor %}
             </p>
         {% else %}

boards/templates/boards/tag.html

@@ -1,5 +1,3 b''
 <div class="post">
-    {% autoescape off %}
-        {{ tag.get_view }}
-    {% endautoescape %}
+    {{ tag.get_view|safe }}
 </div>