From 4892897c0dc68c381c2407a3e45914c76304648b Mon Sep 17 00:00:00 2001 From: Chris Dodge Date: Mon, 4 Mar 2013 12:03:25 -0500 Subject: [PATCH] add depth parameters when loading course and student-state caches. This will make fewer (but fatter) queries to the database --- lms/djangoapps/courseware/grades.py | 2 +- lms/djangoapps/courseware/views.py | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/lms/djangoapps/courseware/grades.py b/lms/djangoapps/courseware/grades.py index 89e7371efa..07366b71cb 100644 --- a/lms/djangoapps/courseware/grades.py +++ b/lms/djangoapps/courseware/grades.py @@ -270,7 +270,7 @@ def progress_summary(student, request, course, student_module_cache): # would be simpler course_module = get_module(student, request, course.location, student_module_cache, - course.id) + course.id, depth=None) if not course_module: # This student must not have access to the course. return None diff --git a/lms/djangoapps/courseware/views.py b/lms/djangoapps/courseware/views.py index 8b48572818..c249aaf4df 100644 --- a/lms/djangoapps/courseware/views.py +++ b/lms/djangoapps/courseware/views.py @@ -570,7 +570,7 @@ def progress(request, course_id, student_id=None): Course staff are allowed to see the progress of students in their class. """ - course = get_course_with_access(request.user, course_id, 'load') + course = get_course_with_access(request.user, course_id, 'load', depth=None) staff_access = has_access(request.user, course, 'staff') if student_id is None or student_id == request.user.id: @@ -590,7 +590,7 @@ def progress(request, course_id, student_id=None): student = User.objects.prefetch_related("groups").get(id=student.id) student_module_cache = StudentModuleCache.cache_for_descriptor_descendents( - course_id, student, course) + course_id, student, course, depth=None) courseware_summary = grades.progress_summary(student, request, course, student_module_cache)