From 862a16cac0b905b166d82a35750264d9c5883593 Mon Sep 17 00:00:00 2001
From: Greg Price <gprice@edx.org>
Date: Thu, 17 Apr 2014 12:36:03 -0400
Subject: [PATCH] Show full body when expanding inline forum thread

This was a bug in 3af90ef.

JIRA: FOR-559
---
 .../view/discussion_thread_view_inline_spec.coffee  | 13 +++++++++++++
 .../views/discussion_thread_view_inline.coffee      |  2 ++
 2 files changed, 15 insertions(+)

diff --git a/common/static/coffee/spec/discussion/view/discussion_thread_view_inline_spec.coffee b/common/static/coffee/spec/discussion/view/discussion_thread_view_inline_spec.coffee
index 88304ae82b..9e8582e504 100644
--- a/common/static/coffee/spec/discussion/view/discussion_thread_view_inline_spec.coffee
+++ b/common/static/coffee/spec/discussion/view/discussion_thread_view_inline_spec.coffee
@@ -5,6 +5,7 @@ describe "DiscussionThreadInlineView", ->
             <script type="text/template" id="_inline_thread">
                 <article class="discussion-article">
                     <div class="non-cohorted-indicator"/>
+                    <div class="post-body"/>
                     <div class="post-extended-content">
                         <div class="response-count"/> 
                         <ol class="responses"/>
@@ -19,6 +20,7 @@ describe "DiscussionThreadInlineView", ->
             <script type="text/template" id="_inline_thread_cohorted">
                 <article class="discussion-article">
                     <div class="cohorted-indicator"/>
+                    <div class="post-body"/>
                     <div class="post-extended-content">
                         <div class="response-count"/> 
                         <ol class="responses"/>
@@ -87,3 +89,14 @@ describe "DiscussionThreadInlineView", ->
             assertExpandedContentVisible(@view, true)
             @view.collapsePost()
             assertExpandedContentVisible(@view, false)
+
+        it "switches between the abbreviated and full body", ->
+            DiscussionViewSpecHelper.setNextResponseContent({resp_total: 0, children: []})
+            @thread.set("body", new Array(100).join("test "))
+            @view.abbreviateBody()
+            expect(@thread.get("body")).not.toEqual(@thread.get("abbreviatedBody"))
+            @view.render()
+            @view.expandPost()
+            expect(@view.$el.find(".post-body").text()).toEqual(@thread.get("body"))
+            @view.collapsePost()
+            expect(@view.$el.find(".post-body").text()).toEqual(@thread.get("abbreviatedBody"))
diff --git a/common/static/coffee/src/discussion/views/discussion_thread_view_inline.coffee b/common/static/coffee/src/discussion/views/discussion_thread_view_inline.coffee
index b15f361472..bd4494f9a6 100644
--- a/common/static/coffee/src/discussion/views/discussion_thread_view_inline.coffee
+++ b/common/static/coffee/src/discussion/views/discussion_thread_view_inline.coffee
@@ -45,6 +45,7 @@ if Backbone?
 
     expandPost: (event) =>
       @$el.addClass('expanded')
+      @$el.find('.post-body').html(@model.get('body'))
       @$el.find('.expand-post').css('display', 'none')
       @$el.find('.collapse-post').css('display', 'block')
       @$el.find('.post-extended-content').show()
@@ -58,6 +59,7 @@ if Backbone?
       if postTop < curScroll
         $('html, body').animate({scrollTop: postTop})
       @$el.removeClass('expanded')
+      @$el.find('.post-body').html(@model.get('abbreviatedBody'))
       @$el.find('.expand-post').css('display', 'block')
       @$el.find('.collapse-post').css('display', 'none')
       @$el.find('.post-extended-content').hide()