[Checkins] SVN: Products.CMFCore/trunk/Products/CMFCore/ - use indexObject instead of reindexObject (no need to call notifyModified after 'moved' event)

Yvo Schubbe y.2007- at wcm-solutions.de
Wed Dec 26 11:27:11 EST 2007


Log message for revision 82457:
  - use indexObject instead of reindexObject (no need to call notifyModified after 'moved' event)

Changed:
  U   Products.CMFCore/trunk/Products/CMFCore/CMFCatalogAware.py
  U   Products.CMFCore/trunk/Products/CMFCore/tests/test_CMFCatalogAware.py

-=-
Modified: Products.CMFCore/trunk/Products/CMFCore/CMFCatalogAware.py
===================================================================
--- Products.CMFCore/trunk/Products/CMFCore/CMFCatalogAware.py	2007-12-26 16:18:41 UTC (rev 82456)
+++ Products.CMFCore/trunk/Products/CMFCore/CMFCatalogAware.py	2007-12-26 16:27:10 UTC (rev 82457)
@@ -237,7 +237,7 @@
 
     elif IObjectMovedEvent.providedBy(event):
         if event.newParent is not None:
-            ob.reindexObject()
+            ob.indexObject()
 
     elif IObjectWillBeMovedEvent.providedBy(event):
         if event.oldParent is not None:

Modified: Products.CMFCore/trunk/Products/CMFCore/tests/test_CMFCatalogAware.py
===================================================================
--- Products.CMFCore/trunk/Products/CMFCore/tests/test_CMFCatalogAware.py	2007-12-26 16:18:41 UTC (rev 82456)
+++ Products.CMFCore/trunk/Products/CMFCore/tests/test_CMFCatalogAware.py	2007-12-26 16:27:10 UTC (rev 82457)
@@ -315,7 +315,7 @@
         folder2.manage_pasteObjects(cookie)
 
         self.assertEquals(cat.log, ["unindex /site/folder1/bar",
-                                    "reindex /site/folder2/bar []"])
+                                    "index /site/folder2/bar"])
 
     def test_object_reindexed_after_moving(self):
 
@@ -331,7 +331,7 @@
 
         site.manage_renameObject(id='bar', new_id='baz')
         self.assertEquals(cat.log, ["unindex /site/bar",
-                                    "reindex /site/baz []"])
+                                    "index /site/baz"])
 
 
 def test_suite():



More information about the Checkins mailing list