Changeset 11644

Timestamp:

13 May 2014, 21:34:37 (11 years ago)

Author:

Henrik Bettermann

Message:

Use init_update and samePaymentMade methods

Location:

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

Files:

• interswitch/browser.py (modified) (2 diffs)
• students/utils.py (modified) (1 diff)

main/waeup.uniben/trunk/src/waeup/uniben/interswitch/browser.py

@@ -57 +57 @@
 
     def update(self):
-        student, certificate, xmldict = super(
-            CustomInterswitchPageStudent, self).update()
+        error = self.init_update()
+        if error:
+            self.flash(error, type='danger')
+            self.redirect(self.url(self.context, '@@index'))
+            return
+        student = self.student
+        xmldict = self.xmldict
         # Provider data
         xmldict['detail_ref'] = self.context.p_id
@@ -146 +151 @@
 
     def update(self):
-        super(CustomInterswitchPageApplicant, self).update()
+        error = self.init_update()
+        if error:
+            self.flash(error, type='danger')
+            self.redirect(self.url(self.context, '@@index'))
+            return
         xmldict = {}
         provider_amt = 400.0

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

@@ -203 +203 @@
             category = 'hostel_maintenance'
         # Create ticket.
-        for key in student['payments'].keys():
-            ticket = student['payments'][key]
-            if ticket.p_state == 'paid' and\
-               ticket.p_category == category and \
-               ticket.p_item == p_item and \
-               ticket.p_session == p_session:
-                  return _('This type of payment has already been made.'), None
+        if self.samePaymentMade(student, category, p_item, p_session):
+            return _('This type of payment has already been made.'), None
         if self._isPaymentDisabled(p_session, category, student):
             return _('Payment temporarily disabled.'), None