[Erp5-report] r24972 - in /erp5/trunk/bt5/erp5_banking_check: WorkflowTemplateItem/portal_w...

[nobody at svn.erp5.org]

Author: vincent
Date: Tue Dec 23 13:22:14 2008
New Revision: 24972

URL: http://svn.erp5.org?rev=24972&view=rev
Log:
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.

Modified:
    erp5/trunk/bt5/erp5_banking_check/WorkflowTemplateItem/portal_workflow/stop_payment_workflow/scripts/liftPositionAccounting.xml
    erp5/trunk/bt5/erp5_banking_check/bt/revision

Modified: erp5/trunk/bt5/erp5_banking_check/WorkflowTemplateItem/portal_workflow/stop_payment_workflow/scripts/liftPositionAccounting.xml
URL: http://svn.erp5.org/erp5/trunk/bt5/erp5_banking_check/WorkflowTemplateItem/portal_workflow/stop_payment_workflow/scripts/liftPositionAccounting.xml?rev=24972&r1=24971&r2=24972&view=diff
==============================================================================
--- erp5/trunk/bt5/erp5_banking_check/WorkflowTemplateItem/portal_workflow/stop_payment_workflow/scripts/liftPositionAccounting.xml [utf8] (original)
+++ erp5/trunk/bt5/erp5_banking_check/WorkflowTemplateItem/portal_workflow/stop_payment_workflow/scripts/liftPositionAccounting.xml [utf8] Tue Dec 23 13:22:14 2008
@@ -97,11 +97,12 @@
       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

Modified: erp5/trunk/bt5/erp5_banking_check/bt/revision
URL: http://svn.erp5.org/erp5/trunk/bt5/erp5_banking_check/bt/revision?rev=24972&r1=24971&r2=24972&view=diff
==============================================================================
--- erp5/trunk/bt5/erp5_banking_check/bt/revision [utf8] (original)
+++ erp5/trunk/bt5/erp5_banking_check/bt/revision [utf8] Tue Dec 23 13:22:14 2008
@@ -1,1 +1,1 @@
-415
+416