From 62f6e2d52b8cffc70c5a1c00274ac685bc72ec65 Mon Sep 17 00:00:00 2001
From: Kevin Chugh <kevinchugh@gmail.com>
Date: Mon, 29 Jul 2013 23:33:50 -0400
Subject: [PATCH] home button working, default to home when no thread selected

---
 .../src/discussion/discussion_router.coffee   |  1 +
 .../static/coffee/src/discussion/utils.coffee |  1 +
 .../views/discussion_thread_list_view.coffee  | 24 ++++++++++---------
 lms/lib/comment_client/user.py                |  2 --
 4 files changed, 15 insertions(+), 13 deletions(-)

diff --git a/common/static/coffee/src/discussion/discussion_router.coffee b/common/static/coffee/src/discussion/discussion_router.coffee
index dcfeb367ef..283632a542 100644
--- a/common/static/coffee/src/discussion/discussion_router.coffee
+++ b/common/static/coffee/src/discussion/discussion_router.coffee
@@ -20,6 +20,7 @@ if Backbone?
 
     allThreads: ->
       @nav.updateSidebar()
+      @nav.goHome()
 
     setActiveThread: =>
       if @thread
diff --git a/common/static/coffee/src/discussion/utils.coffee b/common/static/coffee/src/discussion/utils.coffee
index 69ea39ad03..50d65c70dd 100644
--- a/common/static/coffee/src/discussion/utils.coffee
+++ b/common/static/coffee/src/discussion/utils.coffee
@@ -84,6 +84,7 @@ class @DiscussionUtil
       threads                 : "/courses/#{$$course_id}/discussion/forum"
       "enable_notifications"  : "/notification_prefs/enable/"
       "disable_notifications" : "/notification_prefs/disable/"
+      "notifications_status" : "/user_api/v1/user_prefs/?key=notification_pref&use=#{param}"
     }[name]
 
   @safeAjax: (params) ->
diff --git a/common/static/coffee/src/discussion/views/discussion_thread_list_view.coffee b/common/static/coffee/src/discussion/views/discussion_thread_list_view.coffee
index 69b7f3e07f..fe5e726e9a 100644
--- a/common/static/coffee/src/discussion/views/discussion_thread_list_view.coffee
+++ b/common/static/coffee/src/discussion/views/discussion_thread_list_view.coffee
@@ -198,12 +198,20 @@ if Backbone?
       $(".discussion-column").html(@template)
       $(".post-list a").removeClass("active")
       $("input.email-setting").bind "click", @updateEmailNotifications
-      url=DiscussionUtil.urlFor("get_notifications_state",window.user.get("id"))
+      url = DiscussionUtil.urlFor("notifications_status",window.user.get("id"))
       DiscussionUtil.safeAjax
           url: url
-          type: "POST"
+          type: "GET"
           dataType: 'json'
+          success: (response, textStatus) =>
+            if response.count==1
+              $('input.email-setting').attr('checked','checked')
+            else
+              $('input.email-setting').removeAttr('checked')
       thread_id = null
+      @trigger("thread:removed")  
+      #select all threads
+
 
     toggleTopicDrop: (event) =>
       event.preventDefault()
@@ -435,25 +443,19 @@ if Backbone?
       @retrieveFirstPage(event)
 
     updateEmailNotifications: () =>
-      if $('input.email-notification').attr('checked')
+      if $('input.email-setting').attr('checked')
         DiscussionUtil.safeAjax
           url: DiscussionUtil.urlFor("enable_notifications")
           type: "POST"
-          dataType: 'json'
-          data:
-            user: window.user.get("id")
           error: () =>
-            $('input.email-notification').removeAttr('checked')
+            $('input.email-setting').removeAttr('checked')
 
             
       else
         DiscussionUtil.safeAjax
           url: DiscussionUtil.urlFor("disable_notifications")
           type: "POST"
-          dataType: 'json'
-          data:
-            user: window.user.get("id")
           error: () =>
-            $('input.email-notification').attr('checked','checked')
+            $('input.email-setting').attr('checked','checked')
 
           
diff --git a/lms/lib/comment_client/user.py b/lms/lib/comment_client/user.py
index e686b9a052..1af64cc18c 100644
--- a/lms/lib/comment_client/user.py
+++ b/lms/lib/comment_client/user.py
@@ -97,7 +97,6 @@ def _url_for_subscription(user_id):
 
 
 def _url_for_user_active_threads(user_id):
-    print("\n\n\n*********** ACTIVE THREADS **********\n\n\n")
     return "{prefix}/users/{user_id}/active_threads".format(prefix=settings.PREFIX, user_id=user_id)
 
 
@@ -105,7 +104,6 @@ def _url_for_user_subscribed_threads(user_id):
     return "{prefix}/users/{user_id}/subscribed_threads".format(prefix=settings.PREFIX, user_id=user_id)
 
 def _url_for_user_stats(user_id,course_id):
-    print("\n\n\n*********** USER STATS **********\n\n\n")
     return "{prefix}/users/{user_id}/stats?course_id={course_id}".format(prefix=settings.PREFIX, user_id=user_id,course_id=course_id)