[Erp5-report] r32399 kazuhiko - /erp5/trunk/products/ERP5/Document/AcceptSolver.py

[nobody at svn.erp5.org]

Author: kazuhiko
Date: Wed Feb 10 10:34:23 2010
New Revision: 32399

URL: http://svn.erp5.org?rev=32399&view=rev
Log:
record only solved property.

Modified:
    erp5/trunk/products/ERP5/Document/AcceptSolver.py

Modified: erp5/trunk/products/ERP5/Document/AcceptSolver.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5/Document/AcceptSolver.py?rev=32399&r1=32398&r2=32399&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/Document/AcceptSolver.py [utf8] (original)
+++ erp5/trunk/products/ERP5/Document/AcceptSolver.py [utf8] Wed Feb 10 10:34:23 2010
@@ -84,19 +84,20 @@
             value_dict.update({'quantity':new_quantity})
           else:
             value_dict.update({solved_property:new_value})
-        self._solveRecursively(simulation_movement, value_dict)
+        self._solveRecursively(simulation_movement, value_dict,
+                               solved_property)
         simulation_movement.expand()
     # Finish solving
     self.succeed()
 
-  def _solveRecursively(self, simulation_movement, value_dict=None):
+  def _solveRecursively(self, simulation_movement, value_dict=None,
+                        property_id=None):
     """
       Update value of the current simulation movement, and update
       his parent movement.
     """
-    for property_id in value_dict.iterkeys():
-      if not simulation_movement.isPropertyRecorded(property_id):
-        simulation_movement.recordProperty(property_id)
+    if not simulation_movement.isPropertyRecorded(property_id):
+      simulation_movement.recordProperty(property_id)
     simulation_movement.edit(**value_dict)
 
     applied_rule = simulation_movement.getParentValue()
@@ -104,4 +105,5 @@
     if parent_movement.getPortalType() == 'Simulation Movement' and \
            not parent_movement.isFrozen():
       # backtrack to the parent movement while it is not frozen
-      self._solveRecursively(parent_movement, value_dict=value_dict)
+      self._solveRecursively(parent_movement, value_dict=value_dict,
+                             property_id=property_id)