Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #23664 from edx/saad/PROD-1404-logs-cleanup4

Browse Source 
[PROD-1404] - handle DeprecationWarning for usage_key.course_key for logs cleanup.
This commit is contained in:
Saad Yousaf
2020-04-15 15:58:48 +05:00
committed by GitHub
parent fe1ef21bcd db95b28b85
commit d33e2c9d74
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
lms/djangoapps/courseware/user_state_client.py
View File

@@ -247,7 +247,7 @@ class DjangoXBlockUserStateClient(XBlockUserStateClient):
try:
student_module, created = StudentModule.objects.get_or_create(
student=user,
course_id=usage_key.course_key,
course_id=usage_key.context_key,
module_state_key=usage_key,
defaults={
'state': json.dumps(state),
@@ -260,7 +260,7 @@ class DjangoXBlockUserStateClient(XBlockUserStateClient):
# process. This seems to happen frequently, and ignoring it is the
# best course of action for now
log.warning(u"set_many: IntegrityError for student {} - course_id {} - usage key {}".format(
user, repr(six.text_type(usage_key.course_key)), usage_key
user, repr(six.text_type(usage_key.context_key)), usage_key
))
return
@@ -283,7 +283,7 @@ class DjangoXBlockUserStateClient(XBlockUserStateClient):
# The UPDATE above failed. Log information - but ignore the error.
# See https://openedx.atlassian.net/browse/TNL-5365
log.warning(u"set_many: IntegrityError for student {} - course_id {} - usage key {}".format(
user, repr(six.text_type(usage_key.course_key)), usage_key
user, repr(six.text_type(usage_key.context_key)), usage_key
))
log.warning(u"set_many: All {} block keys: {}".format(
len(block_keys_to_state), list(block_keys_to_state.keys())