Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0029620Openbravo ERPZ. Otherspublic2015-04-16 18:232015-04-20 14:20
Reporterngarcia 
Assigned Toumartirena 
PriorityimmediateSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target Version3.0PR15Q1.4Fixed in Version 
Merge Request Status
Review Assigned Tovmromanos
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression levelProduction - Confirmed Stable
Regression date2014-06-19
Regression introduced in release3.0PR14Q3
Regression introduced by commithttps://code.openbravo.com/erp/devel/pi/rev/f94f08e02e3a [^]
Triggers an Emergency PackYes
Summary0029620: Wrong amount in Used Credit Source tab when paying an invoice with credit and refunding the difference to the customer
DescriptionWrong amount in Used Credit Source tab when paying an invoice with credit and refunding the difference to the customer
Steps To ReproduceAs group admin role:
   Create a Payment In where you leave 100 EUR as credit to a Business Partner
   Create a Sales Invoice of 75 EUR for the same Business Partner
   Create a new Payment In, select the credit to use, the invoice to pay and Refund the amount to the Customer
   Check two payments are created:
      One with the refund that do not have any record in the Used Credit tab
      One with the invoice payment that has a record of 100 EUR in the Used Credit Source tab
   Post this last payment and check the difference of 25 is posted to Currency balancing account
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
blocks defect 0029615 closed umartirena Wrong amount in Used Credit Source tab when paying an invoice with credit and refunding the difference to the customer 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2015-04-16 19:38umartirenaTypedefect => backport
2015-04-16 19:38umartirenaTarget Version => 3.0PR15Q1.3
2015-04-17 14:25umartirenaAssigned ToTriage Finance => umartirena
2015-04-17 14:47umartirenaTarget Version3.0PR15Q1.3 => 3.0PR15Q1.4
2015-04-20 14:19hgbotCheckin
2015-04-20 14:19hgbotNote Added: 0076628
2015-04-20 14:19hgbotStatusscheduled => resolved
2015-04-20 14:19hgbotResolutionopen => fixed
2015-04-20 14:19hgbotFixed in SCM revision => http://code.openbravo.com/erp/backports/3.0PR15Q1.4/rev/8bace9b56a9b0a1e145015e99ec5043967d75af9 [^]
2015-04-20 14:20vmromanosReview Assigned To => vmromanos
2015-04-20 14:20vmromanosNote Added: 0076630
2015-04-20 14:20vmromanosStatusresolved => closed

Notes
(0076628)
hgbot   
2015-04-20 14:19   
Repository: erp/backports/3.0PR15Q1.4
Changeset: 8bace9b56a9b0a1e145015e99ec5043967d75af9
Author: Unai Martirena <unai.martirena <at> openbravo.com>
Date: Mon Apr 20 09:49:30 2015 +0200
URL: http://code.openbravo.com/erp/backports/3.0PR15Q1.4/rev/8bace9b56a9b0a1e145015e99ec5043967d75af9 [^]

Fixes bug 29620: Correctly used credit when paying an invoice and refunding

There were two issues while managing this scenario:

[1] In 'addCredit' method inside AddPaymentActionHandler.java class, a loop of Credit to Use grid is done, creating Payment Credit Used records assigned to the new created payment of the invoice. This method was not taking into account that could be credit amount that exceeds the invoice amount (amount to be refunded) and it was assigning all the credit amount as used to the payment of the invoice. So the refund payment that is created right after was been left without any used credit payment.

[2] After fixing the previous method, the second issue appears. The 'updateUsedCredit' method inside FIN_PaymentProcess.java class retrieves all the payments of the business partner that has generated credit, and it assigns as used to the refund payment. The problem with this function is that before the Add Payment refactor it was not possible to select which credit payments we wanted to consume, it was only a total credit amount to be consumed and the system behind was being consuming any of them. But after the refactor it is possible to select the ones that want to be consumed.

In order to fix this last issue the selected credit line ids are added as a parameter to 'updateUsedCredit' method.

---
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/actionHandler/AddPaymentActionHandler.java
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/process/FIN_AddPayment.java
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/process/FIN_PaymentProcess.java
---
(0076630)
vmromanos   
2015-04-20 14:20   
Code review + testing OK