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

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0030077
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] Z. Othersmajoralways2015-06-02 16:482015-08-20 23:16
ReportermaiteView Statuspublic 
Assigned Toaferraz 
PriorityurgentResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revision9d85d9a430b2
ProjectionnoneETAnoneTarget Version3.0PR15Q4
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Tovmromanos
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0030077: Customer Balance not working properly with multicurrency credit

DescriptionCustomer Balance not working properly with multicurrency credit
Steps To Reproduce1. Create new Business Partner defined as Customer. Set pricelist, payemnt method, financial account...
2. Register new Payment In for this BP. Set amount=100 and process payment generating credit
3. Access Business Partner window and realize that Customer Balance is -100 (which is correct)
4. From Business Partner window, Run "Set new currency" process, to convert your credit to -50 MXN
5. Access Business Partner Financial Account and configure Payment Method as "receive payments in multiple currencies"
6. Access Payment In window and create new record for same business partner. Set Currency=MXN and any any conversion rate. Run Add details process, select pending credit and use it against a gl item. Process payment
7. Access Business Partner window and realize that Customer Balance is not 0.00, when you are trying to consume all already converted credit
TagsNo tags attached.
Attached Files

- Relationships [ Relation Graph ] [ Dependency Graph ]
related to defect 00301343.0PR15Q3 closedNaroaIriarte Error when executing a Process Definition if a parameter has defined a numeric default value 
depends on defect 00300953.0PR15Q3 closedaferraz BP current balance not properly updated when generating credit from Payment In/Out 
depends on defect 00301383.0PR15Q3 closedaferraz Error in Add Payment when generating credit and adding a negative line 
related to defect 00324343.0PR16Q2 closedmarkmm82 Wrong Exchange Rate set in Payment created from "Set New currency" process, when converting existent credit 

-  Notes
(0078206)
aferraz (manager)
2015-06-11 11:26
edited on: 2015-07-03 14:05

 Test plan:

- Create a new Business Partner and configure it as Customer and also as Vendor with EUR currency.

- Configure "Transferencia" Payment Method in "Cuenta de Banco" Financial Account as "Receive Payments in Multiple Currencies" = Y and "Make Payments in Multiple Currencies" = Y

- Create a Payment In for created business partner generating 100 EUR of credit.

- Realize current balance of business partner is now -100 EUR

- Create a Payment Out for created business partner generating 100 EUR of credit.

- Realize current balance of business partner is now 0 EUR

- Click on Set New Currency in Business Partner window
  Realize a Warning message is shown
  Set Currency = MXN, Set Amount = Y, Foreign Amount = -50 and G/L Item = Capital Social
  Click on Done
  Realize an Error message is shown

- Create a Payment In for created business partner generating 100 EUR of credit.

- Realize current balance of business partner is now -100 EUR

- Click on Set New Currency in Business Partner window
  Realize a Warning message is shown
  Set Currency = MXN, Set Amount = Y, Foreign Amount = -50 and G/L Item = Capital Social
  Click on Done
  Realize "Currency updated Successfully" message is shown.

- Go to Payment In window
  Realize two created payments with credit have been converted to new business partner currency following the wiki documentation

- Go to Payment Out window
  Realize created payment with credit has been converted to new business partner currency following the wiki documentation

- Realize current balance of business partner is now -50 MXN

- Create a Payment In for created business partner, refunding 50 MXN of credit.

- Realize current balance of business partner is now 0 MXN

- Click on Set New Currency in Business Partner window
  Realize a Warning message is shown
  Set Currency = EUR, Set Amount = N, Use default conversion rate = N, Rate = 2 and G/L Item empty
  Realize Done button is disabled
(0078660)
hgbot (developer)
2015-07-06 11:17

 Repository: erp/devel/pi
Changeset: 9d85d9a430b2eb9bad6881c4525a144133175941
Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com>
Date: Thu Jun 11 12:43:23 2015 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/9d85d9a430b2eb9bad6881c4525a144133175941 [^]

Fixes issue 30077: Wrong customer balance with multicurrency credits

Customer balance was not correctly updated when consuming credit generated in one currency in a payment with another currency.
Credit To Use grid in AddPayment will only show available credit in the currency of current payment.
Also, when changing the currency of a business partner with Set New Currency process, if exists available credit, the process will be able to convert automatically this credit into credit in the new curreny if a GLItem is selected, following this process: http://wiki.openbravo.com/wiki/How_to_convert_credit_to_a_different_currency [^]

