Some more optimization to the post html

boards/templates/boards/post.html

@@ -1,100 +1,105 b''
 {% load i18n %}
 {% load board %}
 {% load cache %}
 
 {% get_current_language as LANGUAGE_CODE %}
 
 {% cache 600 post post.id thread.last_edit_time truncated moderator LANGUAGE_CODE need_open_link %}
+
+    {% with is_opening=post.is_opening %}
     {% spaceless %}
         {% with thread=post.thread_new %}
             {% if thread.archived %}
                 <div class="post archive_post" id="{{ post.id }}">
             {% elif thread.can_bump %}
                 <div class="post" id="{{ post.id }}">
             {% else %}
                 <div class="post dead_post" id="{{ post.id }}">
             {% endif %}
 
         {% if post.image %}
             <div class="image">
                 <a
                         class="thumb"
                         href="{{ post.image.url }}"><img
                         src="{{ post.image.url_200x150 }}"
                         alt="{{ post.id }}"
                         width="{{ post.image_pre_width }}"
                         height="{{ post.image_pre_height }}"
                         data-width="{{ post.image_width }}"
                         data-height="{{ post.image_height }}"/>
                 </a>
             </div>
         {% endif %}
         <div class="message">
             <div class="post-info">
                 <span class="title">{{ post.title }}</span>
-                <a class="post_id" href="{% post_url post.id %}">
-                    ({{ post.id }}) </a>
+                {% cache 600 post_id post.id %}
+                    <a class="post_id" href="{% post_url post.id %}">
+                        ({{ post.id }}) </a>
+                {% endcache %}
                 [<span class="pub_time">{{ post.pub_time }}</span>]
                 {% if thread.archived %}
                     — [{{ thread.bump_time }}]
                 {% endif %}
                 {% if not truncated and not thread.archived %}
                     [<a href="#" onclick="javascript:addQuickReply('{{ post.id }}')
                         ; return false;">&gt;&gt;</a>]
                 {% endif %}
-                {% if post.is_opening and need_open_link %}
+                {% if is_opening and need_open_link %}
                     {% if post.thread_new.archived %}
                         [<a class="link" href="{% url 'thread' post.id %}">{% trans "Open" %}</a>]
                     {% else %}
                         [<a class="link" href="{% url 'thread' post.id %}#form">{% trans "Reply" %}</a>]
                     {% endif %}
                 {% endif %}
 
                 {% if moderator %}
                     <span class="moderator_info">
                         [<a href="{% url 'post_admin' post_id=post.id %}"
                             >{% trans 'Edit' %}</a>]
                         [<a href="{% url 'delete' post_id=post.id %}"
                             >{% trans 'Delete' %}</a>]
                         ({{ post.poster_ip }})
                         [<a href="{% url 'ban' post_id=post.id %}?next={{ request.path }}"
                             >{% trans 'Ban IP' %}</a>]
                     </span>
                 {% endif %}
             </div>
             {% autoescape off %}
                 {% if truncated %}
                     {{ post.text.rendered|truncatewords_html:50 }}
                 {% else %}
                     {{ post.text.rendered }}
                 {% endif %}
             {% endautoescape %}
             {% cache 600 post_replies post.id post.last_edit_time moderator LANGUAGE_CODE %}
-            {% if post.is_referenced %}
-                <div class="refmap">
-                    {% trans "Replies" %}:
-                    {% for ref_post in post.get_sorted_referenced_posts %}
-                        <a href="{% post_url ref_post.id %}">&gt;&gt;{{ ref_post.id }}</a
-                                >{% if not forloop.last %},{% endif %}
-                    {% endfor %}
-                </div>
-            {% endif %}
+                {% if post.is_referenced %}
+                    <div class="refmap">
+                        {% trans "Replies" %}:
+                        {% for ref_post in post.get_sorted_referenced_posts %}
+                            <a href="{% post_url ref_post.id %}">&gt;&gt;{{ ref_post.id }}</a
+                                    >{% if not forloop.last %},{% endif %}
+                        {% endfor %}
+                    </div>
+                {% endif %}
             {% endcache %}
         </div>
-        {% if post.is_opening and thread.tags.exists %}
+        {% if is_opening and thread.tags.exists %}
             <div class="metadata">
-                {% if post.is_opening and need_open_link %}
+                {% if is_opening and need_open_link %}
                     {{ thread.get_images_count }} {% trans 'images' %}.
                 {% endif %}
                 <span class="tags">
                     {% for tag in thread.get_tags %}
                         <a class="tag" href="{% url 'tag' tag.name %}">
                             #{{ tag.name }}</a>{% if not forloop.last %},{% endif %}
                     {% endfor %}
                 </span>
             </div>
         {% endif %}
         </div>
         {% endwith %}
     {% endspaceless %}
+    {% endwith %}
 {% endcache %}

boards/templatetags/board.py

@@ -1,76 +1,76 b''
 from django.core.urlresolvers import reverse
 from django.shortcuts import get_object_or_404
 from boards.models import Post
 from boards.views import thread, api
 from django import template
 
 register = template.Library()
 
 actions = [
     {
         'name': 'google',
         'link': 'http://google.com/searchbyimage?image_url=%s',
     },
     {
         'name': 'iqdb',
         'link': 'http://iqdb.org/?url=%s',
     },
 ]
 
 
 @register.simple_tag(name='post_url')
 def post_url(*args, **kwargs):
     post_id = args[0]
 
     post = get_object_or_404(Post, id=post_id)
 
     if not post.is_opening():
         link = reverse('thread', kwargs={
             'post_id': post.thread_new.get_opening_post().id}) + '#' + str(
             post_id)
     else:
         link = reverse('thread', kwargs={'post_id': post_id})
 
     return link
 
 
 @register.simple_tag(name='image_actions')
 def image_actions(*args, **kwargs):
     image_link = args[0]
     if len(args) > 1:
         image_link = 'http://' + args[1] + image_link # TODO https?
 
     result = ''
 
     for action in actions:
         result += '[<a href="' + action['link'] % image_link + '">' + \
                   action['name'] + '</a>]'
 
     return result
 
 
 @register.inclusion_tag('boards/post.html', name='post_view')
 def post_view(post, moderator=False):
     """
     Get post
     """
 
     return {
         'post': post,
         'moderator': moderator,
     }
 
 
 @register.inclusion_tag('boards/post.html', name='post_view_truncated')
 def post_view_truncated(post, need_open_link=False, moderator=False):
     """
     Get post with truncated text. If the 'open' or 'reply' link is needed, pass
     the second parameter as True.
     """
 
     return {
         'post': post,
         'truncated': True,
         'need_open_link': need_open_link,
         'moderator': moderator,
-    }
  No newline at end of file
+    }