[Erp5-report] r33809 jerome - in /erp5/trunk/products: ERP5/tests/ ERP5Catalog/tests/ ERP5O...

nobody at svn.erp5.org nobody at svn.erp5.org
Wed Mar 17 16:45:23 CET 2010


Author: jerome
Date: Wed Mar 17 16:45:20 2010
New Revision: 33809

URL: http://svn.erp5.org?rev=33809&view=rev
Log:
stop passing immediate_reindex=1 to newContent this API should not exist

Modified:
    erp5/trunk/products/ERP5/tests/testAccounting.py
    erp5/trunk/products/ERP5/tests/testInventoryAPI.py
    erp5/trunk/products/ERP5/tests/testInvoiceVAT.py
    erp5/trunk/products/ERP5/tests/testPredicate.py
    erp5/trunk/products/ERP5Catalog/tests/testERP5Catalog.py
    erp5/trunk/products/ERP5OOo/tests/testIngestion.py

Modified: erp5/trunk/products/ERP5/tests/testAccounting.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5/tests/testAccounting.py?rev=33809&r1=33808&r2=33809&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/tests/testAccounting.py [utf8] (original)
+++ erp5/trunk/products/ERP5/tests/testAccounting.py [utf8] Wed Mar 17 16:45:20 2010
@@ -2954,8 +2954,7 @@
         if not cat in path.objectIds():
           path = path.newContent(
             portal_type='Category',
-            id=cat,
-            immediate_reindex=1)
+            id=cat,)
         else:
           path = path[cat]
           

Modified: erp5/trunk/products/ERP5/tests/testInventoryAPI.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5/tests/testInventoryAPI.py?rev=33809&r1=33808&r2=33809&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/tests/testInventoryAPI.py [utf8] (original)
+++ erp5/trunk/products/ERP5/tests/testInventoryAPI.py [utf8] Wed Mar 17 16:45:20 2010
@@ -170,8 +170,7 @@
         if not cat in path.objectIds() :
           path = path.newContent(
                     portal_type='Category',
-                    id=cat,
-                    immediate_reindex=1 )
+                    id=cat,)
         else:
           path = path[cat]
     # check categories have been created
@@ -179,6 +178,8 @@
       self.assertNotEquals(None,
                 self.getCategoryTool().restrictedTraverse(cat_string),
                 cat_string)
+    transaction.commit()
+    self.tic()
                 
   def getNeededCategoryList(self):
     """return a list of categories that should be created."""
@@ -2319,8 +2320,7 @@
         group = unit_module.newContent(
                  id=base,
                  portal_type='Quantity Unit Conversion Group',
-                 quantity_unit="%s/%s" % (base, standard),
-                 immediate_reindex=1 )
+                 quantity_unit="%s/%s" % (base, standard),)
       if group.getValidationState() in ('draft', 'invalidated'):
         group.validate()
 
@@ -2331,8 +2331,7 @@
                         id=unit,
                         portal_type="Quantity Unit Conversion Definition",
                         quantity_unit="%s/%s" % (base, unit),
-                        quantity=amount,
-                        immediate_reindex=1)
+                        quantity=amount,)
         if definition.getValidationState() in ('draft', 'invalidated'):
           definition.validate()
 
@@ -2342,12 +2341,12 @@
     self.setUpUnitDefinition()
     self._safeTic()
 
+  @reindex
   def makeMovement(self, quantity, resource, *variation, **kw):
     m = self._makeMovement(quantity=quantity, resource_value=resource,
       source_value=self.node, destination_value=self.mirror_node, **kw)
     if variation:
       m.setVariationCategoryList(variation)
-      self._safeTic()
 
   def convertedSimulation(self, metric_type, **kw):
     return self.getSimulationTool().getInventory(
@@ -2500,16 +2499,14 @@
 
     base_unit = self.resource_bylot_overriding.newContent(
                   portal_type='Quantity Unit Conversion Group',
-                  quantity_unit='unit/unit',
-                  immediate_reindex=1)
+                  quantity_unit='unit/unit',)
     base_unit.validate()
 
 
     unit = base_unit.newContent(
             portal_type='Quantity Unit Conversion Definition',
             quantity_unit='unit/lot',
-            quantity=50,
-            immediate_reindex=1)
+            quantity=50,)
     unit.validate()
 
     self._safeTic()

