Only update the stop date if we are on "started" state (the same script is...

Only update the stop date if we are on "started" state (the same script is called to go to delivered).
Fix a stupid mistake when checking if stop_date was given.

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

bt5/erp5_banking_check/WorkflowTemplateItem/portal_workflow/stop_payment_workflow/scripts/liftPositionAccounting.xml

@@ -97,11 +97,12 @@ for movement in movement_list:\n
       raise ValidationFailed, (msg,)\n
 debit_required = transaction.isDebitRequired()\n
 if debit_required:\n
-  stop_date = state_change.kwargs.get(\'stop_date\')\n
-  if \'stop_date\' is None:\n
-    msg = Message(domain = "ui", message="No stop date provided")\n
-    raise ValidationFailed, (msg,)\n
-  transaction.setStopDate(stop_date)\n
+  if transaction.getSimulationState() == \'started\':\n
+    stop_date = state_change.kwargs.get(\'stop_date\')\n
+    if stop_date is None:\n
+      msg = Message(domain = "ui", message="No stop date provided")\n
+      raise ValidationFailed, (msg,)\n
+    transaction.setStopDate(stop_date)\n
 \n
   # Source and destination will be updated automaticaly based on the category of bank account\n
   # The default account chosen should act as some kind of *temp* account or *parent* account\n

bt5/erp5_banking_check/bt/revision

-415
+416
\ No newline at end of file