[Erp5-report] r38183 yo - /erp5/trunk/products/ERP5/Document/OrderBuilder.py

[nobody at svn.erp5.org]

Author: yo
Date: Wed Sep  8 07:44:37 2010
New Revision: 38183

URL: http://svn.erp5.org?rev=38183&view=rev
Log:
It is useless to traverse from a builder to find movements. Use the portal object instead.

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=38183&r1=38182&r2=38183&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/Document/OrderBuilder.py [utf8] (original)
+++ erp5/trunk/products/ERP5/Document/OrderBuilder.py [utf8] Wed Sep  8 07:44:37 2010
@@ -121,7 +121,8 @@ class OrderBuilder(XMLObject, Amount, Pr
                                         delivery_relative_url_list=delivery_relative_url_list,
                                         applied_rule_uid=applied_rule_uid,**kw)
       else:
-        movement_list = [self.restrictedTraverse(relative_url) for relative_url \
+        restrictedTraverse = self.getPortalObject().restrictedTraverse
+        movement_list = [restrictedTraverse(relative_url) for relative_url \
                          in movement_relative_url_list]
     if not movement_list:
       return []
@@ -131,7 +132,7 @@ class OrderBuilder(XMLObject, Amount, Pr
     delivery_list = self.buildDeliveryList(
                        root_group_node,
                        delivery_relative_url_list=delivery_relative_url_list,
-                       movement_list=movement_list,**kw)
+                       movement_list=movement_list, **kw)
     # Call a script after building
     self.callAfterBuildingScript(delivery_list, movement_list, **kw)
     # XXX Returning the delivery list is probably not necessary