0028318: Payment Proposal's reactivation doesn't modify related Purchase Invoices values correctly.

Description

When reactivating a Payment Proposal, the related Purchase Invoices matain "Paid in Full Date" and "Days to Pay in Full" fields values instead of setting to null.

Steps To Reproduce

Select a record in Payment Proposal window which status is "Awaiting Payment".
Click Reactivate button.
Open related Purchase Invoices.
Check that "Paid in Full Date" and "Days to Pay in Full" values are not null.

Proposed Solution

Set the values to null when calling FIN_PaymentProcess.java class.

Tags

No tags attached.

Attached Files

patch.diff [^] (1,035 bytes) 2014-11-28 12:36 [Show Content] [Hide Content]

diff --git a/modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/process/FIN_PaymentProcess.java b/modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/process/FIN_PaymentProcess.java
--- a/modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/process/FIN_PaymentProcess.java
+++ b/modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/process/FIN_PaymentProcess.java
@@ -839,6 +839,8 @@
                         .getInvoicePaymentSchedule() != null ? paymentScheduleDetail
                         .getInvoicePaymentSchedule().getInvoice() : null;
                     paidAmount = BigDecimal.ZERO;
+                    invoiceForConversion.setFinalSettlementDate(null);
+                    invoiceForConversion.setDaysSalesOutstanding(null);
                     if (!(businessPartner == null)) {
                       final Currency fromCurrency = payment.getCurrency();
                       // At this point the BP must have a currency, because it is set when

Relationships [ Relation Graph ] [ Dependency Graph ]

Notes
There are no notes attached to this issue.

Issue History
Date Modified	Username	Field	Change
2014-11-28 12:36	asiermartirena	New Issue
2014-11-28 12:36	asiermartirena	Assigned To	=> dmiguelez
2014-11-28 12:36	asiermartirena	File Added: patch.diff
2014-11-28 12:36	asiermartirena	Modules	=> Core
2014-11-28 12:36	asiermartirena	Triggers an Emergency Pack	=> No