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

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0035689
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] 09. Financial managementmajoralways2017-04-04 16:522017-05-29 17:43
ReportermaiteView Statuspublic 
Assigned Tocollazoandy4 
PrioritynormalResolutionfixedFixed in Version3.0PR17Q3
StatusclosedFix in branchFixed in SCM revisione16423fae9e6
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Toaferraz
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0035689: "Bank Revaluation" accounts defined in Financial Account | Accounting Configuration tab not used

DescriptionInstead of using specific account defined in Financial Account | Accounting Configuration tab, value configured in General Ledger Configuration | Defaults tab is used
Steps To Reproduce0. Access to General Ledger Configuration "F&B España, S.A. US/A/Euro" and set "Bank Revaluation Gain" = 10000 - Capital social ; "Bank Revaluation Loss"= 10100 - Fondo social
1. Access to "Cuenta de Banco - EUR" financial account and define "Transferencia" payment method as "allow payment in multiple currencies" and "In Transit Payment Account" as Upon Payment Use account
2. Verify that proper Bank Revaluation accounts are defined in Accounting Configuration tab for Cuenta de Banco financial account
3. Create new Price List for "F&B España, S.A" organization, "MXN" currency. Add price list version and add "Agua sin Gas 1L" product in Product Price setting price=5.00
4. Register new Purchase Invoice for "F&B España - Región Norte" organization, "Bebidas Alegres, S.L." business aprtner, invoice date=04/04/2017, using previous pricelist and add line for "Agua sin Gas 1L" product. Define conversion in Exchange Rates: from MXN to EUR as rate=2; from MXN to USD as rate=3. Process invoice
5. Register new Payment Out for same org, business partner, payment date= 05/04/2017, "Cuenta de Banco - EUR" financial account, "Transferencia" payment method, currency=MXN and Exchange rate =8. Add previous invoice and process
6. Post payment out and realize that instead of using "Bank Revaluation Loss" account defined in Financial Account | Accounting Configuration tab, the one defined in General Ledger Configuration | Defaults tab is used
TagsNo tags attached.
Attached Files

- Relationships [ Relation Graph ] [ Dependency Graph ]

-  Notes
(0096360)
collazoandy4 (reporter)
2017-05-03 23:26

 Test Plan
  As group admin role
    Go to General Ledger Configuration window
      Select F&B España, S.A. US/A/Euro record
        In The default tab edit the record and set "Bank Revaluation Gain" = 10000 - Capital social and "Bank Revaluation Loss"= 10100 - Fondo social
    Go to Financial Account window and select Cuenta de Banco - EUR record:
      Edit the Payment Method Transferencia and check Make Payments in Multiple Currencies option and In Transit Payment Account as Upon Payment Use account
    Go to Price List window and create a new record:
      Name: Issue Price List
      Currency: MXN
      Create a new record in Price List Version tab:
        Valid From Date: 01-01-2017
      In Product Price tab create a new record:
        Product: Agua sin Gas 1L
        Unit / List Price: 5
    Go to Purchase Invoice window and create a new record
      Business Partner: Bebidas Alegres, S.L.
      Price List: Issue Price List
      Create a new record in Lines tab:
        Product: Agua sin Gas 1L
      Create a new record in Exchange Rates tab:
        To Currency: EUR
        Rate: 2
      Create a new record in Exchange Rates tab:
        To Currency: USD
        Rate: 3
    Process the invoice
    Go to Payment Out and create a new record:
      Paying To: Bebidas Alegres, S.L.
      Currency: MXN
      Exchange Rate: 8
      Clic on Add Details and select previous invoice and process
      In Exchange Rate Tab create a new record:
        To Currency: USD
        Rate: 9
    Post Payment out and check that Bank Revaluation Loss account defined in Financial Account | Accounting Configuration tab is used
(0096380)
hgbot (developer)
2017-05-04 13:13

 Repository: erp/devel/pi
Changeset: e16423fae9e6882bd819da729263d4714848e3cd
Author: Armaignac <collazoandy4 <at> gmail.com>
Date: Wed May 03 17:18:51 2017 -0400
URL: http://code.openbravo.com/erp/devel/pi/rev/e16423fae9e6882bd819da729263d4714848e3cd [^]

Fixes issue 35689: Bank Revaluation Loss account defined in FA is not used

