From 98844701c31e1bb8286b861ed85b7559ea35db42 Mon Sep 17 00:00:00 2001
From: Romain Courteaud <romain@nexedi.com>
Date: Thu, 21 Jul 2005 13:07:49 +0000
Subject: [PATCH] Check if delivery_value is empty on simulation movement,
 before storing a new value.

git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@3483 20353a03-c40f-0410-a6d1-a30d3c3de9de
---
 product/ERP5/Document/DeliveryBuilder.py | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/product/ERP5/Document/DeliveryBuilder.py b/product/ERP5/Document/DeliveryBuilder.py
index 4cd4f0f920..f0c0f49810 100755
--- a/product/ERP5/Document/DeliveryBuilder.py
+++ b/product/ERP5/Document/DeliveryBuilder.py
@@ -106,13 +106,14 @@ class DeliveryBuilder(OrderBuilder):
                             self, delivery_movement, 
                             simulation_movement, property_dict,
                             update_existing_movement=update_existing_movement)
+    # Check if simulation movement is not already linked to a existing
+    # movement
+    if simulation_movement.getDeliveryValue() is not None:
+      raise "SelectMovementError",\
+            "simulation_movement '%s' must not be selected !" %\
+            simulation_movement.getRelativeUrl()
     # Update simulation movement
-    #simulation_movement.setDeliveryValue(delivery_movement)
     simulation_movement.edit(delivery_value=delivery_movement)
-    # To update the divergence status, the simulation movement must be reindexed
-    # and the delivery must be touched.
-    #simulation_movement.immediateReindexObject()
-    #delivery_movement.activate(after_path_and_method_id = (simulation_movement.getPath(), ['immediateReindexObject', 'recursiveImmediateReindexObject'])).edit()
 
   # Simulation consistency propagation
   security.declareProtected(Permissions.ModifyPortalContent, 
-- 
2.30.9