From 12dbdfa0f8ee15e7846707784a63573701a9851f Mon Sep 17 00:00:00 2001 From: Ibrahim Awwal Date: Mon, 10 Sep 2012 20:07:41 -0700 Subject: [PATCH] Removed some dead code and marked a method that I think is dead. Related templates should probably also be deleted. I did this mainly because it's confusing to have so many functions with similar names and there's a lot of unused stuff from discussion v1. --- .../django_comment_client/forum/views.py | 33 +------------------ 1 file changed, 1 insertion(+), 32 deletions(-) diff --git a/lms/djangoapps/django_comment_client/forum/views.py b/lms/djangoapps/django_comment_client/forum/views.py index f0565c56ac..560e19a97b 100644 --- a/lms/djangoapps/django_comment_client/forum/views.py +++ b/lms/djangoapps/django_comment_client/forum/views.py @@ -34,7 +34,7 @@ def _should_perform_search(request): def render_accordion(request, course, discussion_id): - + # TODO: Delete if obsolete discussion_info = utils.get_categorized_discussion_info(request, course) context = { @@ -178,16 +178,6 @@ def inline_discussion(request, course_id, discussion_id): 'num_pages': query_params['num_pages'] }) -def render_search_bar(request, course_id, discussion_id=None, text=''): - if not discussion_id: - return '' - context = { - 'discussion_id': discussion_id, - 'text': text, - 'course_id': course_id, - } - return render_to_string('discussion/_search_bar.html', context) - def forum_form_discussion(request, course_id): """ Renders the main Discussion page @@ -241,27 +231,6 @@ def forum_form_discussion(request, course_id): # print "start rendering.." return render_to_response('discussion/index.html', context) -def render_single_thread(request, discussion_id, course_id, thread_id): - - thread = cc.Thread.find(thread_id).retrieve(recursive=True).to_dict() - threads, query_params = get_threads(request, course_id) - - user_info = cc.User.from_django_user(request.user).to_dict() - - annotated_content_info = utils.get_annotated_content_infos(course_id, thread=thread, user=request.user, user_info=user_info) - - context = { - 'discussion_id': discussion_id, - 'thread': thread, - 'annotated_content_info': json.dumps(annotated_content_info), - 'course_id': course_id, - 'request': request, - 'discussion_data': json.dumps({ discussion_id: [utils.safe_content(thread)] }), - 'threads': threads, - } - - return render_to_string('discussion/_single_thread.html', context) - def single_thread(request, course_id, discussion_id, thread_id): if request.is_ajax():