diff --git a/lms/djangoapps/verify_student/tests/test_views.py b/lms/djangoapps/verify_student/tests/test_views.py index fb94fefba7..eac86ff3bc 100644 --- a/lms/djangoapps/verify_student/tests/test_views.py +++ b/lms/djangoapps/verify_student/tests/test_views.py @@ -885,12 +885,7 @@ class TestCreateOrder(ModuleStoreTestCase): self.client.post(url, params) # Verify that the client's session contains the new donation amount - self.assertIn('donation_for_course', self.client.session) - self.assertIn(unicode(self.course.id), self.client.session['donation_for_course']) - - actual_amount = self.client.session['donation_for_course'][unicode(self.course.id)] - expected_amount = decimal.Decimal('1.23') - self.assertEqual(actual_amount, expected_amount) + self.assertNotIn('donation_for_course', self.client.session) def _verify_student(self): """ Simulate that the student's identity has already been verified. """ diff --git a/lms/djangoapps/verify_student/views.py b/lms/djangoapps/verify_student/views.py index 0f37cad32b..3a0a9d8ae2 100644 --- a/lms/djangoapps/verify_student/views.py +++ b/lms/djangoapps/verify_student/views.py @@ -636,17 +636,12 @@ def create_order(request): course_id = request.POST['course_id'] course_id = CourseKey.from_string(course_id) donation_for_course = request.session.get('donation_for_course', {}) - current_donation = donation_for_course.get(unicode(course_id), decimal.Decimal(0)) contribution = request.POST.get("contribution", donation_for_course.get(unicode(course_id), 0)) try: amount = decimal.Decimal(contribution).quantize(decimal.Decimal('.01'), rounding=decimal.ROUND_DOWN) except decimal.InvalidOperation: return HttpResponseBadRequest(_("Selected price is not valid number.")) - if amount != current_donation: - donation_for_course[unicode(course_id)] = amount - request.session['donation_for_course'] = donation_for_course - # prefer professional mode over verified_mode current_mode = CourseMode.verified_mode_for_course(course_id)