From 22fcc12d245502be082fccf5699f0faff97e44c8 Mon Sep 17 00:00:00 2001 From: Christie Rice <8483753+crice100@users.noreply.github.com> Date: Fri, 29 Mar 2019 15:01:45 -0400 Subject: [PATCH] REVEM-262 Add more logging of ids --- lms/djangoapps/experiments/utils.py | 27 +++++++++++++++++++++------ 1 file changed, 21 insertions(+), 6 deletions(-) diff --git a/lms/djangoapps/experiments/utils.py b/lms/djangoapps/experiments/utils.py index b3ac41c412..949bc5701a 100644 --- a/lms/djangoapps/experiments/utils.py +++ b/lms/djangoapps/experiments/utils.py @@ -95,15 +95,30 @@ def check_and_get_upgrade_link_and_date(user, enrollment=None, course=None): if course is None: course = enrollment.course elif enrollment.course_id != course.id: - logger.warn(u'{} refers to a different course than {} which was supplied'.format( - enrollment, course - )) + logger.warn(u'{} refers to a different course than {} which was supplied. Enrollment course id={}, ' + u'repr={!r}, deprecated={}. Course id={}, repr={!r}, deprecated={}.' + .format(enrollment, + course, + enrollment.course_id, + enrollment.course_id, + enrollment.course_id.deprecated, + course.id, + course.id, + course.id.deprecated + ) + ) return (None, None) if enrollment.user_id != user.id: - logger.warn(u'{} refers to a different user than {} which was supplied'.format( - enrollment, user - )) + logger.warn(u'{} refers to a different user than {} which was supplied. Enrollment user id={}, repr={!r}. ' + u'User id={}, repr={!r}.'.format(enrollment, + user, + enrollment.user_id, + enrollment.user_id, + user.id, + user.id, + ) + ) return (None, None) if enrollment is None: