From 16d637c63f6c75d90fc3d63c6a59f642140fdda5 Mon Sep 17 00:00:00 2001 From: adeelehsan Date: Wed, 23 Oct 2019 17:56:05 +0500 Subject: [PATCH] Added logs to investigate the issue --- openedx/core/djangoapps/user_api/models.py | 8 ++++++++ openedx/core/djangoapps/user_api/preferences/api.py | 4 ++++ 2 files changed, 12 insertions(+) diff --git a/openedx/core/djangoapps/user_api/models.py b/openedx/core/djangoapps/user_api/models.py index 256bda5b6b..7ae8de7d81 100644 --- a/openedx/core/djangoapps/user_api/models.py +++ b/openedx/core/djangoapps/user_api/models.py @@ -3,6 +3,8 @@ Django ORM model specifications for the User API application """ from __future__ import absolute_import +import logging + from django.contrib.auth.models import User from django.core.validators import RegexValidator from django.db import models @@ -30,6 +32,8 @@ from student.models import ( ) from util.model_utils import emit_setting_changed_event, get_changed_fields_dict +log = logging.getLogger(__name__) + class RetirementStateError(Exception): pass @@ -96,7 +100,11 @@ def post_save_callback(sender, **kwargs): """ Event changes to user preferences. """ + user_preference = kwargs["instance"] + if user_preference.key == u'pref-lang': + log.info(u"Updated the language for the user:{username} to {new}".format( + username=user_preference.user.username, new=user_preference.value)) emit_setting_changed_event( user_preference.user, sender._meta.db_table, user_preference.key, user_preference._old_value, user_preference.value diff --git a/openedx/core/djangoapps/user_api/preferences/api.py b/openedx/core/djangoapps/user_api/preferences/api.py index e114aabe79..a35d1ce7ec 100644 --- a/openedx/core/djangoapps/user_api/preferences/api.py +++ b/openedx/core/djangoapps/user_api/preferences/api.py @@ -118,6 +118,9 @@ def update_user_preferences(requesting_user, update, user=None): # First validate each preference setting errors = {} serializers = {} + if u'pref-lang' in update.keys(): + log.info(u"Updating site language to {pref_lang} for the user: {username}" + .format(pref_lang=update['pref-lang'], username=user)) for preference_key in update.keys(): preference_value = update[preference_key] if preference_value is not None: @@ -147,6 +150,7 @@ def update_user_preferences(requesting_user, update, user=None): except Exception as error: raise _create_preference_update_error(preference_key, preference_value, error) else: + log.info(u"Deleting language preference for the user: {username}".format(username=user)) delete_user_preference(requesting_user, preference_key)