[Erp5-report] r36522 kazuhiko - /erp5/trunk/products/ERP5/Tool/SolverProcessTool.py
nobody at svn.erp5.org
nobody at svn.erp5.org
Tue Jun 22 18:04:04 CEST 2010
Author: kazuhiko
Date: Tue Jun 22 18:04:04 2010
New Revision: 36522
URL: http://svn.erp5.org?rev=36522&view=rev
Log:
add more comments on current issues.
Modified:
erp5/trunk/products/ERP5/Tool/SolverProcessTool.py
Modified: erp5/trunk/products/ERP5/Tool/SolverProcessTool.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5/Tool/SolverProcessTool.py?rev=36522&r1=36521&r2=36522&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/Tool/SolverProcessTool.py [utf8] (original)
+++ erp5/trunk/products/ERP5/Tool/SolverProcessTool.py [utf8] Tue Jun 22 18:04:04 2010
@@ -83,6 +83,8 @@ class SolverProcessTool(BaseTool):
or a list thereof
"""
# Do not create a new solver process if no divergence
+ # XXX (possible performance issue) Here it calls all divergence
+ # testers, but they should be called later.
if not self.isDivergent(delivery_or_movement=delivery_or_movement):
return None
@@ -93,10 +95,13 @@ class SolverProcessTool(BaseTool):
new_solver.buildSolverDecisionList(delivery_or_movement=delivery_or_movement,
temp_object=temp_object)
- # Append the solver process into the delivery's solver category
- delivery = delivery_or_movement.getRootDeliveryValue()
- solver_list = delivery.getSolverValueList()
- solver_list.append(new_solver)
- delivery.setSolverValueList(solver_list)
- return new_solver
-
+ if not temp_object:
+ # Append the solver process into the delivery's solver category
+ # XXX using delivery's solver category is not so good idea,
+ # because we might want to solve several deliveries with one
+ # solver process, several users want to solve one document etc.
+ delivery = delivery_or_movement.getRootDeliveryValue()
+ solver_list = delivery.getSolverValueList()
+ solver_list.append(new_solver)
+ delivery.setSolverValueList(solver_list)
+ return new_solver
More information about the Erp5-report
mailing list