Merge pull request #7285 from edx/aamir/ECOM-1177-price-display-bug

ECOM-1177: Removed the session variable on pay_and_verify view

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. """

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)