fix comments and replies

2012-12-04 15:47:42 -05:00
parent a5a38c0116
commit 8ad01c7340
6 changed files with 65 additions and 80 deletions
--- a/lms/static/coffee/src/discussion/content.coffee
+++ b/lms/static/coffee/src/discussion/content.coffee
@@ -79,6 +79,16 @@ if Backbone?
          @getContent(id).updateInfo(info)
      $.extend @contentInfos, infos
    
+    flagAbuse: ->
+      temp_array = @get("abuse_flaggers")
+      temp_array.push(window.user.get('id'))
+      @set("abuse_flaggers",temp_array)
+      @trigger "change", @
+
+    unflagAbuse: ->
+      @get("abuse_flaggers").pop(window.user.get('id'))
+      @trigger "change", @
+
    
  class @Thread extends @Content
    urlMappers:
@@ -116,16 +126,6 @@ if Backbone?
      @get("votes")["up_count"] = parseInt(@get("votes")["up_count"]) - 1
      @trigger "change", @
      
-    flagAbuse: ->
-      temp_array = @get("abuse_flaggers")
-      temp_array.push(window.user.get('id'))
-      @set("abuse_flaggers",temp_array)
-      @trigger "change", @
-
-    unflagAbuse: ->
-      @get("abuse_flaggers").pop(window.user.get('id'))
-      @trigger "change", @
-
    display_body: ->
      if @has("highlighted_body")
        String(@get("highlighted_body")).replace(/<highlight>/g, '<mark>').replace(/<\/highlight>/g, '</mark>')
@@ -164,6 +164,9 @@ if Backbone?
      'endorse': -> DiscussionUtil.urlFor('endorse_comment', @id)
      'update': -> DiscussionUtil.urlFor('update_comment', @id)
      'delete': -> DiscussionUtil.urlFor('delete_comment', @id)
+      'flagAbuse'   : -> DiscussionUtil.urlFor("flagAbuse_#{@get('type')}", @id)
+      'unFlagAbuse' : -> DiscussionUtil.urlFor("unFlagAbuse_#{@get('type')}", @id)
+


    getCommentsCount: ->
--- a/lms/static/coffee/src/discussion/views/discussion_content_view.coffee
+++ b/lms/static/coffee/src/discussion/views/discussion_content_view.coffee
@@ -104,3 +104,39 @@ if Backbone?
    initialize: ->
      @initLocal()
      @model.bind('change', @renderPartialAttrs, @)
+      
+     
+     
+    toggleFlagAbuse: (event) ->
+      event.preventDefault()
+      if window.user.id in @model.get("abuse_flaggers")
+        @unFlagAbuse()
+      else
+        @flagAbuse()
+      
+    flagAbuse: ->
+      url = @model.urlFor("flagAbuse")
+      DiscussionUtil.safeAjax
+        $elem: @$(".discussion-flag-abuse")
+        url: url
+        type: "POST"
+        success: (response, textStatus) =>
+          if textStatus == 'success'
+            ###
+            note, we have to clone the array in order to trigger a change event
+            ###
+            temp_array = _.clone(@model.get('abuse_flaggers'));
+            temp_array.push(window.user.id)
+            @model.set('abuse_flaggers', temp_array)      
+       
+    unFlagAbuse: ->
+      url = @model.urlFor("unFlagAbuse")
+      DiscussionUtil.safeAjax
+        $elem: @$(".discussion-flag-abuse")
+        url: url
+        type: "POST"
+        success: (response, textStatus) =>
+          if textStatus == 'success'
+            temp_array = _.clone(@model.get('abuse_flaggers'));
+            temp_array.pop(window.user.id)
+            @model.set('abuse_flaggers', temp_array)         
--- a/lms/static/coffee/src/discussion/views/discussion_thread_show_view.coffee
+++ b/lms/static/coffee/src/discussion/views/discussion_thread_show_view.coffee
@@ -66,19 +66,21 @@ if Backbone?
      if window.user.voted(@model)
        @unvote()
      else
-        @vote()
-
-    toggleFlagAbuse: (event) ->
-      event.preventDefault()
-      if window.user.id in @model.get("abuse_flaggers")
-        @unFlagAbuse()
-      else
-        @flagAbuse()
-
+        @vote()   

    toggleFollowing: (event) ->
      $elem = $(event.target)
