[kmymoney4] [Bug 373966] Incorrect rates used during a report

[Ralf Habacker]

https://bugs.kde.org/show_bug.cgi?id=373966

Ralf Habacker  changed:

   What|Removed |Added

 CC||ralf.habac...@freenet.de
   Version Fixed In||5.0.0

-- 
You are receiving this mail because:
You are watching all bug changes.

[kmymoney4] [Bug 373966] Incorrect rates used during a report

[NSLW]

https://bugs.kde.org/show_bug.cgi?id=373966

NSLW  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
  Latest Commit||https://commits.kde.org/kmy
   ||money/622dab3f94878c5015187
   ||95aa00c102d10562a1a
 Resolution|--- |FIXED

--- Comment #3 from NSLW  ---
Git commit 622dab3f94878c501518795aa00c102d10562a1a by Łukasz Wojniłowicz.
Committed on 17/02/2017 at 20:15.
Pushed by wojnilowicz into branch 'master'.

Revise investment transaction report

Summary:
Price of transaction should be taken from transaction and not from
stored prices.
Investment accounts should report asset split as main split for correct
amount value, as it contains fees.
Related: bug 365306
Signed-off-by: Łukasz Wojniłowicz 

Differential Revision: https://phabricator.kde.org/D4445

Signed-off-by: Łukasz Wojniłowicz 

M  +99   -45   kmymoney/reports/querytable.cpp
M  +7-7kmymoney/reports/tests/querytable-test.cpp
M  +1-1kmymoney/reports/tests/reportstestcommon.cpp

https://commits.kde.org/kmymoney/622dab3f94878c501518795aa00c102d10562a1a

-- 
You are receiving this mail because:
You are watching all bug changes.

[kmymoney4] [Bug 373966] Incorrect rates used during a report

[bugzilla_noreply]

https://bugs.kde.org/show_bug.cgi?id=373966

m...@azu.ca changed:

   What|Removed |Added

 CC||m...@azu.ca

--- Comment #2 from m...@azu.ca ---
All the reports appear to be doing a forex rate lookup based on the post date
of the transaction rather than the actual rate used. From looking at the
generated XML, the correct forex rate is stored in the split transaction (rate
= shares / value) but any fix would be complicated by the fact that this rate
may have nothing to do with the currently set base currency. One would
presumably have to look at the currency of the account on other side of the
split and if it is the same as the base currency then the (shares/value) can be
used as the forex rate for reporting.

-- 
You are receiving this mail because:
You are watching all bug changes.

[kmymoney4] [Bug 373966] Incorrect rates used during a report

[bugzilla_noreply]

https://bugs.kde.org/show_bug.cgi?id=373966

--- Comment #1 from m...@azu.ca ---
Created attachment 102979
  --> https://bugs.kde.org/attachment.cgi?id=102979=edit
GBP Account and EUR Category example to reproduce bug

I reproduced this on 4.8.0 (Ubuntu) so 4.7.2 (OSX) behaviour may be different.
Using the attached file, it can be seen that the ledger view is correct but the
"Transactions by Month" report is converting the 80 EUR transaction to 120 GBP
using the incorrect exchange rate on 20/12/2016. It is using the GBP/EUR
exchange rate entered for the second transaction (300GBP=200EUR) instead of the
rate entered for the first transaction (100GBP=80EUR).

-- 
You are receiving this mail because:
You are watching all bug changes.