From 3b06eb7a0a0340329cb8336178bf9976858a90d6 Mon Sep 17 00:00:00 2001 From: Carlos Ruiz Date: Thu, 11 Mar 2010 23:04:11 +0000 Subject: [PATCH] Fix [1904717] Invoice reversing has incorrect doc status Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=1904717 --- base/src/org/compiere/wf/MWFActivity.java | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/base/src/org/compiere/wf/MWFActivity.java b/base/src/org/compiere/wf/MWFActivity.java index a0c3869a6d..39f4e411ff 100644 --- a/base/src/org/compiere/wf/MWFActivity.java +++ b/base/src/org/compiere/wf/MWFActivity.java @@ -905,7 +905,13 @@ public class MWFActivity extends X_AD_WF_Activity implements Runnable success = doc.processIt (m_node.getDocAction()); // ** Do the work setTextMsg(doc.getSummary()); processMsg = doc.getProcessMsg(); - m_docStatus = doc.getDocStatus(); + // Bug 1904717 - Invoice reversing has incorrect doc status + // Just prepare and complete return a doc status to take into account + // the rest of methods return boolean, so doc status must not be taken into account when not successful + if ( DocAction.ACTION_Prepare.equals(m_node.getDocAction()) + || DocAction.ACTION_Complete.equals(m_node.getDocAction()) + || success) + m_docStatus = doc.getDocStatus(); } catch (Exception e) { if (m_process != null)