-      url = null
+      url = nullunFlagAbuse: ->
+      url = @model.urlFor("unFlagAbuse")
+      DiscussionUtil.safeAjax
+        $elem: @$(".discussion-flag-abuse")
+        url: url
+        type: "POST"
+        success: (response, textStatus) =>
+          if textStatus == 'success'
+            temp_array = _.clone(@model.get('abuse_flaggers'));
+            temp_array.pop(window.user.id)
+            @model.set('abuse_flaggers', temp_array)
      if not @model.get('subscribed')
        @model.follow()
        url = @model.urlFor("follow")
@@ -101,21 +103,6 @@ if Backbone?
          if textStatus == 'success'
            @model.set(response, {silent: true})

-    flagAbuse: ->
-      url = @model.urlFor("flagAbuse")
-      DiscussionUtil.safeAjax
-        $elem: @$(".discussion-flag-abuse")
-        url: url
-        type: "POST"
-        success: (response, textStatus) =>
-          if textStatus == 'success'
-            ###
-            note, we have to clone the array in order to trigger a change event
-            that's 5 hours of my life i'll never get back.
-            ###
-            temp_array = _.clone(@model.get('abuse_flaggers'));
-            temp_array.push(window.user.id)
-            @model.set('abuse_flaggers', temp_array)

    unvote: ->
      window.user.unvote(@model)
@@ -128,17 +115,6 @@ if Backbone?
          if textStatus == 'success'
            @model.set(response, {silent: true})

-    unFlagAbuse: ->
-      url = @model.urlFor("unFlagAbuse")
-      DiscussionUtil.safeAjax
-        $elem: @$(".discussion-flag-abuse")
-        url: url
-        type: "POST"
-        success: (response, textStatus) =>
-          if textStatus == 'success'
-            temp_array = _.clone(@model.get('abuse_flaggers'));
-            temp_array.pop(window.user.id)
-            @model.set('abuse_flaggers', temp_array)

    edit: (event) ->
      @trigger "thread:edit", event
--- a/lms/static/coffee/src/discussion/views/response_comment_show_view.coffee
+++ b/lms/static/coffee/src/discussion/views/response_comment_show_view.coffee
@@ -34,3 +34,6 @@ if Backbone?
        @$el.find("a.profile-link").after('<span class="staff-label">staff</span>')
      else if DiscussionUtil.isTA(@model.get("user_id"))
        @$el.find("a.profile-link").after('<span class="community-ta-label">Community&nbsp;&nbsp;TA</span>')
+        
+        
+     
--- a/lms/static/coffee/src/discussion/views/thread_response_show_view.coffee
+++ b/lms/static/coffee/src/discussion/views/thread_response_show_view.coffee
@@ -72,39 +72,6 @@ if Backbone?
          if textStatus == 'success'
            @model.set(response)
            
-    toggleFlagAbuse: (event) ->
-      event.preventDefault()
-      if window.user.id in @model.get("abuse_flaggers")
-        @unFlagAbuse()
-      else
-        @flagAbuse()
-      
-    flagAbuse: ->
-      url = @model.urlFor("flagAbuse")
-      DiscussionUtil.safeAjax
-        $elem: @$(".discussion-flag-abuse")
-        url: url
-        type: "POST"
-        success: (response, textStatus) =>
-          if textStatus == 'success'
-            ###
-            note, we have to clone the array in order to trigger a change event
-            ###
-            temp_array = _.clone(@model.get('abuse_flaggers'));
-            temp_array.push(window.user.id)
-            @model.set('abuse_flaggers', temp_array)      
-       
-    unFlagAbuse: ->
-      url = @model.urlFor("unFlagAbuse")
-      DiscussionUtil.safeAjax
-        $elem: @$(".discussion-flag-abuse")
-        url: url
-        type: "POST"
-        success: (response, textStatus) =>
-          if textStatus == 'success'
-            temp_array = _.clone(@model.get('abuse_flaggers'));
-            temp_array.pop(window.user.id)
-            @model.set('abuse_flaggers', temp_array)            

    edit: (event) ->
        @trigger "response:edit", event
--- a/lms/templates/discussion/_underscore_templates.html
+++ b/lms/templates/discussion/_underscore_templates.html
@@ -138,7 +138,7 @@
  <div id="comment_${'<%- id %>'}">
    <div class="response-body">${'<%- body %>'}</div>
    <div class="discussion-flag-abuse notflagged" data-role="thread-flag" data-tooltip="flag">                
-      <i class="icon"></i><span class="flag-label">Report Misuse</span></div> 
+      <i class="icon"></i></div> 
    <p class="posted-details">&ndash;posted <span class="timeago" title="${'<%- created_at %>'}">${'<%- created_at %>'}</span> by
        ${"<% if (obj.username) { %>"}
        <a href="${'<%- user_url %>'}" class="profile-link">${'<%- username %>'}</a>