From 14e6f3cc2b28dfd20e3cddcec63d9000b281a975 Mon Sep 17 00:00:00 2001 From: Calen Pennington Date: Wed, 3 Dec 2014 11:00:03 -0500 Subject: [PATCH] Don't query for the user every time in FieldDataCache, just use the user_id instead --- lms/djangoapps/courseware/model_data.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lms/djangoapps/courseware/model_data.py b/lms/djangoapps/courseware/model_data.py index b7bb361a57..23b56d1918 100644 --- a/lms/djangoapps/courseware/model_data.py +++ b/lms/djangoapps/courseware/model_data.py @@ -244,7 +244,7 @@ class FieldDataCache(object): assert(isinstance(key.block_scope_id, UsageKey)) field_object, _ = StudentModule.objects.get_or_create( course_id=self.course_id, - student=User.objects.get(id=key.user_id), + student_id=key.user_id, module_state_key=key.block_scope_id, defaults={ 'state': json.dumps({}), @@ -260,12 +260,12 @@ class FieldDataCache(object): field_object, _ = XModuleStudentPrefsField.objects.get_or_create( field_name=key.field_name, module_type=key.block_scope_id, - student=User.objects.get(id=key.user_id), + student_id=key.user_id, ) elif key.scope == Scope.user_info: field_object, _ = XModuleStudentInfoField.objects.get_or_create( field_name=key.field_name, - student=User.objects.get(id=key.user_id), + student_id=key.user_id, ) cache_key = self._cache_key_from_kvs_key(key)