The Bank Revaluation Loss account defined in Financial Account | Accounting Configuration
tab is not used in the posting process of payment out document, the general ledger
configuration was used

A new field was added to the AcctServer to get the Financial Account ID when searching
in the account, the Account type in getAccount was change to the ACCTTYPE_ConvertChargeGainAmt
and ACCTTYPE_ConvertChargeLossAmt and also the selectConvertChargeLossAmt and selectConvertChargeGainAmt
methods was changed to get the Bank Account from financial account configuration

---
M src/org/openbravo/erpCommon/ad_forms/AcctServer.java
M src/org/openbravo/erpCommon/ad_forms/AcctServer_data.xsql
M src/org/openbravo/erpCommon/ad_forms/DocFINPayment.java
---
(0096382)
hgbot (developer)
2017-05-04 13:52

 Repository: erp/devel/pi
Changeset: 7fb423e40bc107c6c179c4dd39ad3fb1cb6d9520
Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com>
Date: Thu May 04 13:51:15 2017 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/7fb423e40bc107c6c179c4dd39ad3fb1cb6d9520 [^]

Related to issue 35689: Code review improvements

Set FIN_Financial_Account_ID also in DocFINFinAccTransaction.java.

---
M src/org/openbravo/erpCommon/ad_forms/DocFINFinAccTransaction.java
---
(0096507)
hgbot (developer)
2017-05-10 13:44

 Repository: erp/devel/pi
Changeset: 4147730ef05b9a945d6a06b52304524234a20ba4
Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com>
Date: Wed May 10 13:43:26 2017 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/4147730ef05b9a945d6a06b52304524234a20ba4 [^]

Related to issue 35689: Code review improvements

Set FIN_Financial_Account_ID also in DocFINReconciliation.java.

---
M src/org/openbravo/erpCommon/ad_forms/DocFINReconciliation.java
---
(0096508)
aferraz (manager)
2017-05-10 13:45

 Code review + Testing OK
(0096937)
hudsonbot (developer)
2017-05-29 17:43

 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/1ee70113bdc4 [^]
Maturity status: Test
(0096938)
hudsonbot (developer)
2017-05-29 17:43

 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/1ee70113bdc4 [^]
Maturity status: Test
(0096953)
hudsonbot (developer)
2017-05-29 17:43

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

- Issue History
Date Modified Username Field Change
2017-04-04 16:52 maite New Issue
2017-04-04 16:52 maite Assigned To => Triage Finance
2017-04-04 16:52 maite Modules => Core
2017-04-04 16:52 maite Resolution time => 1493071200
2017-04-04 16:52 maite Triggers an Emergency Pack => No
2017-04-04 16:52 maite Issue Monitored: networkb
2017-04-04 19:41 aferraz Assigned To Triage Finance => collazoandy4
2017-05-02 15:35 collazoandy4 Status new => scheduled
2017-05-03 23:26 collazoandy4 Note Added: 0096360
2017-05-04 13:13 hgbot Checkin
2017-05-04 13:13 hgbot Note Added: 0096380
2017-05-04 13:13 hgbot Status scheduled => resolved
2017-05-04 13:13 hgbot Resolution open => fixed
2017-05-04 13:13 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/e16423fae9e6882bd819da729263d4714848e3cd [^]
2017-05-04 13:23 aferraz Review Assigned To => aferraz
2017-05-04 13:23 aferraz Note Added: 0096381
2017-05-04 13:23 aferraz Status resolved => closed
2017-05-04 13:23 aferraz Fixed in Version => 3.0PR17Q3
2017-05-04 13:51 aferraz Note Deleted: 0096381
2017-05-04 13:52 hgbot Checkin
2017-05-04 13:52 hgbot Note Added: 0096382
2017-05-04 13:52 aferraz Note Added: 0096383
2017-05-10 13:44 hgbot Checkin
2017-05-10 13:44 hgbot Note Added: 0096507
2017-05-10 13:45 aferraz Note Deleted: 0096383
2017-05-10 13:45 aferraz Note Added: 0096508
2017-05-29 17:43 hudsonbot Checkin
2017-05-29 17:43 hudsonbot Note Added: 0096937
2017-05-29 17:43 hudsonbot Checkin
2017-05-29 17:43 hudsonbot Note Added: 0096938
2017-05-29 17:43 hudsonbot Checkin
2017-05-29 17:43 hudsonbot Note Added: 0096953

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker