[ZCM] [ZC] 1764/ 2 Comment "reindex_all does not reindex but index"

[Collector: Zope Bugs, Features, and Patches ...]

Issue #1764 Update (Comment) "reindex_all does not reindex but index"
 Status Pending, Zope/bug+solution medium
To followup, visit:
  http://www.zope.org/Collectors/Zope/1764

==============================================================
= Comment - Entry #2 by ajung on Apr 29, 2005 6:04 am

An index implementation should in general keep track of changes of already indexed objects.
________________________________________
= Request - Entry #1 by jhb on Apr 26, 2005 4:35 pm

The reindex_all method in lib/python/Products/ZCatalog/CatalogPathAwareness.py recursivly goes through all contained objects, but contrary to the name of the of method it doesn't reindex them, but just indexes them (reindex_object however unindexes and indexes and object), which causes at least problems on my implementation of a catalog.

The solution would be:

116,117c116,117
<         if hasattr(aq_base(obj), 'index_object'):
<             obj.index_object()
---
>         if hasattr(aq_base(obj), 'reindex_object'):
>             obj.reindex_object()


==============================================================