Commit a35a6b73 by Christina Roberts Committed by GitHub

Merge pull request #13116 from edx/christina/remove-extra-save

There is no need to call save after add/remove.
parents 194c5096 c195e7eb
...@@ -95,7 +95,6 @@ class CohortMembership(models.Model): ...@@ -95,7 +95,6 @@ class CohortMembership(models.Model):
if 'force_insert' in kwargs and kwargs['force_insert'] is True: if 'force_insert' in kwargs and kwargs['force_insert'] is True:
with transaction.atomic(): with transaction.atomic():
self.course_user_group.users.add(self.user) self.course_user_group.users.add(self.user)
self.course_user_group.save()
super(CohortMembership, self).save(*args, **kwargs) super(CohortMembership, self).save(*args, **kwargs)
return return
...@@ -127,11 +126,9 @@ class CohortMembership(models.Model): ...@@ -127,11 +126,9 @@ class CohortMembership(models.Model):
self.previous_cohort_name = saved_membership.course_user_group.name self.previous_cohort_name = saved_membership.course_user_group.name
self.previous_cohort_id = saved_membership.course_user_group.id self.previous_cohort_id = saved_membership.course_user_group.id
self.previous_cohort.users.remove(self.user) self.previous_cohort.users.remove(self.user)
self.previous_cohort.save()
saved_membership.course_user_group = self.course_user_group saved_membership.course_user_group = self.course_user_group
self.course_user_group.users.add(self.user) self.course_user_group.users.add(self.user)
self.course_user_group.save()
super(CohortMembership, saved_membership).save(update_fields=['course_user_group']) super(CohortMembership, saved_membership).save(update_fields=['course_user_group'])
......
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