Changeset 6980 for main/waeup.sirp/trunk/src/waeup/sirp/university/certificate.py

Timestamp:

1 Nov 2011, 08:34:05 (13 years ago)

Author:

Henrik Bettermann

Message:

We have to create a real (deep) list copy of self.keys() when deleting items of self. Otherwise the deletion of some of the items might fail.

I have no idea how to write regression tests with justifiable expenditure.

File:

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

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

@@ -41 +41 @@
         """Delete a course referrer denoted by its code.
         """
-        for key in self.keys():
+        keys = list(self.keys()) # create list copy
+        for key in keys:
             if self[key].getCourseCode() != code:
                 continue