Summary: | Balance column sums different currencies without converting on reports with unchecked "convert to base currency" | ||
---|---|---|---|
Product: | [Applications] kmymoney | Reporter: | Tristan Grimaux <info> |
Component: | general | Assignee: | KMyMoney Devel Mailing List <kmymoney-devel> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | asoliverez, onet.cristian |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | unspecified | ||
OS: | Linux | ||
Latest Commit: | http://commits.kde.org/kmymoney/9fc583d3db4fe2c16448790d7ee17145fbe8bd2f | Version Fixed In: | |
Attachments: | This patch contains a test case for this problem which can be committed once the bug is fixed |
Description
Tristan Grimaux
2012-03-08 14:25:24 UTC
Created attachment 69387 [details]
This patch contains a test case for this problem which can be committed once the bug is fixed
I couldn't fix the problem - reports code is too complex for me but I came up with a test case for it to make it easier to develop a fix.
The problem with both comments above is that the report by account should show all transactions in the currency of that account. So, transaction in local account will be in local currency, and transactions in foreign account will be in foreign account. The test is wrong in that regard, too. I'm working on this. It's not an easy fix though. Git commit 9fc583d3db4fe2c16448790d7ee17145fbe8bd2f by Alvaro Soliverez. Committed on 01/05/2012 at 00:33. Pushed by asoliverez into branch 'master'. Multi-currency transaction reports: each individual should be shown in its own currency, not the one of the main split. Fixed the display of currency for those cases when the currency is different from the base currency. Added test for Transaction by Account report with multiple currencies, contributed by Cristian Onet. There might be some more fixing to do, as it's one of the most complex reports. Related: bug 297437 REVIEW:104774 M +18 -10 kmymoney/reports/querytable.cpp M +79 -0 kmymoney/reports/querytabletest.cpp M +1 -0 kmymoney/reports/querytabletest.h http://commits.kde.org/kmymoney/9fc583d3db4fe2c16448790d7ee17145fbe8bd2f |