From 91befca89cfc841868ec7f8ff727261da58c1592 Mon Sep 17 00:00:00 2001 From: Victor Shnayder Date: Wed, 20 Jun 2012 11:31:21 -0400 Subject: [PATCH] Fix render of lms index with no chapter or section. --- lms/djangoapps/courseware/views.py | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/lms/djangoapps/courseware/views.py b/lms/djangoapps/courseware/views.py index aa3d1b7781..5cbbe18d7d 100644 --- a/lms/djangoapps/courseware/views.py +++ b/lms/djangoapps/courseware/views.py @@ -82,7 +82,11 @@ def profile(request, student_id=None): def render_accordion(request, course, chapter, section): ''' Draws navigation bar. Takes current position in accordion as - parameter. Returns (initialization_javascript, content)''' + parameter. + + If chapter and section are '' or None, renders a default accordion. + + Returns (initialization_javascript, content)''' if not course: course = "6.002 Spring 2012" @@ -221,10 +225,8 @@ def index(request, course=None, chapter=None, section=None, ''' Fixes URLs -- we convert spaces to _ in URLs to prevent funny encoding characters and keep the URLs readable. This undoes that transformation. - - TODO: Properly replace underscores. (Q: what is properly?) ''' - return s.replace('_', ' ') + return s.replace('_', ' ') if s is not None else None def get_submodule_ids(module_xml): '''