Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0034589
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] 09. Financial managementmajoralways2016-11-21 11:332017-04-01 12:06
ReporterpsanjuanView Statuspublic 
Assigned Torqueralta 
PriorityurgentResolutionfixedFixed in Version3.0PR17Q1
StatusclosedFix in branchFixed in SCM revision6b520396354e
ProjectionnoneETAnoneTarget Version
OSLinux 64 bitDatabasePostgreSQLJava version1.6.0_18
OS VersionProfessional ApplianceDatabase version8.3.9Ant version1.7.1
Product VersionSCM revision 
Review Assigned Tomarkmm82
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0034589: Error message needed in case of voiding an invoice having an "Awaiting Execution" or "Awaiting Payment" payment related.

DescriptionError message needed in case of voiding an invoice having an "Awaiting Execution" or "Awaiting Payment" payment related.
Steps To Reproduce1) "Awaiting Execution" payment status:

Create a Sales Invoice for F&B España, Alimentos business partner.

Change payment method to an automatic payment method such as "Print Check Simple process" or "Remittance", set up as "Deferred" = Yes.

Add any line to the invoice. Complete and Post the invoice.

Partially pay the invoice and do NOT execute that payment, therefore payment created is in status "Awaiting Execution".

Try to void the invoice.

The invoice is voided, and a new payment is created in status "Payment Received" including remaining sales invoice to pay, and reverse sales invoice amount.

This is not correct.

An error message needs to be shown saying that "Invoice can not be voided as there is an awaiting execution or awaiting payment related. Please either execute that payment or cancel it".


2) "Awaiting Payment" payment status:

Create a Sales Invoice for F&B España, Alimentos business partner.

Change payment method to an automatic payment method such as "Print Check Simple process" or "Remittance", set up as "Deferred" = Yes.

Add any line to the invoice. Complete and Post the invoice.

Partially pay the invoice and do NOT execute that payment, therefore payment created is in status "Awaiting Execution".

Go to the payment and press the button Reactivate. Choose "Reactivate" action, therefore payment status is "Awaiting Payment".

Try to void the invoice.

The invoice is voided, and a new payment is created in status "Payment Received" including remaining sales invoice amount to pay, and reverse sales invoice amount.

This is not correct.

An error message needs to be shown saying that "Invoice can not be voided as there is an awaiting execution or awaiting payment related. Please either execute that payment or cancel it".

Proposed Solution(*) Proposed scenario 1: partially paid invoice or totally paid invoice, but payment related do not executed. Once error is shown, the payment is Voided.

Create a Sales Invoice for F&B España, Alimentos business partner.
Change payment method to an automatic payment method such as "Print Check Simple process" or "Remittance", set up as "Deferred" = Yes.

Add any line to the invoice. Complete and Post the invoice.
Partially pay the invoice and do NOT execute that payment.

Try to void the invoice.

An error message is shown saying that "Invoice can not be voided as there is an awaiting execution payment related. Please either execute that payment or cancel it".

Navigate to the payment and press Reactivate button, select "Void" action. Press OK.

Go back to the invoice and void it once again.

Check that the invoice has been voided by a reverse document. Go to payment plan tab and verify there are 2 payments. Go to Payment details tab and verify that first payment is in status Void and second one as "Payment Received". Navigate to that payment.

Check it does have include the invoice and its reverse. Amount is 0.00 and status is "Payment Received".

Repeat same test by using below listed actions in the payment window:

- Navigate to the payment and press Reactivate button, select "Reactivate and Delete lines" action. Press OK. Same result is obtained.

(*) Proposed scenario 2: partially paid invoice or totally paid invoice, but payment related do not executed. Once error is shown, the payment is Executed.

Create a Sales Invoice for F&B España, Alimentos business partner.
Change payment method to an automatic payment method such as "Print Check Simple process" or "Remittance", set up as "Deferred" = Yes.

Add any line to the invoice. Complete and Post the invoice.
Partially pay the invoice and do NOT execute that payment.

Try to void the invoice.

An error message is shown saying that "Invoice can not be voided as there is an awaiting execution payment related. Please either execute that payment or cancel it".

Navigate to the payment and press Execute button to execute the payment.

