Commit b1fc36d5 by Greg Price

Merge pull request #2514 from edx/gprice/fix-single-thread-recursive

Avoid requesting unnecessary thread responses
parents 2086fcb8 bc0ca63f
...@@ -239,8 +239,11 @@ def single_thread(request, course_id, discussion_id, thread_id): ...@@ -239,8 +239,11 @@ def single_thread(request, course_id, discussion_id, thread_id):
cc_user = cc.User.from_django_user(request.user) cc_user = cc.User.from_django_user(request.user)
user_info = cc_user.to_dict() user_info = cc_user.to_dict()
# Currently, the front end always loads responses via AJAX, even for this
# page; it would be a nice optimization to avoid that extra round trip to
# the comments service.
thread = cc.Thread.find(thread_id).retrieve( thread = cc.Thread.find(thread_id).retrieve(
recursive=True, recursive=request.is_ajax(),
user_id=request.user.id, user_id=request.user.id,
response_skip=request.GET.get("resp_skip"), response_skip=request.GET.get("resp_skip"),
response_limit=request.GET.get("resp_limit") response_limit=request.GET.get("resp_limit")
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment