Changeset 11453 for main/waeup.kwarapoly/trunk/src/waeup/kwarapoly/students

Timestamp:

27 Feb 2014, 10:57:32 (11 years ago)

Author:

Henrik Bettermann

Message:

Method for disabling student payments customized. We can now
disable school fee payment for non-pg students.

Location:

main/waeup.kwarapoly/trunk/src/waeup/kwarapoly/students

Files:

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

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

@@ -110 +110 @@
 
         # In KwaraPoly only returning students can create school fee payment
-        # without haveing paid accommodation fee
+        # without having paid accommodation fee
         IWorkflowState(self.student).setState('returning')
         configuration = createObject('waeup.SessionConfiguration')
@@ -464 +464 @@
 
         return
+
+    def test_payment_disabled(self):
+        IWorkflowState(self.student).setState('returning')
+        configuration = createObject('waeup.SessionConfiguration')
+        configuration.academic_session = 2005
+        self.app['configuration'].addSessionConfiguration(configuration)
+        self.browser.addHeader('Authorization', 'Basic mgr:mgrpw')
+        self.browser.open(self.payments_path)
+        self.browser.getLink("Add current session payment ticket").click()
+        self.browser.getControl(name="form.p_category").value = ['schoolfee']
+        self.browser.getControl("Create ticket").click()
+        self.assertMatches('...ticket created...',
+                           self.browser.contents)
+        self.app['configuration']['2005'].payment_disabled = ['sf_all']
+        self.browser.getLink("Add current session payment ticket").click()
+        self.browser.getControl(name="form.p_category").value = ['schoolfee']
+        self.browser.getControl("Create ticket").click()
+        self.assertMatches('...Payment temporarily disabled...',
+                           self.browser.contents)
+        self.app['configuration']['2005'].payment_disabled = ['sf_non_pg']
+        # Non-PG students can't pay ...
+        self.browser.getControl(name="form.p_category").value = ['schoolfee']
+        self.browser.getControl("Create ticket").click()
+        self.assertMatches('...Payment temporarily disabled...',
+                           self.browser.contents)
+        # ... but PG can pay.
+        self.certificate.study_mode = 'pg_ft'
+        self.browser.getControl(name="form.p_category").value = ['schoolfee']
+        self.browser.getControl("Create ticket").click()
+        self.assertMatches('...ticket created...',
+                           self.browser.contents)
+        return

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

@@ -152 +152 @@
             # Bed has not yet been booked but beds are available.
             return True
+        return False
+
+    def _isPaymentDisabled(self, p_session, category, student):
+        academic_session = self._getSessionConfiguration(p_session)
+        if category == 'schoolfee':
+            if 'sf_all' in academic_session.payment_disabled:
+                return True
+            if 'sf_non_pg' in academic_session.payment_disabled and \
+                not student.is_postgrad:
+                return True
         return False
 
@@ -258 +268 @@
         elif category == 'schoolfee':
             amount += academic_session.penalty_ug
+        if self._isPaymentDisabled(p_session, category, student):
+            return _('Payment temporarily disabled.'), None
         payment = createObject(u'waeup.StudentOnlinePayment')
         timestamp = ("%d" % int(time()*10000))[1:]