Go back to the invoice and void it once again.

Check that the invoice has been voided by a reverse document and no additional payment in has been created. Go to payment plan tab and verify there is only payment.

Reverse invoice does not have any payment related.

(*) Proposed scenario 3: partially paid invoice or totally paid invoice, but payment related do not execute, but reactivated.

Create a Sales Invoice for F&B España, Alimentos business partner.
Change payment method to an automatic payment method such as "Print Check Simple process" or "Remittance", set up as "Deferred" = Yes.

Add any line to the invoice. Complete and Post the invoice.
Partially pay the invoice and do NOT execute that payment, but just Reactivate it by unsing the option "Reactivate", Action "Reactivate".

Try to void the invoice.

An error message is shown saying that "Invoice can not be voided as there is an awaiting execution payment related. Please either execute that payment or cancel it".

Go back to the payment and press add details, select the invoice to pay and either execute the payment, or void it or reactivate it and delete its lines.
TagsNo tags attached.
Attached Files

- Relationships [ Relation Graph ] [ Dependency Graph ]
related to defect 0034322 closedrqueralta Payment of 0 amount created when voiding an unpaid invoice is created in Awaiting Execution status 
related to defect 0037963 closedmarkmm82 Payment in Awaiting payment status is created when an invoice cannot be voided because document cancelled is a return 

-  Notes
(0091693)
psanjuan (manager)
2016-11-21 12:18

 Test Case 1- voice partially paid by an automatic payment method (Check) and void. First Payment not executed.

Go to Sales Invoice and create a new invoice for F&B España, Alimentos business partner. Select Check payment method. Complete the invoice.

Partially paid the invoice.
Verify that a payment in is created with status "Awaiting Execution".
Do not execute the payment created.

Void the invoice.

Check that a payment in has been created including Invoice remaining amount to pay and Reverse Amount to pay. This is not correct. Implement proposed scenario 2 above. Same would apply in case of totally paying the invoice, but the payment related in "Awaiting Execution" status is not executed.

Repeat same test case for a purchase invoice.


Test Case 2 - Invoice partially paid by an automatic payment method (Check) and void. First Payment executed.

Go to Sales Invoice and create a new invoice for F&B España, Alimentos business partner. Select Check payment method. Complete the invoice.

Partially paid the invoice.
Verify that a payment in is created with status "Awaiting Execution".
Execute the payment created. Check that payment status has changed to "Payment Received".

Verify that sales invoice shows a received amount of the payment received.

Void the invoice.

Verify that there is no new payment in created and both sales invoice and reversed sales invoice remain as they are.

Repeat same test case for a purchase invoice.
(0092907)
hgbot (developer)
2016-12-21 15:10

 Repository: erp/devel/pi
Changeset: 6b520396354e6df4f96f72121221c1729c0fe0c9
Author: Rafael Queralta Pozo <rqueralta <at> nauta.cu>
Date: Mon Nov 21 10:14:44 2016 -0500
URL: http://code.openbravo.com/erp/devel/pi/rev/6b520396354e6df4f96f72121221c1729c0fe0c9 [^]

Fixes issue 34589: Error message when voiding an invoice with a related payment

An error message is shown in case of voiding an invoice having an "Awaiting Execution" or "Awaiting Payment" payment related.

---
M modules/org.openbravo.advpaymentmngt/src-db/database/sourcedata/AD_MESSAGE.xml
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/ad_actionbutton/ProcessInvoice.java
---
(0092908)
hgbot (developer)
2016-12-21 15:10

 Repository: erp/devel/pi
Changeset: c305082199cf920f705e80a85c9b170a85d3bfe4
Author: Mark <markmm82 <at> gmail.com>
Date: Mon Dec 19 13:56:15 2016 -0500
URL: http://code.openbravo.com/erp/devel/pi/rev/c305082199cf920f705e80a85c9b170a85d3bfe4 [^]

Related to issue 34589: Code review improvements

Merged two conditions in only one to evaluate if an Invoice has an awaiting execution payment related.

---
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/ad_actionbutton/ProcessInvoice.java
---
(0092909)
hgbot (developer)
2016-12-21 15:19

 Repository: erp/devel/pi
