Changeset 14893 for main/waeup.uniben/trunk/src/waeup/uniben/students

Timestamp:

13 Nov 2017, 09:04:23 (7 years ago)

Author:

Henrik Bettermann

Message:

Use custom_float_1 for old returning student discount.

Location:

main/waeup.uniben/trunk/src/waeup/uniben/students

Files:

• tests/test_browser.py (modified) (2 diffs)
• utils.py (modified) (1 diff)

main/waeup.uniben/trunk/src/waeup/uniben/students/tests/test_browser.py

@@ -555 +555 @@
         #self.student['payments']['anykey'] = payment
 
-
         error, payment = utils.setPaymentDetails('schoolfee',
             self.student, None, None)
@@ -564 +563 @@
         self.assertEqual(error, None)
 
-        # Old returning students pay less
+        # Old returning students may pay less.
         self.certificate.school_fee_2 = 50000.0
-        self.certificate.custom_textline_1 = 'sciEnce'
+        self.certificate.custom_float_1 = 30000.0
         error, payment = utils.setPaymentDetails(
             'schoolfee', self.student, None, None)
-        # minus 30000
         self.assertEqual(payment.amount_auth, 20088.0)
-        self.certificate.custom_textline_1 = 'non-Science'
-        error, payment = utils.setPaymentDetails(
-            'schoolfee', self.student, None, None)
-        # minus 28000
-        self.assertEqual(payment.amount_auth, 22088.0)
 
         # Staff members pay less.
-        self.certificate.custom_textline_1 = ''
+        self.certificate.custom_float_1 = None
         self.student.is_staff = True
         error, payment = utils.setPaymentDetails('schoolfee',

main/waeup.uniben/trunk/src/waeup/uniben/students/utils.py

@@ -351 +351 @@
                     else:
                         amount = getattr(certificate, 'school_fee_2', 0.0)
-                        # Old returning students get a discount of
-                        # N30000 and N28000 respectively.
+                        # Old returning students might get a discount.
                         if student.entry_session < 2017 \
-                            and certificate.custom_textline_1:
-                            if certificate.custom_textline_1.lower() == 'science':
-                                amount -= 30000
-                            elif certificate.custom_textline_1.lower() == 'non-science':
-                                amount -= 28000
+                            and certificate.custom_float_1:
+                            amount -= certificate.custom_float_1
             # Give 50% school fee discount to staff members.
             if student.is_staff: