Changeset 6322 for main/waeup.sirp/trunk/src/waeup/sirp/applicants/browser.py

Timestamp:

10 Jun 2011, 06:58:09 (13 years ago)

Author:

Henrik Bettermann

Message:

• Use user_data attribute to display more appropriate workflow messages.

• confirm_passport must not be an attribute of Applicant. Use confirm_passport only in form to allow submission.

• application_state must not be an attribute of Applicant.

File:

• main/waeup.sirp/trunk/src/waeup/sirp/applicants/browser.py (modified) (7 diffs)

main/waeup.sirp/trunk/src/waeup/sirp/applicants/browser.py

@@ -76 +76 @@
 TRANSITION_OBJECTS = create_workflow()
 
-TRANSITION_DICT = dict([
-    (transition_object.transition_id,transition_object.title)
-    for transition_object in TRANSITION_OBJECTS])
+#TRANSITION_DICT = dict([
+#    (transition_object.transition_id,transition_object.title)
+#    for transition_object in TRANSITION_OBJECTS])
 
 class ApplicantsRootPage(WAeUPPage):
@@ -419 +419 @@
         if state == INITIALIZED:
             IWorkflowInfo(self.context[pin]).fireTransition('start')
-            timestamp = datetime.now().strftime("%d/%m/%Y %H:%M")
-            transition_title = TRANSITION_DICT['start']
-            self.context[pin].messages += '<br />%s - %s' % (
-                timestamp,transition_title)
         self.redirect(self.url(self.context[pin], 'edit'))
         return
@@ -496 +492 @@
     form_fields['date_of_birth'].custom_widget = FriendlyDateWidget('le-year')
     grok.template('form_edit')
+    manage_applications = True
     pnav = 3
 
@@ -551 +548 @@
     form_fields['date_of_birth'].custom_widget = FriendlyDateWidget('le-year')
     grok.template('form_edit')
+    manage_applications = False
+
 
     def emitLockMessage(self):
@@ -566 +565 @@
 
     def dataNotComplete(self):
-        if self.context.confirm_passport is not True:
+        #import pdb; pdb.set_trace()
+        if not self.request.form.get('confirm_passport', False):
             return 'Passport confirmation box not ticked.'
         if len(self.errors) > 0:
@@ -572 +572 @@
         return False
 
-    @property
-    def getTransitions(self):
-        return None
-
     @grok.action('Save')
     def save(self, **data):
@@ -597 +593 @@
             return
         state = IWorkflowState(self.context).getState()
-        # Cannot happen but anyway ...
+        # This shouldn't happen, but the application officer
+        # might have forgotten to lock the form after changing the state
         if state != STARTED:
-            self.flash('This form cannot be submitted.')
+            self.flash('This form cannot be submitted. Wrong state!')
             return
         IWorkflowInfo(self.context).fireTransition('submit')