From 259e6e46adb7a67eb63715e639a32fc5a5646487 Mon Sep 17 00:00:00 2001 From: Ibrahim Awwal Date: Mon, 10 Sep 2012 12:59:14 -0700 Subject: [PATCH] Fix discussion module It worked in Chrome because Chrome appears to magically infer some missing arguments to methods. This is the second time this happened, should be more careful about it in the future, because it causes confusion when features work for some students but not others. --- .../coffee/src/discussion/discussion_module_view.coffee | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lms/static/coffee/src/discussion/discussion_module_view.coffee b/lms/static/coffee/src/discussion/discussion_module_view.coffee index 3697faddee..670703e2f4 100644 --- a/lms/static/coffee/src/discussion/discussion_module_view.coffee +++ b/lms/static/coffee/src/discussion/discussion_module_view.coffee @@ -56,12 +56,12 @@ if Backbone? url: url type: "GET" dataType: 'json' - success: (response, textStatus, jqXHR) => @renderDiscussion(event, response, textStatus, discussionId) + success: (response, textStatus, jqXHR) => @renderDiscussion($elem, response, textStatus, discussionId) - renderDiscussion: (event, response, textStatus, discussionId) => + renderDiscussion: ($elem, response, textStatus, discussionId) => window.user = new DiscussionUser(response.user_info) Content.loadContentInfos(response.annotated_content_info) - $(event.target).html("Hide Discussion") + $elem.html("Hide Discussion") @discussion = new Discussion() @discussion.reset(response.discussion_data, {silent: false}) $discussion = $(Mustache.render $("script#_inline_discussion").html(), {'threads':response.discussion_data, 'discussionId': discussionId})