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

Timestamp:

7 Feb 2015, 10:25:56 (10 years ago)

Author:

Henrik Bettermann

Message:

Add new application category and type (ticket #966).

Implement hostel application fee payment (ticket #959). Uniben is still waiting for an agrrement with Interwsicth.

Location:

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

Files:

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

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

@@ -249 +249 @@
             name="form.p_category").value = ['tempmaint_1']
         self.browser.getControl("Create ticket").click()
+        self.assertMatches('...You have not yet paid the hostel application fee...',
+                           self.browser.contents)
+        self.browser.open(self.payments_path + '/addop')
+        self.browser.getControl(name="form.p_category").value = ['hostel_application']
+        self.browser.getControl("Create ticket").click()
+        self.assertMatches('...ticket created...',
+                           self.browser.contents)
+        ha_ticket = self.student['payments'].values()[2]
+        ha_ticket.approve()
+        self.browser.open(self.payments_path + '/addop')
+        self.browser.getControl(
+            name="form.p_category").value = ['tempmaint_1']
+        self.browser.getControl("Create ticket").click()
         self.assertMatches('...ticket created...',
                            self.browser.contents)
@@ -579 +592 @@
         self.assertEqual(error, None)
 
-        error, payment = utils.setPaymentDetails('hostel_maintenance',
+        #error, payment = utils.setPaymentDetails('hostel_maintenance',
+        #    self.student, None, None)
+        #self.assertEqual(payment.p_level, 100)
+        #self.assertEqual(payment.p_session, 2004)
+        #self.assertEqual(payment.amount_auth, 180.0)
+        #self.assertEqual(payment.p_item, u'')
+        #self.assertEqual(error, None)
+
+        #error, payment = utils.setPaymentDetails('bed_allocation',
+        #    self.student, None, None)
+        #self.assertEqual(payment.p_level, 100)
+        #self.assertEqual(payment.p_session, 2004)
+        #self.assertEqual(payment.amount_auth, 150.0)
+        #self.assertEqual(payment.p_item, u'')
+        #self.assertEqual(error, None)
+
+        error, payment = utils.setPaymentDetails('hostel_application',
             self.student, None, None)
         self.assertEqual(payment.p_level, 100)
         self.assertEqual(payment.p_session, 2004)
-        self.assertEqual(payment.amount_auth, 180.0)
+        self.assertEqual(payment.amount_auth, 1000.0)
         self.assertEqual(payment.p_item, u'')
         self.assertEqual(error, None)
 
-        error, payment = utils.setPaymentDetails('bed_allocation',
-            self.student, None, None)
-        self.assertEqual(payment.p_level, 100)
-        self.assertEqual(payment.p_session, 2004)
-        self.assertEqual(payment.amount_auth, 150.0)
-        self.assertEqual(payment.p_item, u'')
-        self.assertEqual(error, None)
+        payment.approve()
+        self.student['payments'][payment.p_id] = payment
 
         error, payment = utils.setPaymentDetails('tempmaint_1',

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

@@ -53 +53 @@
                 if ticket.p_state == 'paid' and \
                     ticket.p_category == 'schoolfee' and \
+                    ticket.p_session == session:
+                    return True
+        return False
+
+    def _hostelApplicationPaymentMade(self, student, session):
+        if len(student['payments']):
+            for ticket in student['payments'].values():
+                if ticket.p_state == 'paid' and \
+                    ticket.p_category == 'hostel_application' and \
                     ticket.p_session == session:
                     return True
@@ -93 +102 @@
         elif category == 'gown':
             amount = academic_session.gown_fee
-        elif category == 'bed_allocation':
-            amount = academic_session.booking_fee
-        elif category == 'hostel_maintenance':
-            amount = academic_session.maint_fee
-        elif category == 'tempmaint_1':
-            amount = 8150.0
-        elif category == 'tempmaint_2':
-            amount = 12650.0
-        elif category == 'tempmaint_3':
-            amount = 9650.0
+        #elif category == 'bed_allocation':
+        #    amount = academic_session.booking_fee
+        #elif category == 'hostel_maintenance':
+        #    amount = academic_session.maint_fee
+        elif category == 'hostel_application':
+            amount = 1000.0
+        elif category.startswith('tempmaint'):
+            if not self._hostelApplicationPaymentMade(
+                student, student.current_session):
+                return _(
+                    'You have not yet paid the hostel application fee.'), None
+            if category == 'tempmaint_1':
+                amount = 8150.0
+            elif category == 'tempmaint_2':
+                amount = 12650.0
+            elif category == 'tempmaint_3':
+                amount = 9650.0
         elif category == 'clearance':
             p_item = student.certcode
@@ -196 +212 @@
         elif category == 'schoolfee':
             amount += academic_session.penalty_ug
-        # XXX: Obsolete in 2013
         if category.startswith('tempmaint'):
             p_item = getUtility(IKofaUtils).PAYMENT_CATEGORIES[category]
             p_item = unicode(p_item)
             # Now we change the category because tempmaint payments
-            # will be obsolete in 2013
+            # will be obsolete when Uniben returns to Kofa bed allocation.
             category = 'hostel_maintenance'
         # Create ticket.