From 4a9e8a128b0e5dc73053a21ad864d617669c2a4d Mon Sep 17 00:00:00 2001
From: Jean-Paul Smets <jp@nexedi.com>
Date: Sat, 7 Aug 2010 13:00:40 +0000
Subject: [PATCH] Only build what is buildable. Invoke mixin rather than non
 existing class.

git-svn-id: https://svn.erp5.org/repos/public/erp5/sandbox/amount_generator@37628 20353a03-c40f-0410-a6d1-a30d3c3de9de
---
 product/ERP5/Document/SimulatedDeliveryBuilder.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/product/ERP5/Document/SimulatedDeliveryBuilder.py b/product/ERP5/Document/SimulatedDeliveryBuilder.py
index e653092631..cc9116d11f 100644
--- a/product/ERP5/Document/SimulatedDeliveryBuilder.py
+++ b/product/ERP5/Document/SimulatedDeliveryBuilder.py
@@ -123,7 +123,7 @@ class SimulatedDeliveryBuilder(BuilderMixin):
       movement_list = select_method(**kw)
     # XXX Use buildSQLQuery will be better
     movement_list = [x for x in movement_list if \
-                     x.getDeliveryValueList()==[]]
+                     x.getDeliveryValueList()==[] and x.isBuildable()]
     # XXX  Add predicate test
     # XXX FIXME Check that there is no double in the list
     # Because we can't trust simulation_select_method
@@ -150,7 +150,7 @@ class SimulatedDeliveryBuilder(BuilderMixin):
       Create the relation between simulation movement
       and delivery movement.
     """
-    GeneratedDeliveryBuilder._setDeliveryMovementProperties(
+    BuilderMixin._setDeliveryMovementProperties(
                             self, delivery_movement,
                             simulation_movement, property_dict,
                             update_existing_movement=update_existing_movement,
-- 
2.30.9