Changeset 8081 for main

Timestamp:

9 Apr 2012, 15:38:39 (13 years ago)

Author:

Henrik Bettermann

Message:

Wow, we didn't notify the students_catalog when a sudycourse was saved and changed. The browser tests didn't catch this malfunction because there we saved data manually and notified the catalog afterwards.

Location:

main/waeup.kofa/trunk/src/waeup/kofa

Files:

• applicants/interfaces.py (modified) (1 diff)
• students/browser.py (modified) (3 diffs)

main/waeup.kofa/trunk/src/waeup/kofa/applicants/interfaces.py

@@ -33 +33 @@
 from waeup.kofa.interfaces import MessageFactory as _
 from waeup.kofa.payments.interfaces import IOnlinePayment
-from waeup.kofa.schoolgrades import ResultEntryField
+#from waeup.kofa.schoolgrades import ResultEntryField
 from waeup.kofa.students.vocabularies import GenderSource
 from waeup.kofa.university.vocabularies import (

main/waeup.kofa/trunk/src/waeup/kofa/students/browser.py

@@ -186 +186 @@
             return
         if self.searchtype == 'current_session':
-            self.searchterm = int(self.searchterm)
+            try:
+                self.searchterm = int(self.searchterm)
+            except ValueError:
+                self.flash('Only year dates allowed (e.g. 2011).')
+                return
         self.hitlist = search(query=self.searchterm,
             searchtype=self.searchtype, view=self)
@@ -625 +629 @@
     def save(self, **data):
         msave(self, **data)
+        notify(grok.ObjectModifiedEvent(self.context.__parent__))
         return
 
@@ -1030 +1035 @@
                 return
         students_utils = getUtility(IStudentsUtils)
-        pay_details  = students_utils.getPaymentDetails(
-            p_category,student)
+        try:
+            pay_details  = students_utils.getPaymentDetails(
+                p_category,student)
+        except AttributeError:
+            self.flash(
+                _('Study course data are incomplete.'))
+            self.redirect(self.url(self.context))
+            return
         if pay_details['error']:
             self.flash(pay_details['error'])