Changeset 6709

Timestamp:

10 Sep 2011, 00:29:02 (13 years ago)

Author:

uli

Message:

Add some onFailure implementation for password input. This one cares
for displaying invariant error messages properly (instead of a plain
'there were errors' message). Although this method is implemented in a
way usable also for other pages/view, please do not simply copy this
method over to other classes. Instead, put it somewhere in w.s.browser,
where it can be accessed from each interested form as it could be some
general form extension.

File:

• main/waeup.sirp/branches/uli-studentpw/src/waeup/sirp/students/browser.py (modified) (2 diffs)

main/waeup.sirp/branches/uli-studentpw/src/waeup/sirp/students/browser.py

@@ -571 +571 @@
 
     def update(self):
-        #datepicker.need() # Enable jQuery datepicker in date fields.
         super(StudentBaseEditFormPage, self).update()
         self.wf_info = IWorkflowInfo(self.context)
         return
 
-    @grok.action('Save')
+    def onFailure(self, action, data, errors):
+        new_status = []
+        other_errors = False
+        for error in errors:
+            msg = getattr(error, 'message', '')
+            if isinstance(msg, basestring) and msg != '':
+                new_status.append(msg)
+            else:
+                other_errors = True
+        if other_errors:
+            if new_status:
+                new_status.append('see below for further errors')
+            else:
+                new_status.append('See below for details.')
+        if new_status:
+            self.status = u'There were errors: %s' % ', '.join(new_status)
+        return
+
+    @grok.action('Save', failure=onFailure)
     def save(self, **data):
-        print "PW1: ", self.context.password
         form = self.request.form
         ob_class = self.__implemented__.__name__.replace('waeup.sirp.','')
@@ -588 +604 @@
         if fields_string:
             self.context.loggerInfo(ob_class, 'saved: % s' % fields_string)
-        print "PW2: ", self.context.password
         return