Commit 73af5d01 by Julia Hansbrough

Changed create_enrollment to create_or_update_enrollment

parent de8b378d
...@@ -713,7 +713,7 @@ class CourseEnrollment(models.Model): ...@@ -713,7 +713,7 @@ class CourseEnrollment(models.Model):
).format(self.user, self.course_id, self.created, self.is_active) ).format(self.user, self.course_id, self.created, self.is_active)
@classmethod @classmethod
def create_enrollment(cls, user, course_id, mode="honor", is_active=False): def create_or_update_enrollment(cls, user, course_id, mode="honor", is_active=False):
""" """
Create an enrollment for a user in a class. By default *this enrollment Create an enrollment for a user in a class. By default *this enrollment
is not active*. This is useful for when an enrollment needs to go is not active*. This is useful for when an enrollment needs to go
...@@ -818,7 +818,7 @@ class CourseEnrollment(models.Model): ...@@ -818,7 +818,7 @@ class CourseEnrollment(models.Model):
It is expected that this method is called from a method which has already It is expected that this method is called from a method which has already
verified the user authentication and access. verified the user authentication and access.
""" """
return cls.create_enrollment(user, course_id, mode, is_active=True) return cls.create_or_update_enrollment(user, course_id, mode, is_active=True)
@classmethod @classmethod
def enroll_by_email(cls, email, course_id, mode="honor", ignore_errors=True): def enroll_by_email(cls, email, course_id, mode="honor", ignore_errors=True):
......
...@@ -443,7 +443,7 @@ class EnrollInCourseTest(TestCase): ...@@ -443,7 +443,7 @@ class EnrollInCourseTest(TestCase):
# Creating an enrollment doesn't actually enroll a student # Creating an enrollment doesn't actually enroll a student
# (calling CourseEnrollment.enroll() would have) # (calling CourseEnrollment.enroll() would have)
enrollment = CourseEnrollment.create_enrollment(user, course_id) enrollment = CourseEnrollment.create_or_update_enrollment(user, course_id)
self.assertFalse(CourseEnrollment.is_enrolled(user, course_id)) self.assertFalse(CourseEnrollment.is_enrolled(user, course_id))
self.assert_no_events_were_emitted() self.assert_no_events_were_emitted()
......
...@@ -467,7 +467,7 @@ class CertificateItem(OrderItem): ...@@ -467,7 +467,7 @@ class CertificateItem(OrderItem):
try: try:
course_enrollment = CourseEnrollment.objects.get(user=order.user, course_id=course_id) course_enrollment = CourseEnrollment.objects.get(user=order.user, course_id=course_id)
except ObjectDoesNotExist: except ObjectDoesNotExist:
course_enrollment = CourseEnrollment.create_enrollment(order.user, course_id, mode=mode) course_enrollment = CourseEnrollment.create_or_update_enrollment(order.user, course_id, mode=mode)
# do some validation on the enrollment mode # do some validation on the enrollment mode
valid_modes = CourseMode.modes_for_course_dict(course_id) valid_modes = CourseMode.modes_for_course_dict(course_id)
......
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