Changeset 6245 for main/waeup.sirp/trunk/src/waeup/sirp/browser/pages.py

Timestamp:

30 May 2011, 07:41:18 (13 years ago)

Author:

Henrik Bettermann

Message:

Use and raise DuplicationError? for Courses too.

File:

• main/waeup.sirp/trunk/src/waeup/sirp/browser/pages.py (modified) (1 diff)

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

@@ -1292 +1292 @@
         course = createObject(u'waeup.Course')
         self.applyData(course, **data)
-        message = self.context.courses.addCourse(course)
+        try:
+            self.context.courses.addCourse(course)
+        except DuplicationError, error:
+            # signals duplication error
+            entries = error.entries
+            for entry in entries:
+                # do not use error.msg but render a more detailed message instead
+                # and show links to all certs with same code
+                message = 'A course 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'@@addcourse'))
+            return
+        message = u'Course %s successfully created.' % (course.code)
         self.flash(message)
         self.redirect(self.url(self.context, u'@@manage')+'#tab-2')