From 0ff8108ebe4893044854490214cef5da405ca905 Mon Sep 17 00:00:00 2001
From: Tanshu <tanshu@gmail.com>
Date: Fri, 7 Jun 2013 18:47:15 +0530
Subject: [PATCH] Allow filtering of Threads based on open / closed / all on
 home.html

---
 brewman/brewman/static/offline.appcache  |  2 +-
 brewman/brewman/static/partial/home.html |  6 ++--
 brewman/brewman/static/scripts/home.js   | 14 +++++----
 brewman/brewman/views/messaging.py       | 36 ++++++++++++++----------
 4 files changed, 34 insertions(+), 24 deletions(-)

diff --git a/brewman/brewman/static/offline.appcache b/brewman/brewman/static/offline.appcache
index d3497fa9..247d8b7c 100644
--- a/brewman/brewman/static/offline.appcache
+++ b/brewman/brewman/static/offline.appcache
@@ -1,6 +1,6 @@
 CACHE MANIFEST
       
-# version 2013-06-06.3
+# version 2013-06-07.1
 
 CACHE:
 /partial/404.html
diff --git a/brewman/brewman/static/partial/home.html b/brewman/brewman/static/partial/home.html
index 6647a80a..62f3e84a 100644
--- a/brewman/brewman/static/partial/home.html
+++ b/brewman/brewman/static/partial/home.html
@@ -3,9 +3,9 @@
 <div class="widget-box">
     <div class="widget-title">
         <ul class="nav nav-tabs">
-            <li><a href="#">Open</a></li>
-            <li><a href="#">Closed</a></li>
-            <li><a href="#">All</a></li>
+            <li ng-class="{active: chosen == 'open'}"><a href ng-click="getMessages('open')">Open</a></li>
+            <li ng-class="{active: chosen == 'closed'}"><a href ng-click="getMessages('closed')">Closed</a></li>
+            <li ng-class="{active: chosen == 'all'}"><a href ng-click="getMessages('all')">All</a></li>
             <li class="dropdown pull-right">
                 <a href="#" class="dropdown-toggle" data-toggle="dropdown">Tags
                     <b class="caret"></b>
diff --git a/brewman/brewman/static/scripts/home.js b/brewman/brewman/static/scripts/home.js
index ede236d6..4de9cd94 100644
--- a/brewman/brewman/static/scripts/home.js
+++ b/brewman/brewman/static/scripts/home.js
@@ -1,12 +1,16 @@
 'use strict';
 
-var HomeCtrl = ['$scope', '$location', 'messages', function ($scope, $location, messages) {
+var HomeCtrl = ['$scope', '$location', 'messages', 'Message', function ($scope, $location, messages, Message) {
+    $scope.chosen = messages.Type;
     $scope.info = messages.Threads;
     $scope.tags = messages.Tags
-    $scope.openMessage = function (item) {
-        $location.path('/Message/' + item.ThreadID);
+    $scope.getMessages = function (type) {
+        Message.query({type: type}, function (result) {
+            $scope.info = result.Threads;
+            $scope.tags = result.Tags;
+            $scope.chosen = result.Type;
+        });
     }
-
 }];
 HomeCtrl.resolve = {
     messages: ['$q', '$route', 'Message', function ($q, $route, Message) {
@@ -16,7 +20,7 @@ HomeCtrl.resolve = {
             deferred.resolve(result);
         };
 
-        Message.query({}, successCb);
+        Message.query({type: 'open'}, successCb);
         return deferred.promise;
     }]
 };
\ No newline at end of file
diff --git a/brewman/brewman/views/messaging.py b/brewman/brewman/views/messaging.py
index 22e33525..032dab9d 100644
--- a/brewman/brewman/views/messaging.py
+++ b/brewman/brewman/views/messaging.py
@@ -54,12 +54,13 @@ def save(request):
 @TryCatchFunction
 def update(request):
     user_id = uuid.UUID(authenticated_userid(request))
+    super_user = 'Messages' in groupfinder(user_id, request)
     thread = Thread.by_id(uuid.UUID(request.matchdict['id']))
     public = request.json_body['Public']
     closed = request.json_body['Closed']
-    if thread.public and not public:
+    if super_user or (thread.public and not public):
         thread.public = False
-    if not thread.closed and closed:
+    if super_user or (not thread.closed and closed):
         thread.closed = True
     thread.priority = request.json_body['Priority']
 
@@ -105,12 +106,23 @@ def show_id(request):
 
 @view_config(request_method='GET', route_name='api_message', renderer='json', request_param='list')
 def show_list(request):
-    user_id = None if  authenticated_userid(request) is None else uuid.UUID(authenticated_userid(request))
-    list = Thread.query().filter(Thread.closed == False)
+    user_id = authenticated_userid(request) and uuid.UUID(authenticated_userid(request))
+    list = Thread.query()
+
+    type = request.params.get('type', 'open')
+    if type == 'all':
+        pass
+    elif type == 'closed':
+        list = list.filter(Thread.closed == True)
+    else:
+        type = 'open'
+        list = list.filter(Thread.closed == False)
+
     if user_id is None:
         list = list.filter(Thread.public == True)
     elif 'Messages' not in groupfinder(user_id, request):
         list = list.filter(Thread.subscribers.any(Subscriber.user_id == user_id))
+
     list = list.order_by(Thread.priority).all()
     tags = {}
     threads = []
@@ -118,24 +130,18 @@ def show_list(request):
         thread = {'ThreadID': item.id, 'Title': item.title, 'Age': get_age(item.creation_date),
                   'CreationDate': item.creation_date.strftime('%d-%b-%Y %H:%M'), 'User': item.user.name,
                   'Priority': item.priority, 'Public': item.public, 'Tags': [], 'Posts': [], 'Subscribers': []}
+        last_updated = max(post.creation_date for post in item.posts) or item.creation_date
+        thread['LastUpdated'] = get_age(last_updated)
+
         for tag in item.tags:
             thread['Tags'].append(tag.name)
             if not tag.name in tags:
                 tags[tag.name] = 1
             else:
                 tags[tag.name] += 1
-        for subscriber in item.subscribers:
-            thread['Subscribers'].append(subscriber.user.name)
-        last_updated = item.creation_date
-        for post in item.posts:
-            if post.creation_date > last_updated:
-                last_updated = post.creation_date
-                # thread['Posts'].append({'PostID': post.id, 'Content': post.content, 'User': post.user.name,
-                #                         'Date': post.date.strftime('%d-%b-%Y %H:%M'),
-                #                         'CreationDate': post.creation_date.strftime('%d-%b-%Y %H:%M')})
+
         threads.append(thread)
-        thread['LastUpdated'] = get_age(last_updated)
-    return {'Tags': tags, 'Threads': threads}
+    return {'Tags': tags, 'Threads': threads, 'Type': type}
 
 
 def thread_info(id):