Changeset 4486 for waeup/branches

Timestamp:

30 Jul 2009, 01:23:27 (15 years ago)

Author:

uli

Message:

• Be gentle when looking up a catalog.

• Fix nasty error in delCourseRef().

File:

• waeup/branches/ulif-rewrite/src/waeup/university/certificate.py (modified) (3 diffs)

waeup/branches/ulif-rewrite/src/waeup/university/certificate.py

@@ -5 +5 @@
 from zope.app.catalog.interfaces import ICatalog
 from zope.component import getUtility
-from zope.component.interfaces import IFactory
+from zope.component.interfaces import IFactory, ComponentLookupError
 from zope.interface import implementedBy
 from waeup.interfaces import (ICertificate, ICertificateContainer,
@@ -55 +55 @@
         """
         for key in self.keys():
-            if self[key].getCourseCode != code:
+            if self[key].getCourseCode() != code:
                 continue
             del self[key]
@@ -111 +111 @@
 
     # Find all certcourses that reference the given course...
-    cat = getUtility(ICatalog, name='certcourses_catalog')
+    try:
+        cat = getUtility(ICatalog, name='certcourses_catalog')
+    except ComponentLookupError:
+        # catalog not available. This might happen during tests.
+        return
+        
     results = cat.searchResults(course_code=(code, code))
     for certcourse in results: