# HG changeset patch
# User neko259 <neko259@gmail.com>
# Date 2013-09-10 17:57:54
# Node ID cc12beaafca2acd1e3484c7369bc621803c50a7b
# Parent  bc8fce57a613175450b8b6d933cdd85f22c04658

Optimized thread view a lot.

diff --git a/boards/models.py b/boards/models.py
--- a/boards/models.py
+++ b/boards/models.py
@@ -246,7 +246,9 @@ class Post(models.Model):
 
     def get_images_count(self):
         images_count = 1 if self.image else 0
-        for reply in self._get_replies():
+
+        replies = self._get_replies()
+        for reply in replies:
             if reply.image:
                 images_count += 1
 
@@ -255,7 +257,7 @@ class Post(models.Model):
     def can_bump(self):
         """Check if the thread can be bumped by replying"""
 
-        replies_count = len(Post.objects.get_thread(self.id))
+        replies_count = self.get_reply_count() + 1
 
         return replies_count <= settings.MAX_POSTS_PER_THREAD
 
@@ -292,13 +294,8 @@ class User(models.Model):
         return setting
 
     def get_setting(self, name):
-        settings = Setting.objects.filter(name=name, user=self)
-        if len(settings) > 0:
-            setting = settings[0]
-        else:
-            setting = None
-
-        if setting:
+        if Setting.objects.filter(name=name, user=self).exists():
+            setting = Setting.objects.get(name=name, user=self)
             setting_value = setting.value
         else:
             setting_value = None
diff --git a/boards/templates/boards/posting_general.html b/boards/templates/boards/posting_general.html
--- a/boards/templates/boards/posting_general.html
+++ b/boards/templates/boards/posting_general.html
@@ -23,9 +23,9 @@
         {% for thread in threads %}
             <div class="thread">
             {% if thread.can_bump %}
-		<div class="post" id="{{thread.id}}">
+		        <div class="post" id="{{thread.id}}">
             {% else %}
-		<div class="post dead_post" id="{{ thread.id }}">
+		        <div class="post dead_post" id="{{ thread.id }}">
             {% endif %}
             {% if thread.image %}
                 <div class="image">
@@ -64,7 +64,7 @@
             <div class="metadata">
                 {{ thread.get_reply_count }} {% trans 'replies' %},
                 {{ thread.get_images_count }} {% trans 'images' %}.
-                {% if thread.tags.all %}
+                {% if thread.tags.exists %}
                     <span class="tags">{% trans 'Tags' %}:
                         {% for tag in thread.tags.all %}
                             <a class="tag" href="
@@ -79,9 +79,9 @@
                 <div class="last-replies">
                     {% for post in thread.get_last_replies %}
                         {% if thread.can_bump %}
-			    <div class="post" id="{{ post.id }}">
+			                <div class="post" id="{{ post.id }}">
                         {% else %}
-			    <div class="post dead_post id="{{ post.id }}"">
+			                <div class="post dead_post id="{{ post.id }}">
                         {% endif %}
                             {% if post.image %}
                                 <div class="image">
diff --git a/boards/templates/boards/thread.html b/boards/templates/boards/thread.html
--- a/boards/templates/boards/thread.html
+++ b/boards/templates/boards/thread.html
@@ -13,7 +13,7 @@
 	{% if posts %}
         <div id="posts">
 		{% for post in posts %}
-            {% if posts.0.can_bump %}
+            {% if bumpable %}
                 <div class="post" id="{{ post.id }}">
             {% else %}
                 <div class="post dead_post" id="{{ post.id }}">
@@ -53,7 +53,7 @@
                 {{ post.text.rendered }}
                 {% endautoescape %}
             </div>
-            {% if post.tags.all %}
+            {% if post.id == posts.0.id %}
                 <div class="metadata">
                     <span class="tags">{% trans 'Tags' %}:
                         {% for tag in post.tags.all %}
diff --git a/boards/views.py b/boards/views.py
--- a/boards/views.py
+++ b/boards/views.py
@@ -148,6 +148,7 @@ def thread(request, post_id):
 
     context['posts'] = posts
     context['form'] = form
+    context['bumpable'] = posts[0].can_bump()
 
     return render(request, 'boards/thread.html', context)