Changeset 7211 for main/waeup.sirp/trunk/src/waeup/sirp

Timestamp:

26 Nov 2011, 16:01:30 (13 years ago)

Author:

uli

Message:

Try to fix #65. Manually remove each course/certificate from a
container instead of using the (very fast) clear method. The latter
does not trigger IObjectRemoved events for courses and certificates.
Henrik, does it work better in UI now?
See ​https://trac.waeup.org/ticket/65

File:

• main/waeup.sirp/trunk/src/waeup/sirp/university/catalog.py (modified) (1 diff)

main/waeup.sirp/trunk/src/waeup/sirp/university/catalog.py

@@ -122 +122 @@
     """Clear courses and certificates when a department is killed.
     """
-    obj.courses.clear()
-    obj.certificates.clear()
+    # We cannot use the 'clear()' method of respective subcontainers
+    # (courses, certificates), because that would not trigger
+    # IObjectRemoved events.
+    for subobj_name in ['courses', 'certificates']:
+        key_list = list(getattr(obj, subobj_name, []).keys())
+        for key in key_list:
+            del getattr(obj, subobj_name)[key]
     return