-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123154/
-----------------------------------------------------------

Review request for KMymoney.


Bugs: 345550
    http://bugs.kde.org/show_bug.cgi?id=345550


Repository: kmymoney


Description
-------

Since commit 9fc583d3db4fe2c16448790d7ee17145fbe8bd2f was added as a
fix for some currency conversions performed in reports income/expense
transactions which involved a currency conversion were not properly
displayed in the transactions by account report.

Extended the multi-currency report testcase to catch this error and
fixed the problem by resurrecting some conversion code which was
removed by that commit, but moved it to a place where it's applied
only to income/expense transactions.


Diffs
-----

  kmymoney/reports/querytable.cpp 974668a7923ee69cd88b63c15f6c061338916aa2 
  kmymoney/reports/querytabletest.cpp 008ccb724b5e35156100bd249f17d6d4d602cdee 

Diff: https://git.reviewboard.kde.org/r/123154/diff/


Testing
-------

Succesfully ran the newly added testcase and opened the file attached to BUG 
345550, everything seems to work as it should now.


Thanks,

Cristian Oneț

_______________________________________________
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Reply via email to