Commit 3ba09221 by Eric Fischer Committed by cahrens

Add Logging for CohortMembership

parent db9299cc
......@@ -49,6 +49,8 @@ def move_to_verified_cohort(sender, instance, **kwargs): # pylint: disable=unus
'verified_cohort_name': verified_cohort_name,
'default_cohort_name': random_cohort.name
}
log.info("Queuing automatic cohorting for user '%s' in course '%s'", instance.user.username, course_key)
# Do the update with a 3-second delay in hopes that the CourseEnrollment transaction has been
# completed before the celery task runs. We want a reasonably short delay in case the learner
# immediately goes to the courseware.
......
......@@ -90,6 +90,8 @@ class CohortMembership(models.Model):
def save(self, *args, **kwargs):
self.full_clean(validate_unique=False)
log.info("Saving CohortMembership for '%s' (id=%s) in '%s'", self.user.username, self.user.id, self.course_id)
# Avoid infinite recursion if creating from get_or_create() call below.
# This block also allows middleware to use CohortMembership.get_or_create without worrying about outer_atomic
if 'force_insert' in kwargs and kwargs['force_insert'] is True:
......
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