[CalendarServer-changes] [5433] CalendarServer/branches/users/sagen/deprovision/twistedcaldav/ directory/aggregate.py

[source_changes at macosforge.org]

Revision: 5433
          http://trac.macosforge.org/projects/calendarserver/changeset/5433
Author:   sagen at apple.com
Date:     2010-04-01 10:48:04 -0700 (Thu, 01 Apr 2010)
Log Message:
-----------
Comments

Modified Paths:
--------------
    CalendarServer/branches/users/sagen/deprovision/twistedcaldav/directory/aggregate.py

Modified: CalendarServer/branches/users/sagen/deprovision/twistedcaldav/directory/aggregate.py
===================================================================
--- CalendarServer/branches/users/sagen/deprovision/twistedcaldav/directory/aggregate.py	2010-03-31 22:56:07 UTC (rev 5432)
+++ CalendarServer/branches/users/sagen/deprovision/twistedcaldav/directory/aggregate.py	2010-04-01 17:48:04 UTC (rev 5433)
@@ -69,6 +69,10 @@
         self.realmName = realmName
         self._recordTypes = recordTypes
 
+        # FIXME: This is a temporary workaround until new data store is in
+        # place.  During the purging of deprovisioned users' data, we need
+        # to be able to look up records by guid and shortName.  The purge
+        # tool sticks temporary fake records in here.
         self._tmpRecords = {
             "guids" : { },
             "shortNames" : { },
@@ -116,15 +120,25 @@
             return records
 
     def recordWithShortName(self, recordType, shortName):
+
+        # FIXME: These temporary records shouldn't be needed when we move
+        # to the new data store API.  They're currently needed when purging
+        # deprovisioned users' data.
         record = self._tmpRecords["shortNames"].get(shortName, None)
         if record:
             return record
+
         return self._query("recordWithShortName", recordType, shortName)
 
     def recordWithUID(self, uid):
+
+        # FIXME: These temporary records shouldn't be needed when we move
+        # to the new data store API.  They're currently needed when purging
+        # deprovisioned users' data.
         record = self._tmpRecords["guids"].get(uid, None)
         if record:
             return record
+
         return self._queryAll("recordWithUID", uid)
 
     def recordWithAuthID(self, authID):
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.macosforge.org/pipermail/calendarserver-changes/attachments/20100401/7a41c629/attachment.html>