Changeset: 6d2315567e5fbffabae911a59ec72dc59ff176da
Author: Mark <markmm82 <at> gmail.com>
Date: Wed Dec 21 09:15:42 2016 -0500
URL: http://code.openbravo.com/erp/devel/pi/rev/6d2315567e5fbffabae911a59ec72dc59ff176da [^]

Related to issue 34589: Fixed code format

---
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/ad_actionbutton/ProcessInvoice.java
---
(0092910)
markmm82 (developer)
2016-12-21 15:25

 Code review + Testing OK
(0092926)
hudsonbot (developer)
2016-12-21 20:51

 A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/abc21394bfb8 [^]
Maturity status: Test
(0092927)
hudsonbot (developer)
2016-12-21 20:51

 A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/abc21394bfb8 [^]
Maturity status: Test
(0092928)
hudsonbot (developer)
2016-12-21 20:51

 A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/abc21394bfb8 [^]
Maturity status: Test
(0095767)
hgbot (developer)
2017-03-31 13:22

 Repository: erp/devel/pi
Changeset: 4690240411138ef7779f411988811cc31ecc2d3e
Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com>
Date: Fri Mar 31 13:14:06 2017 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/4690240411138ef7779f411988811cc31ecc2d3e [^]

Related to issue 34589 & Related to issue 34992: Code review improvements

Use lowercase letter in hql to be more readable.
Use ids in where clauses instead of dal objects.
Change in clause with a join in psdQuery query to improve performance.
Use query.list() directly in for statement.

---
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/ad_actionbutton/ProcessInvoice.java
---
(0095781)
hudsonbot (developer)
2017-04-01 12:06

 A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/44185ff76d75 [^]
Maturity status: Test

- Issue History
Date Modified Username Field Change
2016-11-21 11:33 psanjuan New Issue
2016-11-21 11:33 psanjuan Assigned To => Triage Finance
2016-11-21 11:33 psanjuan Modules => Core
2016-11-21 11:33 psanjuan Triggers an Emergency Pack => No
2016-11-21 11:38 psanjuan Steps to Reproduce Updated View Revisions
2016-11-21 11:56 psanjuan Steps to Reproduce Updated View Revisions
2016-11-21 11:56 psanjuan Relationship added related to 0034322
2016-11-21 11:58 psanjuan Proposed Solution updated
2016-11-21 12:06 psanjuan Proposed Solution updated
2016-11-21 12:11 psanjuan Proposed Solution updated
2016-11-21 12:18 psanjuan Note Added: 0091693
2016-11-22 16:08 rqueralta Assigned To Triage Finance => rqueralta
2016-11-22 16:09 rqueralta Status new => scheduled
2016-12-21 15:10 hgbot Checkin
2016-12-21 15:10 hgbot Note Added: 0092907
2016-12-21 15:10 hgbot Status scheduled => resolved
2016-12-21 15:10 hgbot Resolution open => fixed
2016-12-21 15:10 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/6b520396354e6df4f96f72121221c1729c0fe0c9 [^]
2016-12-21 15:10 hgbot Checkin
2016-12-21 15:10 hgbot Note Added: 0092908
2016-12-21 15:19 hgbot Checkin
2016-12-21 15:19 hgbot Note Added: 0092909
2016-12-21 15:25 markmm82 Review Assigned To => markmm82
2016-12-21 15:25 markmm82 Note Added: 0092910
2016-12-21 15:25 markmm82 Status resolved => closed
2016-12-21 15:25 markmm82 Fixed in Version => 3.0PR17Q1
2016-12-21 20:51 hudsonbot Checkin
2016-12-21 20:51 hudsonbot Note Added: 0092926
2016-12-21 20:51 hudsonbot Checkin
2016-12-21 20:51 hudsonbot Note Added: 0092927
2016-12-21 20:51 hudsonbot Checkin
2016-12-21 20:51 hudsonbot Note Added: 0092928
2017-03-31 13:22 hgbot Checkin
2017-03-31 13:22 hgbot Note Added: 0095767
2017-04-01 12:06 hudsonbot Checkin
2017-04-01 12:06 hudsonbot Note Added: 0095781
2018-02-16 15:02 ngarcia Relationship added related to 0037963

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker