From 5aa15954f73319bb493d27c0039f27455aec6de4 Mon Sep 17 00:00:00 2001 From: Nimisha Asthagiri Date: Tue, 12 May 2015 16:44:38 -0400 Subject: [PATCH] Minor update to video_outline API. --- .../mobile_api/video_outlines/serializers.py | 15 +++++++-------- 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/lms/djangoapps/mobile_api/video_outlines/serializers.py b/lms/djangoapps/mobile_api/video_outlines/serializers.py index 1e867c83c3..73f7089df6 100644 --- a/lms/djangoapps/mobile_api/video_outlines/serializers.py +++ b/lms/djangoapps/mobile_api/video_outlines/serializers.py @@ -140,20 +140,19 @@ def find_urls(course_id, block, child_to_parent, request): kwargs = {'course_id': unicode(course_id)} if chapter_id is None: - no_chapter_url = reverse("courseware", kwargs=kwargs, request=request) - return no_chapter_url, no_chapter_url + course_url = reverse("courseware", kwargs=kwargs, request=request) + return course_url, course_url kwargs['chapter'] = chapter_id if section is None: - no_section_url = reverse("courseware_chapter", kwargs=kwargs, request=request) - return no_section_url, no_section_url + chapter_url = reverse("courseware_chapter", kwargs=kwargs, request=request) + return chapter_url, chapter_url kwargs['section'] = section.url_name - if position is None: - no_position_url = reverse("courseware_section", kwargs=kwargs, request=request) - return no_position_url, no_position_url - section_url = reverse("courseware_section", kwargs=kwargs, request=request) + if position is None: + return section_url, section_url + kwargs['position'] = position unit_url = reverse("courseware_position", kwargs=kwargs, request=request) return unit_url, section_url