Changeset 9828 for main/waeup.kofa/trunk/src/waeup/kofa/university/course.py

Timestamp:

5 Jan 2013, 17:26:01 (12 years ago)

Author:

uli

Message:

Simplify last fix a bit and add regression test. Welcome home, by the way.

File:

• main/waeup.kofa/trunk/src/waeup/kofa/university/course.py (modified) (1 diff)

main/waeup.kofa/trunk/src/waeup/kofa/university/course.py

@@ -83 +83 @@
 
     results = cat.searchResults(course_code=(code, code))
-    # Only one certcourse per certificate is needed
-    reduced_results = []
-    for certcourse in results:
-        cert = certcourse.__parent__
-        if cert in  [i.__parent__ for i in reduced_results]:
-            continue
-        reduced_results.append(certcourse)
-    # Now we can apply delCertCourses
-    for certcourse in reduced_results:
-        cert = certcourse.__parent__
+
+    # remove each found referrer (certs might refer to same course multiple
+    # times)
+    certs = [x.__parent__ for x in results]
+    unique_certs = list(set(certs))
+    for cert in unique_certs:
         cert.delCertCourses(code)
     return