Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0028591Openbravo ERPZ. Otherspublic2015-01-13 16:522016-02-02 09:34
ngarcia 
AtulOpenbravo 
immediatemajoralways
closedfixed 
5
 
3.0PR15Q2 
Sandrahuguet
Core
Production - Confirmed Stable
2014-04-22
3.0PR14Q3
https://code.openbravo.com/erp/devel/pi/rev/51d72e493b90896a089dca36e1a362347c15761a [^]
Yes
0028591: Reversing a payment is creating wrong invoice payment plan details
When doing a Reverse Payment of a Payment that is paying more than one
invoice, the payment plan details generated have wrong amount. It seems that it is setting the total amount of the payment instead of the amount of the corresponding invoice amount paid in the payment. The same error happens when doing a payment with an invoice and a GL Item.
0. Enable the Reverse Payment functionality as described in the following wiki document:
http://wiki.openbravo.com/wiki/Projects:Reversepayment/User_Manual [^]

1. Create 2 Sales Invoices using any business partner and product.
2. Pay both invoices in the same Payment In.
3. Reverse the created payment.
4. Go to one of the invoices and open the add payment pick and execute.
5. Check that the outstanding amount is the total amount of the reversed payment
instead of the expected payment.
No tags attached.
depends on defect 0028686 closed airaceburu Reversing a payment is creating wrong invoice payment plan 
caused by defect 0026121 closed AtulOpenbravo Wrong payment schedule detail generated when running Reverse Payment process from Write-off payment 
related to defect 0028912 closed Sandrahuguet Modulescript needed in order to fix the payment proposal lines wrongly created because of 28591 
Issue History
2015-01-13 16:52ngarciaNew Issue
2015-01-13 16:52ngarciaAssigned To => Sandrahuguet
2015-01-13 16:52ngarciaModules => Core
2015-01-13 16:52ngarciaResolution time => 1424127600
2015-01-13 16:52ngarciaTriggers an Emergency Pack => No
2015-01-13 16:53ngarciaIssue Monitored: networkb
2015-01-13 18:15SandrahuguetAssigned ToSandrahuguet => AtulOpenbravo
2015-01-13 18:16ngarciaRegression level => Production - Confirmed Stable
2015-01-13 18:16ngarciaRegression date => 2014-04-22
2015-01-13 18:16ngarciaRegression introduced in release => 3.0PR14Q3
2015-01-13 18:16ngarciaRegression introduced by commit => https://code.openbravo.com/erp/devel/pi/rev/51d72e493b90896a089dca36e1a362347c15761a [^]
2015-01-13 18:16ngarciaTriggers an Emergency PackNo => Yes
2015-01-13 18:18ngarciaPriorityurgent => immediate
2015-01-13 18:25AtulOpenbravoStatusnew => scheduled
2015-01-13 18:25AtulOpenbravofix_in_branch => pi
2015-01-13 18:32ngarciaRelationship addedcaused by 0026121
2015-01-13 20:33AtulOpenbravoNote Added: 0073482
2015-01-20 12:06SandrahuguetRelationship addeddepends on 0028686
2015-01-20 15:47SandrahuguetFile Added: bug_28591.diff
2015-01-20 18:35hgbotCheckin
2015-01-20 18:35hgbotNote Added: 0073655
2015-01-20 18:35hgbotStatusscheduled => resolved
2015-01-20 18:35hgbotResolutionopen => fixed
2015-01-20 18:35hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/9c12d1443b200650ca55861bb6377d2b1d8e964c [^]
2015-01-20 18:36SandrahuguetReview Assigned To => Sandrahuguet
2015-01-20 18:36SandrahuguetNote Added: 0073656
2015-01-20 18:36SandrahuguetStatusresolved => closed
2015-01-20 18:36SandrahuguetFixed in Version => 3.0PR15Q2
2015-01-21 03:52AtulOpenbravoNote Added: 0073669
2015-01-21 03:52AtulOpenbravoStatusclosed => new
2015-01-21 03:52AtulOpenbravoResolutionfixed => open
2015-01-21 03:52AtulOpenbravoFixed in Version3.0PR15Q2 =>
2015-01-21 03:52AtulOpenbravoStatusnew => scheduled
2015-01-21 04:03hgbotCheckin
2015-01-21 04:03hgbotNote Added: 0073670
2015-01-21 04:03hgbotStatusscheduled => resolved
2015-01-21 04:03hgbotResolutionopen => fixed
2015-01-21 04:03hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/devel/pi/rev/9c12d1443b200650ca55861bb6377d2b1d8e964c [^] => http://code.openbravo.com/erp/devel/pi/rev/dad8f2ef7d46cb0d4235c77a45842dc1a9ba9375 [^]
2015-01-21 09:13SandrahuguetFile Deleted: bug_28591.diff
2015-01-21 10:06hgbotCheckin
2015-01-21 10:06hgbotNote Added: 0073677
2015-01-21 10:22SandrahuguetNote Added: 0073678
2015-01-21 10:22SandrahuguetStatusresolved => closed
2015-01-21 10:22SandrahuguetFixed in Version => 3.0PR15Q2
2015-02-04 20:34hudsonbotCheckin
2015-02-04 20:34hudsonbotNote Added: 0074166
2015-02-04 20:34hudsonbotCheckin
2015-02-04 20:34hudsonbotNote Added: 0074168
2015-02-04 20:34hudsonbotCheckin
2015-02-04 20:34hudsonbotNote Added: 0074170
2015-02-10 19:40ngarciaRelationship addedrelated to 0028912
2015-02-23 05:11hudsonbotCheckin
2015-02-23 05:11hudsonbotNote Added: 0074752
2015-02-23 05:11hudsonbotCheckin
2015-02-23 05:11hudsonbotNote Added: 0074753
2015-02-23 05:11hudsonbotCheckin
2015-02-23 05:11hudsonbotNote Added: 0074760
2015-02-23 05:11hudsonbotCheckin
2015-02-23 05:11hudsonbotNote Added: 0074761
2015-09-01 00:03hudsonbotCheckin
2015-09-01 00:03hudsonbotNote Added: 0079966
2015-09-01 00:03hudsonbotCheckin
2015-09-01 00:03hudsonbotNote Added: 0079967
2016-02-02 09:34hgbotCheckin
2016-02-02 09:34hgbotNote Added: 0083805

Notes
(0073482)
AtulOpenbravo   
2015-01-13 20:33   
Test Plan
- Enable the Reverse Payment functionality as described in the following wiki document:
http://wiki.openbravo.com/wiki/Projects:Reversepayment/User_Manual [^] [^]

1. Create 2 Sales Invoices using any business partner and product.
2. Pay both invoices in the same Payment In.
3. Reverse the created payment.
4. Go to one of the invoices and open the add payment pick and execute.
5. Check that the outstanding amount is the expected payment amount.
(0073655)
hgbot   
2015-01-20 18:35   
Repository: erp/devel/pi
Changeset: 9c12d1443b200650ca55861bb6377d2b1d8e964c
Author: Atul Gaware <atul.gaware <at> openbravo.com>
Date: Tue Jan 20 15:42:13 2015 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/9c12d1443b200650ca55861bb6377d2b1d8e964c [^]

Fixes Issue 28591:Reversing a payment is creating wrong invoice payment plan
details

Problem is Schedule detail was being updated by payment amount now it is taken
care to update using individual schedule detail amount while creating reverse
payment schedule and payment schedule details.

---
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/process/FIN_PaymentProcess.java
A modules/org.openbravo.advpaymentmngt/build/classes/org/openbravo/advpaymentmngt/modulescript/Issue28591UpdatePSD.class
A modules/org.openbravo.advpaymentmngt/build/classes/org/openbravo/advpaymentmngt/modulescript/Issue28591UpdatePSDData.class
A modules/org.openbravo.advpaymentmngt/src-util/modulescript/src/org/openbravo/advpaymentmngt/modulescript/Issue28591UpdatePSD.java
A modules/org.openbravo.advpaymentmngt/src-util/modulescript/src/org/openbravo/advpaymentmngt/modulescript/Issue28591UpdatePSD_data.xsql
A modules/org.openbravo.advpaymentmngt/src-util/modulescript/src/src/org/openbravo/advpaymentmngt/modulescript/Issue28591UpdatePSDData.java
---
(0073656)
Sandrahuguet   
2015-01-20 18:36   
Code review + testing OK
(0073669)
AtulOpenbravo   
2015-01-21 03:52   
Re-opened due to wrong fix pushed which fails int-inc-oracle try test.
(0073670)
hgbot   
2015-01-21 04:03   
Repository: erp/devel/pi
Changeset: dad8f2ef7d46cb0d4235c77a45842dc1a9ba9375
Author: Atul Gaware <atul.gaware <at> openbravo.com>
Date: Tue Jan 20 14:57:35 2015 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/dad8f2ef7d46cb0d4235c77a45842dc1a9ba9375 [^]

Fixes Issue 28591:Reversing a payment is creating wrong invoice payment plan
details

Problem is Schedule detail was being updated by payment amount now it is taken
care to update using individual schedule detail amount while creating reverse
payment schedule and payment schedule details. Modulescript to fix errorneous data
consists of update invoice paid, outstanding amt, update payment schedule received
and outstanding amount, update payment credit generated info if in case required.

