Changeset 6228

Timestamp:

29 May 2011, 23:25:13 (13 years ago)

Author:

uli

Message:

Handle DuplicationError? from addCertificate() properly.

File:

• main/waeup.sirp/trunk/src/waeup/sirp/browser/pages.py (modified) (4 diffs)

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

@@ -16 +16 @@
 from zope.authentication.interfaces import (
     IAuthentication, IUnauthenticatedPrincipal, ILogout)
+from zope.catalog.interfaces import ICatalog
 from zope.securitypolicy.interfaces import (
     IPrincipalRoleManager, IPrincipalRoleMap)
@@ -21 +22 @@
     getUtility, createObject,getAllUtilitiesRegisteredFor)
 from zope.component.interfaces import Invalid
-from zope.exceptions import DuplicationError
 from zope.session.interfaces import ISession
 from zope.event import notify
@@ -36 +36 @@
     IWAeUPObject, IUserContainer, IUserAccount, IDataCenter,
     IWAeUPXMLImporter, IWAeUPXMLExporter, IBatchProcessor,
-    ILocalRolesAssignable)
+    ILocalRolesAssignable, DuplicationError)
 from waeup.sirp.permissions import get_users_with_local_roles, getRoles
 from waeup.sirp.university.catalog import search
@@ -1330 +1330 @@
         certificate = createObject(u'waeup.Certificate')
         self.applyData(certificate, **data)
-        message = self.context.certificates.addCertificate(certificate)
+        try:
+            self.context.certificates.addCertificate(certificate)
+        except DuplicationError, error:
+            # signals duplication error
+            entries = error.entries
+            for entry in entries:
+                # show links to all certs with same code
+                message = 'A certificate with same code already exists: '
+                message += '<a href="%s">%s</a>' % (
+                    self.url(entry), getattr(entry, '__name__', u'Unnamed'))
+                self.flash(message)
+            self.redirect(self.url(self.context, u'@@addcertificate'))
+            return
+        message = u'Certificate %s successfully created.' % (certificate.code)
         self.flash(message)
         self.redirect(self.url(self.context, u'@@manage')+'#tab-3')
+        return
 
     @grok.action('Cancel', validator=NullValidator)