---
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/dao/AdvPaymentMngtDao.java
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/filterexpression/PaymentInAddPaymentDisplayLogics.java
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/filterexpression/PaymentOutAddPaymentDisplayLogics.java
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/filterexpression/PurchaseInvoiceAddPaymentDisplayLogics.java
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/filterexpression/PurchaseOrderAddPaymentDisplayLogics.java
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/filterexpression/SalesInvoiceAddPaymentDisplayLogics.java
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/filterexpression/SalesOrderAddPaymentDisplayLogics.java
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/filterexpression/TransactionAddPaymentDisplayLogics.java
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/hqlinjections/CreditToUseTransformer.java
M modules/org.openbravo.client.application/src/org/openbravo/client/application/ApplicationComponentProvider.java
M src-db/database/sourcedata/AD_MESSAGE.xml
M src-db/database/sourcedata/OBUIAPP_PARAMETER.xml
M src-db/database/sourcedata/OBUIAPP_PROCESS.xml
M src/org/openbravo/common/actionhandler/SetNewBPCurrency.java
M src/org/openbravo/financial/FinancialUtils.java
A src/org/openbravo/common/actionhandler/CheckAvailableCreditActionHandler.java
A web/js/checkAvailableCredit.js
---
(0078661)
hgbot (developer)
2015-07-06 11:17

 Repository: erp/devel/pi
Changeset: 317d186e5a3141b3b24deefdd439a0896bf6d578
Author: Víctor Martínez Romanos <victor.martinez <at> openbravo.com>
Date: Fri Jul 03 13:33:24 2015 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/317d186e5a3141b3b24deefdd439a0896bf6d578 [^]

Related to issue 30077: applied js beautifier

---
M web/js/checkAvailableCredit.js
---
(0078662)
hgbot (developer)
2015-07-06 11:17

 Repository: erp/devel/pi
Changeset: b815e49be324a0464825ed8e48ca2b3599ef247d
Author: Víctor Martínez Romanos <victor.martinez <at> openbravo.com>
Date: Fri Jul 03 15:04:14 2015 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/b815e49be324a0464825ed8e48ca2b3599ef247d [^]

Related to issue 30077: code review improvements

Use credit popup when completing an invoice only shown when invoice's currency = business partner's currency.
Call getCustomerPaymentsWithCredit() without checking currency != null, because the method already checks it.
Improved warning message with a link to the wiki page.
Amounts are properly rounded.
Improvements in the created payment messages.
Change Business Partner currency: Forced to select a gl item if credit is detected in order to change the currency.

---
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/ad_actionbutton/ProcessInvoice.java
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/dao/AdvPaymentMngtDao.java
M src-db/database/sourcedata/AD_MESSAGE.xml
M src/org/openbravo/common/actionhandler/SetNewBPCurrency.java
M web/js/checkAvailableCredit.js
---
(0078663)
vmromanos (manager)
2015-07-06 11:17

 Code review + testing OK
(0079468)
hudsonbot (developer)
2015-08-20 23:16

 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/8c91718397a5 [^]
Maturity status: Test
(0079469)
hudsonbot (developer)
2015-08-20 23:16

 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/8c91718397a5 [^]
Maturity status: Test
(0079470)
hudsonbot (developer)
2015-08-20 23:16

 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/8c91718397a5 [^]
Maturity status: Test

- Issue History
Date Modified Username Field Change
2015-06-02 16:48 maite New Issue
2015-06-02 16:48 maite Assigned To => Triage Finance
2015-06-02 16:48 maite Modules => Core
2015-06-02 16:48 maite Resolution time => 1436133600
2015-06-02 16:48 maite Triggers an Emergency Pack => No
2015-06-02 16:48 maite Issue Monitored: networkb
2015-06-02 16:49 maite Steps to Reproduce Updated View Revisions
2015-06-04 09:20 aferraz Assigned To Triage Finance => aferraz
2015-06-04 14:14 aferraz Relationship added depends on 0030095
2015-06-08 12:32 aferraz Status new => scheduled
2015-06-09 12:52 aferraz Relationship added related to 0030134
2015-06-09 13:41 aferraz Relationship added depends on 0030138
2015-06-11 11:26 aferraz Note Added: 0078206
2015-06-11 12:23 aferraz Note Edited: 0078206 View Revisions
2015-06-12 12:32 ngarcia Issue Monitored: ngarcia
2015-06-29 12:45 egoitz Target Version 3.0PR15Q3 => 3.0PR15Q4
2015-07-03 14:05 vmromanos Note Edited: 0078206 View Revisions
2015-07-06 11:17 hgbot Checkin
2015-07-06 11:17 hgbot Note Added: 0078660
2015-07-06 11:17 hgbot Status scheduled => resolved
2015-07-06 11:17 hgbot Resolution open => fixed
2015-07-06 11:17 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/9d85d9a430b2eb9bad6881c4525a144133175941 [^]
2015-07-06 11:17 hgbot Checkin
2015-07-06 11:17 hgbot Note Added: 0078661
2015-07-06 11:17 hgbot Checkin
2015-07-06 11:17 hgbot Note Added: 0078662
2015-07-06 11:17 vmromanos Review Assigned To => vmromanos
2015-07-06 11:17 vmromanos Note Added: 0078663
2015-07-06 11:17 vmromanos Status resolved => closed
2015-08-20 23:16 hudsonbot Checkin
2015-08-20 23:16 hudsonbot Note Added: 0079468
2015-08-20 23:16 hudsonbot Checkin
2015-08-20 23:16 hudsonbot Note Added: 0079469
2015-08-20 23:16 hudsonbot Checkin
2015-08-20 23:16 hudsonbot Note Added: 0079470
2016-03-08 13:09 maite Relationship added related to 0032434

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker