[Erp5-report] r25395 - /erp5/trunk/products/ERP5/TargetSolver/SplitAndDefer.py

[nobody at svn.erp5.org]

Author: jerome
Date: Fri Jan 30 18:19:22 2009
New Revision: 25395

URL: http://svn.erp5.org?rev=25395&view=rev
Log:
don't use "in objectIds" which is O(n)

Modified:
    erp5/trunk/products/ERP5/TargetSolver/SplitAndDefer.py

Modified: erp5/trunk/products/ERP5/TargetSolver/SplitAndDefer.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5/TargetSolver/SplitAndDefer.py?rev=25395&r1=25394&r2=25395&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/TargetSolver/SplitAndDefer.py [utf8] (original)
+++ erp5/trunk/products/ERP5/TargetSolver/SplitAndDefer.py [utf8] Fri Jan 30 18:19:22 2009
@@ -54,7 +54,7 @@
       split_index = 0
       new_id = "%s_split_%s" % (simulation_movement.getId(), split_index)
       applied_rule = simulation_movement.getParentValue()
-      while new_id in applied_rule.objectIds():
+      while getattr(aq_base(applied_rule), new_id, None) is not None:
         split_index += 1
         new_id = "%s_split_%s" % (simulation_movement.getId(), split_index)
       # Adopt different dates for deferred movements