Modified: erp5/trunk/products/ERP5/tests/testInvoiceVAT.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5/tests/testInvoiceVAT.py?rev=33809&r1=33808&r2=33809&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/tests/testInvoiceVAT.py [utf8] (original)
+++ erp5/trunk/products/ERP5/tests/testInvoiceVAT.py [utf8] Wed Mar 17 16:45:20 2010
@@ -113,8 +113,7 @@
         if not cat in path.objectIds() :
           path = path.newContent(
                     portal_type='Category',
-                    id=cat,
-                    immediate_reindex=1 )
+                    id=cat,)
         else:
           path = path[cat]
     # check categories have been created

Modified: erp5/trunk/products/ERP5/tests/testPredicate.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5/tests/testPredicate.py?rev=33809&r1=33808&r2=33809&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/tests/testPredicate.py [utf8] (original)
+++ erp5/trunk/products/ERP5/tests/testPredicate.py [utf8] Wed Mar 17 16:45:20 2010
@@ -84,8 +84,7 @@
         if not cat in path.objectIds() :
           path = path.newContent(
             portal_type = 'Category',
-            id = cat,
-            immediate_reindex = 1 )
+            id = cat,)
         else :
           path = path[cat]
 
@@ -94,6 +93,8 @@
       self.assertNotEquals(None,
                 self.getCategoryTool().restrictedTraverse(cat_string),
                 cat_string)
+    transaction.commit()
+    self.tic()
 
   def getNeededCategoryList(self):
     """return a list of categories that should be created."""

Modified: erp5/trunk/products/ERP5Catalog/tests/testERP5Catalog.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Catalog/tests/testERP5Catalog.py?rev=33809&r1=33808&r2=33809&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Catalog/tests/testERP5Catalog.py [utf8] (original)
+++ erp5/trunk/products/ERP5Catalog/tests/testERP5Catalog.py [utf8] Wed Mar 17 16:45:20 2010
@@ -195,8 +195,8 @@
     self.checkRelativeUrlNotInSQLPathList(path_list)
     # Now we will ask to immediatly reindex
     person = person_module.newContent(id='2',
-                                      portal_type='Person',
-                                      immediate_reindex=1)
+                                      portal_type='Person',)
+    person.immediateReindexObject()
     path_list = [person.getRelativeUrl()]
     self.checkRelativeUrlInSQLPathList(path_list)
     person.immediateReindexObject()
@@ -228,7 +228,8 @@
     # Now we will try the same thing as previous test and look at searchFolder
     folder_object_list = [x.getObject().getId() for x in person_module.searchFolder()]
     self.assertEquals([],folder_object_list)
-    person = person_module.newContent(id='4',portal_type='Person',immediate_reindex=1)
+    person = person_module.newContent(id='4',portal_type='Person',)
+    person.immediateReindexObject()
     folder_object_list = [x.getObject().getId() for x in person_module.searchFolder()]
     self.assertEquals(['4'],folder_object_list)
     person.immediateReindexObject()
@@ -300,7 +301,8 @@
     portal_catalog = self.getCatalogTool()
     portal_catalog.manage_catalogClear()
 
-    person = person_module.newContent(id='4',portal_type='Person',immediate_reindex=1)
+    person = person_module.newContent(id='4',portal_type='Person')
+    person.immediateReindexObject()
     folder_object_list = [x.getObject().getId() for x in person_module.searchFolder()]
     self.assertEquals(['4'],folder_object_list)
 
@@ -910,8 +912,8 @@
     person_module = self.getPersonModule()
     # Now we will ask to immediatly reindex
     person = person_module.newContent(id='2',
-                                      portal_type='Person',
-                                      immediate_reindex=1)
+                                      portal_type='Person',)
+    person.immediateReindexObject()
     path_list = [person.getRelativeUrl()]
     self.checkRelativeUrlInSQLPathList(path_list)
     # We will delete the connector

Modified: erp5/trunk/products/ERP5OOo/tests/testIngestion.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5OOo/tests/testIngestion.py?rev=33809&r1=33808&r2=33809&view=diff
==============================================================================
--- erp5/trunk/products/ERP5OOo/tests/testIngestion.py [utf8] (original)
+++ erp5/trunk/products/ERP5OOo/tests/testIngestion.py [utf8] Wed Mar 17 16:45:20 2010
@@ -206,7 +206,6 @@
               category_type = 'Category'
             base_path_obj.newContent( portal_type       = category_type
                                     , id                = category_id
-                                    , immediate_reindex = 1
                                     )
           base_path_obj = base_path_obj[category_id]
           is_base_category = False




More information about the Erp5-report mailing list