Changeset 6223

Timestamp:

29 May 2011, 13:47:39 (13 years ago)

Author:

uli

Message:

Get closer to the roots of the problem and override setitem to be
sure we do not store duplicate certificate entries.

File:

• main/waeup.sirp/trunk/src/waeup/sirp/university/certificatecontainer.py (modified) (2 diffs)

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

@@ -15 +15 @@
     grok.require('waeup.manageUniversity')
 
-    def addCertificate(self, certificate):
+    def __setitem__(self, name, certificate):
         """XXX: docstring missing.
         """
@@ -31 +31 @@
             # No catalog, then this addition won't do harm to anything.
             pass
-        self[certificate.code] = certificate
+        super(CertificateContainer, self).__setitem__(name, certificate)
+
+    def addCertificate(self, certificate):
+        self[getattr(certificate, 'code', None)] = certificate
 
     def clear(self):