Bug 382245

Summary: Reports not showing correct totals for split transactions
Product: [Applications] kmymoney Reporter: Christopher Tanner <christann404>
Component: generalAssignee: KMyMoney Devel Mailing List <kmymoney-devel>
Status: RESOLVED FIXED    
Severity: major CC: koalinux, lukasz.wojnilowicz, ralf.habacker
Priority: NOR Keywords: regression
Version: unspecified   
Target Milestone: ---   
Platform: OpenMandriva   
OS: Linux   
See Also: https://bugs.kde.org/show_bug.cgi?id=395327
Latest Commit: Version Fixed In: 5.0.0
Attachments: Demo file to duplicate the problem on master
Report screenshot from recent 4.8

Description Christopher Tanner 2017-07-11 15:51:36 UTC
Running kmymoney vs 4.100.3474270. 

When I create a transaction report by category, entries which are splits show the amount from the master transaction, not the actual split amount.
Comment 1 Thomas Baumgart 2017-07-13 14:38:41 UTC
Created attachment 106591 [details]
Demo file to duplicate the problem on master

Using this demo file run a Transaction-Report by Category. It shows 1000 in both income categories but not amounts of the splits (which sum up to 1000). This used to work in the 4.8 branch (and still might - I have not tested it). Maybe someone can check this on a recent 4.8 version.
Comment 2 NSLW 2017-07-13 17:44:18 UTC
Git commit 467f0cc2d07104d4770868c8b8b3b6d16345700d by Łukasz Wojniłowicz.
Committed on 13/07/2017 at 17:43.
Pushed by wojnilowicz into branch 'master'.

Show correct totals for Transactions by Category
FIXED-IN:5.0
Signed-off-by: Łukasz Wojniłowicz <lukasz.wojnilowicz@gmail.com>

M  +9    -1    kmymoney/reports/querytable.cpp

https://commits.kde.org/kmymoney/467f0cc2d07104d4770868c8b8b3b6d16345700d
Comment 3 Christopher Tanner 2017-07-13 18:35:16 UTC
That fixed the problem. Thanks

Chris
Comment 4 NSLW 2017-07-13 19:12:36 UTC
(In reply to Christopher Tanner from comment #3)
> That fixed the problem. Thanks
> 
> Chris

No problem. Please report if something is not working as before. It's very needed. The best way is just like Thomas did it i.e. with .kmy test file attached.

Łukasz
Comment 5 Ralf Habacker 2017-07-14 06:24:13 UTC
Created attachment 106608 [details]
Report screenshot from recent 4.8

(In reply to Thomas Baumgart from comment #1)
> This used to work in the 4.8 branch (and still might - I have not tested
> it). Maybe someone can check this on a recent 4.8 version.

Screenshot from 4.8 has been appended
Comment 6 Thomas Baumgart 2017-07-30 11:52:34 UTC
*** Bug 380194 has been marked as a duplicate of this bug. ***