From 18814196f052fd0dc3915af8b32fd0cbd9101f6f Mon Sep 17 00:00:00 2001 From: Vik Paruchuri Date: Mon, 4 Feb 2013 18:31:32 -0500 Subject: [PATCH] Make things slightly robust --- lms/djangoapps/open_ended_grading/views.py | 24 +++++++++++++--------- 1 file changed, 14 insertions(+), 10 deletions(-) diff --git a/lms/djangoapps/open_ended_grading/views.py b/lms/djangoapps/open_ended_grading/views.py index dfba7b2599..a40645cca3 100644 --- a/lms/djangoapps/open_ended_grading/views.py +++ b/lms/djangoapps/open_ended_grading/views.py @@ -86,19 +86,23 @@ def peer_grading(request, course_id): course_id_parts = course.id.split("/") course_id_norun = "/".join(course_id_parts[0:2]) pg_location = "i4x://" + course_id_norun + "/peergrading/init" - log.debug("PG LOCATION :{0}".format(pg_location)) base_course_url = reverse('courses') - problem_url_parts = search.path_to_location(modulestore(), course.id, pg_location) - problem_url = base_course_url + "/" - for z in xrange(0,len(problem_url_parts)): - part = problem_url_parts[z] - if part is not None: - if z==1: - problem_url += "courseware/" - problem_url += part + "/" + try: + problem_url_parts = search.path_to_location(modulestore(), course.id, pg_location) + problem_url = base_course_url + "/" + for z in xrange(0,len(problem_url_parts)): + part = problem_url_parts[z] + if part is not None: + if z==1: + problem_url += "courseware/" + problem_url += part + "/" - return HttpResponseRedirect(problem_url) + return HttpResponseRedirect(problem_url) + except: + error_message = "Error with initializing peer grading. Centralized module does not exist. Please contact course staff." + log.error(error_message + "Current course is: {0}".format(course_id)) + return HttpResponse(error_message) @cache_control(no_cache=True, no_store=True, must_revalidate=True) def student_problem_list(request, course_id):