Changeset 3920 for waeup/branches

Timestamp:

7 Feb 2009, 00:03:27 (16 years ago)

Author:

uli

Message:

Be smart when looking for new ids.

File:

• waeup/branches/ulif-rewrite/src/waeup/university/facultycontainer.py (modified) (1 diff)

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

@@ -19 +19 @@
         """We need a unified id for each faculty, which acts as key.
         """
-        return self.next_id
+        return str(self.next_id)
     
     def addFaculty(self, faculty):
         if not IFaculty.providedBy(faculty):
             raise TypeError('FacultyContainers contain only IFaculty instances')
-        self[self.next_id] = faculty
-        self.next_id += 1
+        id = self.getId(faculty)
+        self[id] = faculty
+        while self.next_id in self.keys():
+            # Look for next unused int...
+            self.next_id += 1
+        return id
 
 # We register FacultyContainer as a utility. This way we can