From 52fba9840a732698f56c11e627bf93c81cbf4195 Mon Sep 17 00:00:00 2001 From: Awais Jibran Date: Wed, 4 Apr 2018 16:44:12 +0500 Subject: [PATCH] Revert "Use get or create rather than create" --- openedx/core/djangoapps/course_groups/cohorts.py | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/openedx/core/djangoapps/course_groups/cohorts.py b/openedx/core/djangoapps/course_groups/cohorts.py index d058f74380..37fbcdf9e9 100644 --- a/openedx/core/djangoapps/course_groups/cohorts.py +++ b/openedx/core/djangoapps/course_groups/cohorts.py @@ -258,17 +258,10 @@ def get_cohort(user, course_key, assign=True, use_cached=False): else: course_user_group = get_random_cohort(course_key) - membership, created = CohortMembership.objects.get_or_create( + membership = CohortMembership.objects.create( user=user, course_user_group=course_user_group, ) - if not created: - # EDUCATOR-2549: Adding temporary log to test if we hit a situation where user already has - # a member ship group. - log.info( - "COHORT_MEMBERSHIP_FOUND: Membership found %s for course %s and user %s", - membership.course_user_group, course_key, user.id - ) return cache.setdefault(cache_key, membership.course_user_group) except IntegrityError as integrity_error: