Commit ca8164a8 by Chris Dodge Committed by Jonathan Piacenti

cdodge/fix-crash-on-course-not-found: add null guard

Add logging message
parent fbba5ecc
...@@ -613,13 +613,19 @@ class UsersCoursesList(SecureAPIView): ...@@ -613,13 +613,19 @@ class UsersCoursesList(SecureAPIView):
response_data = [] response_data = []
for enrollment in enrollments: for enrollment in enrollments:
descriptor = store.get_course(enrollment.course_id) descriptor = store.get_course(enrollment.course_id)
course_data = { # NOTE: It is possible that a course has been hard deleted from the courseware
"id": enrollment.course_id, # database, but the enrollment row in the SQL database still exists
"uri": '{}/{}'.format(base_uri, enrollment.course_id), if descriptor:
"is_active": enrollment.is_active, course_data = {
"name": descriptor.display_name "id": enrollment.course_id,
} "uri": '{}/{}'.format(base_uri, enrollment.course_id),
response_data.append(course_data) "is_active": enrollment.is_active,
"name": descriptor.display_name
}
response_data.append(course_data)
else:
log.warning("User {0} enrolled in course_id {1}, but course could not be found.".format(user_id, enrollment.course_id))
return Response(response_data, status=status.HTTP_200_OK) return Response(response_data, status=status.HTTP_200_OK)
......
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