From a003c8317726b9290aa690cbd97cbc85af212839 Mon Sep 17 00:00:00 2001 From: Carlos Ruiz Date: Wed, 4 Feb 2015 11:38:55 -0500 Subject: [PATCH] IDEMPIERE-1616 reversing of reconciled payments makes database inconsistent --- org.adempiere.base/src/org/compiere/model/MPayment.java | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/org.adempiere.base/src/org/compiere/model/MPayment.java b/org.adempiere.base/src/org/compiere/model/MPayment.java index 2d4edd1cd4..74958be833 100644 --- a/org.adempiere.base/src/org/compiere/model/MPayment.java +++ b/org.adempiere.base/src/org/compiere/model/MPayment.java @@ -2599,9 +2599,6 @@ public class MPayment extends X_C_Payment } MPeriod.testPeriodOpen(getCtx(), dateAcct, getC_DocType_ID(), getAD_Org_ID()); - // Auto Reconcile if not on Bank Statement - boolean reconciled = getC_BankStatementLine_ID() == 0; //AZ Goodwill - // Create Reversal MPayment reversal = new MPayment (getCtx(), 0, get_TrxName()); copyValues(this, reversal); @@ -2620,7 +2617,7 @@ public class MPayment extends X_C_Payment reversal.setOverUnderAmt(getOverUnderAmt().negate()); // reversal.setIsAllocated(true); - reversal.setIsReconciled(reconciled); // to put on bank statement + reversal.setIsReconciled(false); reversal.setIsOnline(false); reversal.setIsApproved(true); reversal.setR_PnRef(null); @@ -2650,7 +2647,6 @@ public class MPayment extends X_C_Payment // Unlink & De-Allocate deAllocate(accrual); - setIsReconciled (reconciled); setIsAllocated (true); // the allocation below is overwritten // Set Status addDescription("(" + reversal.getDocumentNo() + "<-)");