Commit b23d0ac1 authored by Yoshinori Okuji's avatar Yoshinori Okuji

It is useless to traverse from a builder to find movements. Use the portal object instead.

git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@38183 20353a03-c40f-0410-a6d1-a30d3c3de9de
parent c03e0ffa
...@@ -121,7 +121,8 @@ class OrderBuilder(XMLObject, Amount, Predicate): ...@@ -121,7 +121,8 @@ class OrderBuilder(XMLObject, Amount, Predicate):
delivery_relative_url_list=delivery_relative_url_list, delivery_relative_url_list=delivery_relative_url_list,
applied_rule_uid=applied_rule_uid,**kw) applied_rule_uid=applied_rule_uid,**kw)
else: 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] in movement_relative_url_list]
if not movement_list: if not movement_list:
return [] return []
...@@ -131,7 +132,7 @@ class OrderBuilder(XMLObject, Amount, Predicate): ...@@ -131,7 +132,7 @@ class OrderBuilder(XMLObject, Amount, Predicate):
delivery_list = self.buildDeliveryList( delivery_list = self.buildDeliveryList(
root_group_node, root_group_node,
delivery_relative_url_list=delivery_relative_url_list, delivery_relative_url_list=delivery_relative_url_list,
movement_list=movement_list,**kw) movement_list=movement_list, **kw)
# Call a script after building # Call a script after building
self.callAfterBuildingScript(delivery_list, movement_list, **kw) self.callAfterBuildingScript(delivery_list, movement_list, **kw)
# XXX Returning the delivery list is probably not necessary # XXX Returning the delivery list is probably not necessary
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment