Merge pull request #1249 from edx/fix/cdodge/progress-perf
we are already being passed in the module descriptor so we don't need to...
This commit is contained in:
@@ -283,7 +283,7 @@ def progress_summary(student, request, course, field_data_cache):
|
||||
|
||||
# TODO: We need the request to pass into here. If we could forego that, our arguments
|
||||
# would be simpler
|
||||
course_module = get_module(student, request, course.location, field_data_cache, course.id, depth=None)
|
||||
course_module = get_module_for_descriptor(student, request, course, field_data_cache, course.id)
|
||||
if not course_module:
|
||||
# This student must not have access to the course.
|
||||
return None
|
||||
|
||||
Reference in New Issue
Block a user