Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0020017Openbravo ERP09. Financial managementpublic2012-03-14 10:262012-04-02 12:15
dmiguelez 
dmiguelez 
immediatemajoralways
closedfixed 
20Community Appliance
pi 
3.0MP9 
Core
No
0020017: When making a Payment, it creates unexpected lines in the Payment Schedule Details
Three Sales Orders are created, two of wich are negative.
Then a Sales Invoice is created using the three Orders with a Payment Method that creates three Payment Plan lines.
Then a Payment is made to the first line.
Making this Payment two unexpected lines are created in the Payment Schedule Details with amount Zero.
Test Cases from [EPPf010] to [EPPf050]

[EPPf010]
Create the Sales Order.

Login as:

Username: QAAdmin
Password: QAAdmin
Go to Sales Management || Transactions || Sales order:
Click New and select:

“Transaction Document”: Standard Order
“Business Partner”: Type CUSA and press enter key. Customer A should be showed instead.
“Order date”: System +1 month 5 days.
“Scheduled Delivery date”: System +1 month 5 days.
“Payment Method”: 2.1 (Spain)
“Payment Terms”: 30-60-90
“Invoice Terms”: Immediate
Move to Lines. Click new and select:

“Product”: Type FGA and press enter key. Final good A should be showed instead.
“Order Quantity”: - 101
Click New and select:

“Product”: Type FGB and press enter key. Final good B should be showed instead.
“Order Quantity”: - 102
Click the “Book” button. Click OK.

[EPPf020]
Create the Sales Order.

Login as:

Username: QAAdmin
Password: QAAdmin
Go to Sales Management || Transactions || Sales order:
Click New and select:

“Transaction Document”: Standard Order
“Business Partner”: Type CUSA and press enter key. Customer A should be showed instead.
“Order date”: System +1 month 10 days.
“Scheduled Delivery date”: System +1 month 10 days.
“Payment Method”: 2.1 (Spain)
“Payment Terms”: 30-60-90
“Invoice Terms”: Immediate
Move to Lines. Click new and select:

“Product”: Type FGA and press enter key. Final good A should be showed instead.
“Order Quantity”: 201
Click the “Book” button. Click OK.

[EPPf030]
Create the Sales Order.
Login as:

Username: QAAdmin
Password: QAAdmin
Go to Sales Management || Transactions || Sales order:
Click New and select:

“Transaction Document”: Standard Order
“Business Partner”: Type CUSA and press enter key. Customer A should be showed instead.
“Order date”: System +1 month 15 days.
“Scheduled Delivery date”: System +1 month 15 days.
“Payment Method”: 2.1 (Spain)
“Payment Terms”: 30-60-90
“Invoice Terms”: Immediate
Move to Lines. Click new and select:

“Product”: Type FGA and press enter key. Final good A should be showed instead.
“Order Quantity”: - 301
Click New and select:

“Product”: Type FGB and press enter key. Final good B should be showed instead.
“Order Quantity”: - 302
Click New and select:

“Product”: Type FGC and press enter key. Final good C should be showed instead.
“Order Quantity”: - 303
Click the “Book” button. Click OK.

[EPPf040]
Create the Sales Invoice.

Login as:

Username: QAAdmin
Password: QAAdmin
Go to Sales Management || Transactions || Sales invoice:
Click New and select:

“Business Partner”: Type CUSA and press enter key. Customer A should be showed instead.
“Payment Method”: 2.1 (Spain)
“Payment Terms”: 30-60-90
Click on the button “Create Lines From”:

In the order combo box select the order created in [EPPf010].
Mark all the records that appear in the grid and press OK.
Click on the button “Create Lines From”:

In the order combo box select the order created in [EPPf020].
Mark all the records that appear in the grid and press OK.
Click on the button “Create Lines From”:

In the order combo box select the order created in [EPPf030].
Mark all the records that appear in the grid and press OK.
Click the “Complete” button. Click OK.

[EPPf050]
Create an Executed Payment for the Invoice.

Login as:

Username: QAAdmin
Password: QAAdmin
Go to Sales Financial Management || Receivables & Payables || Transactions || Payment In:
Click New and select:

“Received From”: Type CUSA and press enter key. Customer A should be showed instead.
“Payment Method”: 2.1 (Spain)
“Amount”: -400.00
Then click the “Add Details” button.
In the pop-up select:

“Transaction Type”: Invoices
And then make sure that the only line selected is the one with the values:

“Sales Invoice No.”: The invoice created in [EPPf040].
“Due Date”: System +1 month 5 days.
“Outstanding Amount”: -639.06
Click the “Process” button and then OK.
Click on the “Execute Payment” button and then Execute.

