Log message for revision 115036:
  I don't like lambda's to return static results
  

Changed:
  U   Zope/branches/andig-catalog-report/src/Products/ZCatalog/CatalogReport.py

-=-
Modified: 
Zope/branches/andig-catalog-report/src/Products/ZCatalog/CatalogReport.py
===================================================================
--- Zope/branches/andig-catalog-report/src/Products/ZCatalog/CatalogReport.py   
2010-07-24 16:37:58 UTC (rev 115035)
+++ Zope/branches/andig-catalog-report/src/Products/ZCatalog/CatalogReport.py   
2010-07-24 17:16:22 UTC (rev 115036)
@@ -14,6 +14,8 @@
 import time
 from thread import allocate_lock
 
+from Acquisition import aq_base
+from Acquisition import aq_parent
 from Products.PluginIndexes.interfaces import IUniqueValueIndex
 
 reportlock = allocate_lock()
@@ -120,11 +122,12 @@
         self.request = request
         self.threshold = threshold
 
-        # TODO: how to get an unique id?
-        getPhysicalPath = getattr(catalog.aq_parent,
-                                  'getPhysicalPath',
-                                  lambda: ['', 'DummyCatalog'])
-        self.cid = tuple(getPhysicalPath())
+        path = getattr(aq_parent(catalog), 'getPhysicalPath', None)
+        if path is None:
+            path = ('', 'NonPersistentCatalog')
+        else:
+            path = tuple(path())
+        self.cid = path
 
     def stop(self):
         super(CatalogReport, self).stop()

_______________________________________________
Zope-Checkins maillist  -  Zope-Checkins@zope.org
https://mail.zope.org/mailman/listinfo/zope-checkins

Reply via email to