diff --git a/lms/djangoapps/django_comment_client/forum/views.py b/lms/djangoapps/django_comment_client/forum/views.py index 164ab77d1d..69f426055f 100644 --- a/lms/djangoapps/django_comment_client/forum/views.py +++ b/lms/djangoapps/django_comment_client/forum/views.py @@ -43,12 +43,10 @@ def render_discussion(request, course_id, threads, discussion_id=None, \ 'forum': 'discussion/_forum.html', }[discussion_type] - """base_url = { + base_url = { 'inline': (lambda: reverse('django_comment_client.forum.views.inline_discussion', args=[course_id, discussion_id])), 'forum': (lambda: reverse('django_comment_client.forum.views.forum_form_discussion', args=[course_id, discussion_id])), - }[discussion_type]()""" - - base_url = reverse('django_comment_client.forum.views.inline_discussion', args=[course_id, discussion_id]) + }[discussion_type]() annotated_content_info = {thread['id']: get_annotated_content_info(thread, request.user.id) for thread in threads} @@ -116,13 +114,17 @@ def forum_form_discussion(request, course_id, discussion_id): threads, query_params = get_threads(request, course_id, discussion_id) content = render_forum_discussion(request, course_id, threads, discussion_id=discussion_id, \ query_params=query_params) - context = { - 'csrf': csrf(request)['csrf_token'], - 'course': course, - 'content': content, - 'accordion': render_accordion(request, course, discussion_id), - } - return render_to_response('discussion/index.html', context) + + if request.is_ajax(): + return utils.HtmlResponse(content) + else: + context = { + 'csrf': csrf(request)['csrf_token'], + 'course': course, + 'content': content, + 'accordion': render_accordion(request, course, discussion_id), + } + return render_to_response('discussion/index.html', context) def get_annotated_content_info(content, user_id): diff --git a/lms/static/coffee/src/discussion/discussion.coffee b/lms/static/coffee/src/discussion/discussion.coffee index 4c757e12a6..5d93ca2ad9 100644 --- a/lms/static/coffee/src/discussion/discussion.coffee +++ b/lms/static/coffee/src/discussion/discussion.coffee @@ -122,13 +122,7 @@ initializeFollowDiscussion = (discussion) -> Discussion.bindLocalEvents $local, - "submit .search-wrapper-forum>.discussion-search-form": (event) -> - event.preventDefault() - text = $local(".search-input").val() - isSearchWithinBoard = $local(".discussion-search-within-board").is(":checked") - handleSearch(text, isSearchWithinBoard) - - "submit .search-wrapper-inline>.discussion-search-form": (event) -> + "submit .search-wrapper>.discussion-search-form": (event) -> event.preventDefault() handleAjaxSearch(this) @@ -138,7 +132,7 @@ initializeFollowDiscussion = (discussion) -> "click .discussion-search-link": -> handleAjaxSearch(this) - "click .discussion-inline-sort-link": -> + "click .discussion-sort-link": -> handleAjaxSort(this) $discussion.children(".discussion-paginator").find(".discussion-inline-page-link").click -> diff --git a/lms/templates/discussion/_forum.html b/lms/templates/discussion/_forum.html index 87609f92b9..ea30bb6f4f 100644 --- a/lms/templates/discussion/_forum.html +++ b/lms/templates/discussion/_forum.html @@ -5,7 +5,7 @@
-