Changeset 4755

Timestamp:

9 Jan 2010, 11:14:06 (15 years ago)

Author:

uli

Message:

• Let certificate code field be editable on certificate add-forms only.

• Let certificate update() methods handle workflow problems.

File:

• waeup/branches/ulif-layout/src/waeup/browser/pages.py (modified) (6 diffs)

waeup/branches/ulif-layout/src/waeup/browser/pages.py

@@ -9 +9 @@
 from hurry.query import Eq
 from hurry.query.query import Query, Text
+from hurry.workflow.interfaces import NoTransitionAvailableError
 from waeup.browser import (WAeUPPage, WAeUPEditFormPage, WAeUPAddFormPage,
                            WAeUPDisplayFormPage)
@@ -19 +20 @@
 from waeup.widgets.interfaces import ITableProvider
 from waeup.utils.helpers import getName
+from zope import schema
 from zope.app.security.interfaces import (IAuthentication,
                                           IUnauthenticatedPrincipal,
@@ -585 +587 @@
     title = u'Add certificate'
     pnav = 1
-    
-    # We need a deepcopy here, because otherwise also all other
-    # ICertificate forms would be affected.
-    form_fields = copy.deepcopy(grok.AutoFields(ICertificate))
-    form_fields.get('code').field.readonly = False
-
+    form_fields = grok.AutoFields(ICertificate)
+    
     @grok.action('Add certificate')
     def addCertificate(self, **data):
         certificate = createObject(u'waeup.Certificate')
-        self.applyData(certificate, **data)
+        try:
+            self.applyData(certificate, **data)
+        except NoTransitionAvailableError:
+            self.status = Invalid('Review state cannot be changed to '
+                                  'requested state: no such transition '
+                                  'available in workflow.')
+            return
         try:
             self.context.certificates.addCertificate(certificate)
@@ -604 +608 @@
 
     @grok.action('Cancel')
-    def cancel(self, **data):
+    def cancel(self): #, **data):
         self.redirect(self.url(self.context))
         return
@@ -717 +721 @@
     grok.name('manage')
     grok.require('waeup.manageUniversity')
-    form_fields = grok.AutoFields(ICertificate)
+    # We omit 'code' field from original schema and set prepend a new
+    # one with 'readonly' set to True.
+    form_fields = grok.Fields(
+        code = schema.TextLine(
+            title = u'Code', readonly=True
+            )
+        ) + grok.AutoFields(ICertificate).omit('code')
     pnav = 1
 
@@ -727 +737 @@
     @grok.action('Save')
     def save(self, **data):
-        self.applyData(self.context, **data)
+        try:
+            self.applyData(self.context, **data)
+        except NoTransitionAvailableError:
+            self.status = Invalid('Review state cannot be changed to '
+                                  'requested state: no such transition '
+                                  'available in workflow.')
         return
     
     @grok.action('Save and return')
     def saveAndReturn(self, **data):
-        self.applyData(self.context, **data)
+        try:
+            self.applyData(self.context, **data)
+        except NoTransitionAvailableError:
+            self.status = Invalid('Review state cannot be changed to '
+                                  'requested state: no such transition '
+                                  'available in workflow.')
+            return
         self.redirect(self.url(self.context))
         return