From 74904339dd9344dda596075dee45462e4e2a8f4f Mon Sep 17 00:00:00 2001 From: Feanil Patel Date: Tue, 20 Aug 2019 10:59:42 -0400 Subject: [PATCH] Go from a courselocator to a unicode string. We don't need the ignore since courselocators sholud be managing unicode already. --- lms/djangoapps/course_goals/handlers.py | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/lms/djangoapps/course_goals/handlers.py b/lms/djangoapps/course_goals/handlers.py index d57321aa4d..ef4afb8c9f 100644 --- a/lms/djangoapps/course_goals/handlers.py +++ b/lms/djangoapps/course_goals/handlers.py @@ -21,9 +21,7 @@ def update_course_goal_on_enroll_change(sender, instance, **kwargs): # pylint: 1) Set the course goal to 'certify' when the user enrolls as a verified user. 2) Remove the course goal when the user's enrollment is no longer active. """ - course_id = instance.course_id - if isinstance(course_id, six.binary_type): - course_id = course_id.decode('utf8', 'ignore') + course_id = six.text_type(instance.course_id) if not instance.is_active: remove_course_goal(instance.user, course_id) elif instance.mode == CourseMode.VERIFIED: