From 4735ce0ab10e32e220d4fc6baa30369d9af0749c Mon Sep 17 00:00:00 2001
From: Matthew Mongeau <halogenandtoast@gmail.com>
Date: Wed, 29 Aug 2012 15:22:01 -0400
Subject: [PATCH] Display comments and active thread.

---
 .../src/discussion/discussion_router.coffee       | 15 ++++++---------
 .../views/discussion_thread_list_view.coffee      |  6 ++++++
 .../views/discussion_thread_view.coffee           | 13 ++++++++++++-
 .../discussion/views/response_comment_view.coffee |  6 ++++++
 .../discussion/views/thread_response_view.coffee  | 15 +++++++++++++++
 lms/templates/discussion/_single_thread.html      |  1 -
 lms/templates/discussion/single_thread.html       | 12 +++++++++++-
 7 files changed, 56 insertions(+), 12 deletions(-)
 create mode 100644 lms/static/coffee/src/discussion/views/response_comment_view.coffee
 create mode 100644 lms/static/coffee/src/discussion/views/thread_response_view.coffee

diff --git a/lms/static/coffee/src/discussion/discussion_router.coffee b/lms/static/coffee/src/discussion/discussion_router.coffee
index cd53d56304..7c1a5a4b7a 100644
--- a/lms/static/coffee/src/discussion/discussion_router.coffee
+++ b/lms/static/coffee/src/discussion/discussion_router.coffee
@@ -6,22 +6,19 @@ class @DiscussionRouter extends Backbone.Router
   initialize: (options) ->
       @user = options['user']
       @discussion = options['discussion']
-      @displayNav()
-      @forum = null
+      @nav = new DiscussionThreadListView(collection: @discussion, el: $(".post-list"))
+      @nav.on "thread:selected", @navigateToThread
+      @nav.render()
 
   allThreads: ->
       true
 
   showThread: (forum_name, thread_id) ->
-    @forum = forum_name
+    @nav.setActiveThread(thread_id)
     thread = @discussion.get(thread_id)
     view = new DiscussionThreadView(el: $(".discussion-column"), model: thread, user: @user)
     view.render()
 
-  displayNav: ->
-    view = new DiscussionThreadListView(collection: @discussion, el: $(".post-list"))
-    view.on "thread:selected", @navigateToThread
-    view.render()
-
   navigateToThread: (thread_id) =>
-    @navigate("#{@forum}/threads/#{thread_id}", trigger: true)
+    thread = @discussion.get(thread_id)
+    @navigate("#{thread.get("commentable_id")}/threads/#{thread_id}", trigger: true)
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 58251f18c9..1bca105aeb 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
@@ -9,4 +9,10 @@ class @DiscussionThreadListView extends Backbone.View
     @$el.append(view.el)
 
   threadSelected: (thread_id) =>
+    @setActiveThread(thread_id)
     @trigger("thread:selected", thread_id)
+
+  setActiveThread: (thread_id) ->
+    @$("a").removeClass("active")
+    @$("a[data-id='#{thread_id}']").addClass("active")
+
diff --git a/lms/static/coffee/src/discussion/views/discussion_thread_view.coffee b/lms/static/coffee/src/discussion/views/discussion_thread_view.coffee
index 674c5a6027..ddf1804a7e 100644
--- a/lms/static/coffee/src/discussion/views/discussion_thread_view.coffee
+++ b/lms/static/coffee/src/discussion/views/discussion_thread_view.coffee
@@ -18,9 +18,20 @@ class @DiscussionThreadView extends Backbone.View
 
     if @user.voted(@model)
       @$(".vote-btn").addClass("is-cast")
-
+    @$("span.timeago").timeago()
+    @renderResponses()
     @
 
+  renderResponses: ->
+    $.ajax @model.id, success: (data, textStatus, xhr) =>
+      comments = new Comments(data['content']['children'])
+      comments.each @renderResponse
+
+  renderResponse: (response) =>
+      view = new ThreadResponseView(model: response)
+      view.render()
+      @$(".responses").append(view.el)
+
   toggleVote: ->
     @$(".vote-btn").toggleClass("is-cast")
     if @$(".vote-btn").hasClass("is-cast")
diff --git a/lms/static/coffee/src/discussion/views/response_comment_view.coffee b/lms/static/coffee/src/discussion/views/response_comment_view.coffee
new file mode 100644
index 0000000000..0dbddab7ba
--- /dev/null
+++ b/lms/static/coffee/src/discussion/views/response_comment_view.coffee
@@ -0,0 +1,6 @@
+class @ResponseCommentView extends Backbone.View
+  tagName: "li"
+  template: _.template($("#response-comment-template").html())
+  render: ->
+    @$el.html(@template(@model.toJSON()))
+    @
diff --git a/lms/static/coffee/src/discussion/views/thread_response_view.coffee b/lms/static/coffee/src/discussion/views/thread_response_view.coffee
new file mode 100644
index 0000000000..5d531a2c18
--- /dev/null
+++ b/lms/static/coffee/src/discussion/views/thread_response_view.coffee
@@ -0,0 +1,15 @@
+class @ThreadResponseView extends Backbone.View
+  tagName: "li"
+  template: _.template($("#thread-response-template").html())
+  render: ->
+    @$el.html(@template(@model.toJSON()))
+    @renderComments()
+    @
+
+  renderComments: ->
+      @model.get("comments").each @renderComment
+
+  renderComment: (comment) =>
+    view = new ResponseCommentView(model: comment)
+    view.render()
+    @$(".comments").append(view.el)
diff --git a/lms/templates/discussion/_single_thread.html b/lms/templates/discussion/_single_thread.html
index 1939bfd753..bc08efb8b1 100644
--- a/lms/templates/discussion/_single_thread.html
+++ b/lms/templates/discussion/_single_thread.html
@@ -30,4 +30,3 @@
 </article>
 
 <%include file="_js_data.html" />
-<script type="text/javascript">$(document).ready(function() { $("span.timeago").timeago() })</script>
diff --git a/lms/templates/discussion/single_thread.html b/lms/templates/discussion/single_thread.html
index 60eae9e9c9..64d31f2f66 100644
--- a/lms/templates/discussion/single_thread.html
+++ b/lms/templates/discussion/single_thread.html
@@ -125,8 +125,18 @@
   </article>
 </script>
 
+<script type="text/template" id="thread-response-template">
+  <div class="response-body">${"<%= body %>"}</div>
+  <ol class="comments">
+  </ol>
+</script>
+
+<script type="text/template" id="response-comment-template">
+  <p>${'<%= body %>'}</p>
+</script>
+
 <script type="text/template" id="thread-list-item-template">
-    <a href="${'<%= id %>'}"><span class="title">${"<%= title %>"}</span> <span class="comments-count">${"<%= comments_count %>"}</span><span class="votes-count">+${"<%= votes['up_count'] %>"}</span></a>
+    <a href="${'<%= id %>'}" data-id="${'<%= id %>'}"><span class="title">${"<%= title %>"}</span> <span class="comments-count">${"<%= comments_count %>"}</span><span class="votes-count">+${"<%= votes['up_count'] %>"}</span></a>
 </script>
 
 <script>