From 90f37b3344274db67cfc118f0f377c09227f5cdc Mon Sep 17 00:00:00 2001
From: Arjun Singh <arjun810@gmail.com>
Date: Mon, 17 Sep 2012 14:10:28 -0700
Subject: [PATCH] Don't use the search API to retrieve all threads.

---
 .../django_comment_client/forum/views.py      | 20 -------------------
 .../views/discussion_thread_list_view.coffee  | 15 +++++++++++++-
 lms/templates/discussion/_accordion.html      |  0
 3 files changed, 14 insertions(+), 21 deletions(-)
 delete mode 100644 lms/templates/discussion/_accordion.html

diff --git a/lms/djangoapps/django_comment_client/forum/views.py b/lms/djangoapps/django_comment_client/forum/views.py
index 573badbb92..0abcbf5fbb 100644
--- a/lms/djangoapps/django_comment_client/forum/views.py
+++ b/lms/djangoapps/django_comment_client/forum/views.py
@@ -28,26 +28,6 @@ PAGES_NEARBY_DELTA = 2
 escapedict = {'"': '&quot;'}
 log = logging.getLogger("edx.discussions")
 
-def _general_discussion_id(course_id):
-    return course_id.replace('/', '_').replace('.', '_')
-
-def _should_perform_search(request):
-    return bool(request.GET.get('text', False) or \
-            request.GET.get('tags', False))
-
-def render_accordion(request, course, discussion_id):
-    # TODO: Delete if obsolete
-    discussion_info = utils.get_categorized_discussion_info(request, course)
-
-    context = {
-        'course': course,
-        'discussion_info': discussion_info,
-        'active': discussion_id,
-        'csrf': csrf(request)['csrf_token'],
-    }
-
-    return render_to_string('discussion/_accordion.html', context)
-
 def get_threads(request, course_id, discussion_id=None, per_page=THREADS_PER_PAGE):
     """
     This may raise cc.utils.CommentClientError or
diff --git a/lms/static/coffee/src/discussion/views/discussion_thread_list_view.coffee b/lms/static/coffee/src/discussion/views/discussion_thread_list_view.coffee
index d4c144f808..6b8abbdc91 100644
--- a/lms/static/coffee/src/discussion/views/discussion_thread_list_view.coffee
+++ b/lms/static/coffee/src/discussion/views/discussion_thread_list_view.coffee
@@ -244,7 +244,8 @@ if Backbone?
         item = $(event.target).closest('li')
         if item.find("span.board-name").data("discussion_id") == "#all"
           @discussionIds = ""
-          @clearSearch()
+          @$(".post-search-field").val("")
+          @retrieveAllThreads()
         else
           discussionIds = _.map item.find(".board-name[data-discussion_id]"), (board) -> $(board).data("discussion_id").id
           @retrieveDiscussions(discussionIds)
@@ -277,6 +278,18 @@ if Backbone?
           Content.loadContentInfos(response.content_info)
           @displayedCollection.reset(@collection.models)
 
+    retrieveAllThreads: () ->
+      url = DiscussionUtil.urlFor("threads")
+      DiscussionUtil.safeAjax
+        url: url
+        type: "GET"
+        success: (response, textStatus) =>
+          @collection.current_page = response.page
+          @collection.pages = response.num_pages
+          @collection.reset(response.discussion_data)
+          Content.loadContentInfos(response.content_info)
+          @displayedCollection.reset(@collection.models)
+
     sortThreads: (event) ->
       @$(".sort-bar a").removeClass("active")
       $(event.target).addClass("active")
diff --git a/lms/templates/discussion/_accordion.html b/lms/templates/discussion/_accordion.html
deleted file mode 100644
index e69de29bb2..0000000000