From eaa76ae7584d0938db0298714303e2c23b3eacfd Mon Sep 17 00:00:00 2001
From: Dan McGee <dan@archlinux.org>
Date: Thu, 9 Sep 2010 13:36:41 -0500
Subject: Improve todo list view page

Add total package count and incomplete package count columns. Also reduce
the number of total queries by killing the query per row that was happening
before.

Signed-off-by: Dan McGee <dan@archlinux.org>
---
 templates/todolists/list.html | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

(limited to 'templates/todolists')

diff --git a/templates/todolists/list.html b/templates/todolists/list.html
index 29829caa..2e75bdac 100644
--- a/templates/todolists/list.html
+++ b/templates/todolists/list.html
@@ -20,6 +20,8 @@ <h2>Package Todo Lists</h2>
                 <th>Creation Date</th>
                 <th>Creator</th>
                 <th>Description</th>
+                <th>Package Count</th>
+                <th>Incomplete Count</th>
                 <th>Status</th>
             </tr>
         </thead>
@@ -31,8 +33,10 @@ <h2>Package Todo Lists</h2>
                 <td>{{ list.date_added }}</td>
                 <td>{{ list.creator.get_full_name }}</td>
                 <td class="wrap">{{ list.description|safe }}</td>
-                <td>{% if list.complete %}<span class="complete">Complete</span>
-                    {% else %}<span class="incomplete">Incomplete</span>{% endif %}</td>
+                <td>{{ list.pkg_count }}</td>
+                <td>{{ list.incomplete_count }}</td>
+                <td>{% ifequal list.incomplete_count 0 %}<span class="complete">Complete</span>
+                    {% else %}<span class="incomplete">Incomplete</span>{% endifequal %}</td>
             </tr>
             {% endfor %}
         </tbody>
@@ -42,7 +46,10 @@ <h2>Package Todo Lists</h2>
 <script type="text/javascript" src="/media/jquery.tablesorter.min.js"></script>
 <script type="text/javascript">
 $(document).ready(function() {
-    $(".results").tablesorter({widgets: ['zebra'], sortList: [[1,1]]});
+    // I'm not sure why it didn't autodetect digit, but it has to be explicit
+    // http://stackoverflow.com/questions/302749/jquery-tablesorter-problem
+    $(".results").tablesorter({widgets: ['zebra'], sortList: [[1,1]],
+        headers: { 4: { sorter: 'digit' }, 5: { sorter: 'digit' } } });
 });
 </script>
 {% endblock %}
-- 
cgit v1.2.3-54-g00ecf