fix a bug that creats a new simulation movement for already simulated movement in BPM case.

git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@32394 20353a03-c40f-0410-a6d1-a30d3c3de9de

product/ERP5/Document/DeliveryRule.py

@@ -270,10 +270,17 @@ class DeliveryRule(Rule):
   def _getInputMovementList(self, applied_rule):
     """Return list of movements from delivery"""
     delivery = applied_rule.getDefaultCausalityValue()
+    movement_list = []
     if delivery is not None:
-      return delivery.getMovementList(
-                     portal_type=delivery.getPortalDeliveryMovementTypeList())
-    return []
+      existing_movement_list = applied_rule.objectValues()
+      for movement in delivery.getMovementList(
+        portal_type=delivery.getPortalDeliveryMovementTypeList()):
+        simulation_movement = self._getDeliveryRelatedSimulationMovement(
+          movement)
+        if simulation_movement is None or \
+               simulation_movement in existing_movement_list:
+          movement_list.append(movement)
+    return movement_list
 
   def _getExpandablePropertyUpdateDict(self, applied_rule, movement,
       business_path, current_property_dict):