[Erp5-report] r38224 yo - /erp5/trunk/products/ERP5/Document/OrderBuilder.py
nobody at svn.erp5.org
nobody at svn.erp5.org
Thu Sep 9 13:12:43 CEST 2010
Author: yo
Date: Thu Sep 9 13:12:41 2010
New Revision: 38224
URL: http://svn.erp5.org?rev=38224&view=rev
Log:
Add comments for things to do.
Modified:
erp5/trunk/products/ERP5/Document/OrderBuilder.py
Modified: erp5/trunk/products/ERP5/Document/OrderBuilder.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5/Document/OrderBuilder.py?rev=38224&r1=38223&r2=38224&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/Document/OrderBuilder.py [utf8] (original)
+++ erp5/trunk/products/ERP5/Document/OrderBuilder.py [utf8] Thu Sep 9 13:12:41 2010
@@ -300,6 +300,14 @@ class OrderBuilder(XMLObject, Amount, Pr
def _findUpdatableObject(self, instance_list, movement_group_node_list,
divergence_list):
+ # FIXME this code may generate inconsistent results, because
+ # MovementGroupNode.test can return anything else but the
+ # property dict. So it would be better to use the test method
+ # in all cases. It is, however, not so easy to do, because
+ # Movement Group classes might not be prepared for receiving
+ # None instead of a real document object. So it would be safer
+ # to pass a temp object, but generating a temp object is not
+ # very fast.
instance = None
property_dict = {}
if not instance_list:
@@ -396,6 +404,8 @@ class OrderBuilder(XMLObject, Amount, Pr
"""
Build delivery from a list of movement
"""
+ # FIXME make sure that activate_kw is used for all active objects
+ # generated from this method and methods called from this method.
if movement_group_node_list is None:
movement_group_node_list = []
if divergence_list is None:
@@ -576,6 +586,9 @@ class OrderBuilder(XMLObject, Amount, Pr
else:
self._processDeliveryCellGroup(
delivery_line,
+ # FIXME passing current movement group node
+ # has bad side effect, so the logic must
+ # be corrected.
movement_group_node,
[],
update_existing_line=update_existing_line,
@@ -793,6 +806,8 @@ class OrderBuilder(XMLObject, Amount, Pr
def getDeliveryCellMovementGroupList(self, **kw):
return self.getMovementGroupList(collect_order_group='cell')
+ # XXX this method is not used in OrderBuilder but in DeliveryBuilder.
+ # So it should perhaps be moved to DeliveryBuilder.
def _searchUpByPortalType(self, obj, portal_type):
limit_portal_type = self.getPortalObject().getPortalType()
while obj is not None:
More information about the Erp5-report
mailing list