Bug 365306

Summary: Investment Transactions report Price and Amount in error
Product: [Applications] kmymoney Reporter: Pete <peterzay1>
Component: generalAssignee: KMyMoney Devel Mailing List <kmymoney-devel>
Status: RESOLVED FIXED    
Severity: major CC: ralf.habacker
Priority: NOR    
Version: 4.8.0   
Target Milestone: ---   
Platform: Ubuntu   
OS: Linux   
Latest Commit: Version Fixed In: 5.0.0
Sentry Crash Report:

Description Pete 2016-07-10 01:21:51 UTC
The first buy price for each buy date is wrong.  Amount values are also wrong.  Total amount for each stock is consequently also wrong.

As a result, this report cannot be used to assess gains and losses for each security.  There does not appear to be any other way to get this info from KMM.


Reproducible: Always

Steps to Reproduce:
1.click on Reports icon in left vertical pane
2.expand 5. Investments
3.launch Investment Transactions Default Report


Actual Results:  
Incorrect Price and Amount values for many lines of data.  Commission handling appears to be confused.

Expected Results:  
Each line should be correct.  The Total should give the correct gain or loss for that security.
Comment 1 wojnilowicz 2017-02-17 20:34:56 UTC
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 373966
Signed-off-by: Łukasz Wojniłowicz <lukasz.wojnilowicz@gmail.com>

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

Signed-off-by: Łukasz Wojniłowicz <lukasz.wojnilowicz@gmail.com>

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

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