Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[10.0] Balance sheet is wrong for accounts in foreign currencies #73

Closed
fdegrave opened this issue Oct 31, 2018 · 2 comments
Closed

[10.0] Balance sheet is wrong for accounts in foreign currencies #73

fdegrave opened this issue Oct 31, 2018 · 2 comments

Comments

@fdegrave
Copy link

I don't know if I miss something, but it seems the balance of accounts in foreign currencies is wrong in the balance sheet. Indeed, it computes the sum of the transactions in euro, rather than the sum of all the transactions in currency, then converted into euros.

Here is a simple example: let's say I have a bank account in USD, and let's say I buy 100 USD when the rate is 1USD = 1 EUR. The credit / debit of the move lines are in EUR, 100 each (and the balance is 100 EUR on the 55 account); and the amount_currency is 100 USD.

Let us say that a few months later, the rate is 1USD = 0.7 EUR. Since I own 100 USD, the asset in the balance sheet (= the balance of the 55 account in USD) should be converted into its current value in EUR, thus 70 (= sum of the "amount_currency" fields, converted with the current rate). But currently it gives 100 EUR (= sum of the "balance" fields, already in EUR).

@sbidoul
Copy link
Member

sbidoul commented Oct 31, 2018

I assume you refer to l10n_be_mis_reports?

Yes this is normal. If mis builder would attempt to do currency conversion for such accounts, the resulting balance would not be... balanced anymore.

Foreign currency revaluations must be posted explicitly (with an unrealized gain loss counterpart). The good news is that there is an excellent OCA module for that: https://github.com/OCA/account-closing/tree/11.0/account_multicurrency_revaluation.

@fdegrave
Copy link
Author

Indeed I refer to l10n_be_mis_reports, sorry for the omission.

And yes, you are right; now I need to explain this to my CFO ;-) I will have a thorough look at the module you suggest. Thanks a lot!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants