Commit 154a4410 by Calen Pennington

Only use .pk for queries, and not on inserts

parent cbed6628
...@@ -240,7 +240,7 @@ class ModelDataCache(object): ...@@ -240,7 +240,7 @@ class ModelDataCache(object):
if key.scope == Scope.student_state: if key.scope == Scope.student_state:
field_object, _ = StudentModule.objects.get_or_create( field_object, _ = StudentModule.objects.get_or_create(
course_id=self.course_id, course_id=self.course_id,
student=self.user.pk, student=self.user,
module_type=key.block_scope_id.category, module_type=key.block_scope_id.category,
module_state_key=key.block_scope_id.url(), module_state_key=key.block_scope_id.url(),
defaults={'state': json.dumps({})}, defaults={'state': json.dumps({})},
...@@ -259,12 +259,12 @@ class ModelDataCache(object): ...@@ -259,12 +259,12 @@ class ModelDataCache(object):
field_object, _ = XModuleStudentPrefsField.objects.get_or_create( field_object, _ = XModuleStudentPrefsField.objects.get_or_create(
field_name=key.field_name, field_name=key.field_name,
module_type=key.block_scope_id, module_type=key.block_scope_id,
student=self.user.pk, student=self.user,
) )
elif key.scope == Scope.student_info: elif key.scope == Scope.student_info:
field_object, _ = XModuleStudentInfoField.objects.get_or_create( field_object, _ = XModuleStudentInfoField.objects.get_or_create(
field_name=key.field_name, field_name=key.field_name,
student=self.user.pk, student=self.user,
) )
cache_key = self._cache_key_from_kvs_key(key) cache_key = self._cache_key_from_kvs_key(key)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment