[Zope-Checkins] SVN: Zope/trunk/lib/python/Products/ZCatalog/ZCatalog.py - Removed unused imports

Chris Withers chris at simplistix.co.uk
Thu May 27 10:51:55 EDT 2004


Log message for revision 25048:
- Removed unused imports
- refactored slightly
- logged when object cannot be resolved on re-indexing of an index
- made manage_reindexIndex more efficient.


-=-
Modified: Zope/trunk/lib/python/Products/ZCatalog/ZCatalog.py
===================================================================
--- Zope/trunk/lib/python/Products/ZCatalog/ZCatalog.py	2004-05-27 14:29:22 UTC (rev 25047)
+++ Zope/trunk/lib/python/Products/ZCatalog/ZCatalog.py	2004-05-27 14:51:54 UTC (rev 25048)
@@ -36,10 +36,10 @@
      import PluggableIndexInterface
 from Products.PluginIndexes.TextIndex import Splitter
 import urllib, time, sys
-import string, logging
+import string,logging
 from IZCatalog import IZCatalog
-from zLOG import LOG, INFO, BLATHER, ERROR
 
+LOG = logging.getLogger('Zope.ZCatalog')
 
 manage_addZCatalogForm=DTMLFile('dtml/addZCatalog',globals())
 
@@ -461,17 +461,20 @@
 
 
     def reindexIndex(self, name, REQUEST):
-        paths = self._catalog.uids.keys()
-
-        for p in paths:
+        if isinstance(name, str):
+            name = (name,)
+        for p in self._catalog.uids.keys():
             obj = self.resolve_path(p)
             if not obj:
                 obj = self.resolve_url(p, REQUEST)
-            if obj is not None:
+            if obj is None:
+                LOG.error('reindexIndex could not resolve '
+                          'an object from the uid %r.' % (uid))
+            else:
                 # don't update metadata when only reindexing a single
                 # index via the UI
                 try:
-                    self.catalog_object(obj, p, idxs=[name],
+                    self.catalog_object(obj, p, idxs=name,
                                         update_metadata=0)
                 except TypeError:
                     # Fall back to Zope 2.6.2 interface. This is necessary for
@@ -495,8 +498,7 @@
         if isinstance(ids, str):
             ids = (ids,)
 
-        for name in ids:
-            self.reindexIndex(name, REQUEST)
+        self.reindexIndex(name, REQUEST, ids)
 
         if REQUEST and RESPONSE:
             RESPONSE.redirect(




More information about the Zope-Checkins mailing list