Realize that in the Lines of the Payment, lines with amount Zero are created.
No tags attached.
Issue History
2012-03-14 10:26dmiguelezNew Issue
2012-03-14 10:26dmiguelezAssigned To => dmiguelez
2012-03-14 10:26dmiguelezModules => Core
2012-03-14 10:27hgbotCheckin
2012-03-14 10:27hgbotNote Added: 0046376
2012-03-14 10:27hgbotStatusnew => resolved
2012-03-14 10:27hgbotResolutionopen => fixed
2012-03-14 10:27hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/ba9872fbd663c8daae6b3fd58ff41fdab20cd4fd [^]
2012-03-14 17:37hgbotCheckin
2012-03-14 17:37hgbotNote Added: 0046425
2012-03-14 17:37hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/devel/pi/rev/ba9872fbd663c8daae6b3fd58ff41fdab20cd4fd [^] => http://code.openbravo.com/erp/devel/pi/rev/872bbf30468cf1d84aec5c727658cc12b2f4d3d6 [^]
2012-03-14 17:57hgbotCheckin
2012-03-14 17:57hgbotNote Added: 0046427
2012-03-20 12:20eduardo_ArgalNote Added: 0046540
2012-03-20 12:20eduardo_ArgalStatusresolved => closed
2012-03-21 16:25dmiguelezNote Added: 0046599
2012-04-01 20:33hgbotCheckin
2012-04-01 20:33hgbotNote Added: 0047010
2012-04-01 20:33hgbotStatusclosed => resolved
2012-04-01 20:33hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/devel/pi/rev/872bbf30468cf1d84aec5c727658cc12b2f4d3d6 [^] => http://code.openbravo.com/erp/devel/pi/rev/f3389d7ef1310be570266f7f98610f8d6e086134 [^]
2012-04-01 20:34hgbotCheckin
2012-04-01 20:34hgbotNote Added: 0047014
2012-04-02 06:10hudsonbotCheckin
2012-04-02 06:10hudsonbotNote Added: 0047127
2012-04-02 06:10hudsonbotCheckin
2012-04-02 06:10hudsonbotNote Added: 0047140
2012-04-02 06:10hudsonbotCheckin
2012-04-02 06:10hudsonbotNote Added: 0047142
2012-04-02 12:15gorka_gilNote Added: 0047298
2012-04-02 12:15gorka_gilStatusresolved => closed
2012-04-02 12:15gorka_gilFixed in Version => 3.0MP9

Notes
(0046376)
hgbot   
2012-03-14 10:27   
Repository: erp/devel/pi
Changeset: ba9872fbd663c8daae6b3fd58ff41fdab20cd4fd
Author: David Miguelez <david.miguelez <at> openbravo.com>
Date: Wed Mar 14 10:27:00 2012 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/ba9872fbd663c8daae6b3fd58ff41fdab20cd4fd [^]

Fixes issue 0020017: Checks if the amount is Zero before save the line.

---
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/process/FIN_AddPayment.java
---
(0046425)
hgbot   
2012-03-14 17:37   
Repository: erp/devel/pi
Changeset: 872bbf30468cf1d84aec5c727658cc12b2f4d3d6
Author: David Miguelez <david.miguelez <at> openbravo.com>
Date: Wed Mar 14 17:36:39 2012 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/872bbf30468cf1d84aec5c727658cc12b2f4d3d6 [^]

Fixes issue 0020017: Checks if the amount is Zero before saving the line.

---
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/process/FIN_AddPayment.java
---
(0046427)
hgbot   
2012-03-14 17:57   
Repository: erp/devel/pi
Changeset: 3079cc5712a3d5c0cd3e96fa12e9def9262dcbc8
Author: David Miguelez <david.miguelez <at> openbravo.com>
Date: Wed Mar 14 17:56:21 2012 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/3079cc5712a3d5c0cd3e96fa12e9def9262dcbc8 [^]

Related to Issue 20017: Removes unnecesary check in conditional.

---
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/process/FIN_AddPayment.java
---
(0046540)
eduardo_Argal   
2012-03-20 12:20   
Tested and working for me.
(0046599)
dmiguelez   
2012-03-21 16:25   
Test plan:

Follow the steps to reproduce, Test Cases from [EPPf010] to [EPPf050]

After executing the Payment, move to the Lines tab and realize that there is only one line, with amount -400.00
(0047010)
hgbot   
2012-04-01 20:33   
Repository: erp/devel/pi
Changeset: f3389d7ef1310be570266f7f98610f8d6e086134
Author: David Miguelez <david.miguelez <at> openbravo.com>
Date: Wed Mar 14 17:36:39 2012 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/f3389d7ef1310be570266f7f98610f8d6e086134 [^]

Fixes issue 0020017: Checks if the amount is Zero before saving the line.

---
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/process/FIN_AddPayment.java
---
(0047014)
hgbot   
2012-04-01 20:34   
Repository: erp/devel/pi
Changeset: 799471bc25a1044646caefe51fe88003789275a2
Author: David Miguelez <david.miguelez <at> openbravo.com>
Date: Wed Mar 14 17:56:21 2012 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/799471bc25a1044646caefe51fe88003789275a2 [^]

Related to Issue 20017: Removes unnecesary check in conditional.

---
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/process/FIN_AddPayment.java
---
(0047127)
hudsonbot   
2012-04-02 06:10   
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/7814864461ac [^]

Maturity status: Test
(0047140)
hudsonbot   
2012-04-02 06:10   
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/7814864461ac [^]

Maturity status: Test
(0047142)
hudsonbot   
2012-04-02 06:10   
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/7814864461ac [^]

Maturity status: Test
(0047298)
gorka_gil   
2012-04-02 12:15   
Reopened by merge.