---
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/process/FIN_PaymentProcess.java
A modules/org.openbravo.advpaymentmngt/build/classes/org/openbravo/advpaymentmngt/modulescript/Issue28591UpdatePSD.class
A modules/org.openbravo.advpaymentmngt/build/classes/org/openbravo/advpaymentmngt/modulescript/Issue28591UpdatePSDData.class
A modules/org.openbravo.advpaymentmngt/src-util/modulescript/src/org/openbravo/advpaymentmngt/modulescript/Issue28591UpdatePSD.java
A modules/org.openbravo.advpaymentmngt/src-util/modulescript/src/org/openbravo/advpaymentmngt/modulescript/Issue28591UpdatePSD_data.xsql
A modules/org.openbravo.advpaymentmngt/src-util/modulescript/src/src/org/openbravo/advpaymentmngt/modulescript/Issue28591UpdatePSDData.java
---
(0073677)
hgbot   
2015-01-21 10:06   
Repository: erp/devel/pi
Changeset: 235d47fa148da3752e54dccbef539650710464df
Author: Atul Gaware <atul.gaware <at> openbravo.com>
Date: Wed Jan 21 13:29:26 2015 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/235d47fa148da3752e54dccbef539650710464df [^]

Related to Issue 28591:Removed unwanted code from the modulescript to update
payment plan and payment plan details

---
M modules/org.openbravo.advpaymentmngt/build/classes/org/openbravo/advpaymentmngt/modulescript/Issue28591UpdatePSD.class
M modules/org.openbravo.advpaymentmngt/build/classes/org/openbravo/advpaymentmngt/modulescript/Issue28591UpdatePSDData.class
M modules/org.openbravo.advpaymentmngt/src-util/modulescript/src/org/openbravo/advpaymentmngt/modulescript/Issue28591UpdatePSD.java
M modules/org.openbravo.advpaymentmngt/src-util/modulescript/src/org/openbravo/advpaymentmngt/modulescript/Issue28591UpdatePSD_data.xsql
M modules/org.openbravo.advpaymentmngt/src-util/modulescript/src/src/org/openbravo/advpaymentmngt/modulescript/Issue28591UpdatePSDData.java
---
(0073678)
Sandrahuguet   
2015-01-21 10:22   
Code review + testing OK
(0074166)
hudsonbot   
2015-02-04 20:34   
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/f36c91d0ad63 [^]
Maturity status: Test
(0074168)
hudsonbot   
2015-02-04 20:34   
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/f36c91d0ad63 [^]
Maturity status: Test
(0074170)
hudsonbot   
2015-02-04 20:34   
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/f36c91d0ad63 [^]
Maturity status: Test
(0074752)
hudsonbot   
2015-02-23 05:11   
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/6856352d7235 [^]
Maturity status: Test
(0074753)
hudsonbot   
2015-02-23 05:11   
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/6856352d7235 [^]
Maturity status: Test
(0074760)
hudsonbot   
2015-02-23 05:11   
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/6856352d7235 [^]
Maturity status: Test
(0074761)
hudsonbot   
2015-02-23 05:11   
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/6856352d7235 [^]
Maturity status: Test
(0079966)
hudsonbot   
2015-09-01 00:03   
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/880a20775a45 [^]
Maturity status: Test
(0079967)
hudsonbot   
2015-09-01 00:03   
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/880a20775a45 [^]
Maturity status: Test
(0083805)
hgbot   
2016-02-02 09:34   
Repository: tools/automation/int
Changeset: 266ab2fdd707a7a201eae0c7f17e4d4db3d7081c
Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com>
Date: Fri Sep 25 09:31:29 2015 +0200
URL: http://code.openbravo.com/tools/automation/int/rev/266ab2fdd707a7a201eae0c7f17e4d4db3d7081c [^]

Related to issue 28591: Added some automated tests

---
M src-test/com/openbravo/test/integration/erp/gui/financial/receivablespayables/transactions/AddPaymentProcess.java
M src-test/com/openbravo/test/integration/erp/gui/financial/receivablespayables/transactions/paymentin/PaymentInTab.java
M src-test/com/openbravo/test/integration/erp/gui/financial/receivablespayables/transactions/paymentout/PaymentOutTab.java
M src-test/com/openbravo/test/integration/erp/gui/popups/RequestProcessPopUp.java
M src-test/com/openbravo/test/integration/erp/modules/client/application/gui/GeneratedTab.java
M src-test/com/openbravo/test/integration/erp/modules/functionalteam/addpayment/testsuites/APR_Regressions/RegressionSuite.java
M src-test/com/openbravo/test/integration/erp/modules/functionalteam/addpayment/testsuites/APR_Regressions/build.xml
M src-test/com/openbravo/test/integration/erp/testscripts/financial/receivablespayables/transactions/PaymentIn.java
M src-test/com/openbravo/test/integration/erp/testscripts/financial/receivablespayables/transactions/PaymentOut.java
A src-test/com/openbravo/test/integration/erp/modules/functionalteam/addpayment/testsuites/APR_Regressions/APRRegression018.java
A src-test/com/openbravo/test/integration/erp/modules/functionalteam/addpayment/testsuites/APR_Regressions/APRRegression019.java
---