KMM version 4.8.2 for Windows Same as BUG reports 212097, 330944 and 382245. Reports not showing correct values for split transaction. Reports by Category total okay Reports by Month do not total properly - ignores slit categories
Hi Ralph, thanks for this bug report. I took a look at the mentioned bugs and found: > Same as BUG reports 212097, 330944 these bugs were caused by user configuration and are not a kmymoney app issue >and 382245 4.8.2 is not affected by this bug If your issue is still present, please provide more information to be able to reproduce this issue either by appending a simple kmymoney test file showing the issue (please create a related report and mark it as favorite) or by reporting the detailed steps starting from a newly created file.
Created attachment 113257 [details] E8FD0EDB59D8483FA3D70AB792FFA284.png I used actual data as there isn’t any personal info being shared. From another Bug report dialog, I understand this is the way the program is designed and not a bug. I find it very confusing as I only had AUTO&SUBS selected for the report. Not being an accountant, I thought there was a filter problem with the REPORT by MONTH. As you can see, there is a $245 difference between the two reports which is the TRAILER category amount. If you tell me this is normal report behavior, then cancel my bug report and I’ll remember not to rely on MONTH reports where splits are concerned. Report by month – only AUTO& SUBS categories selected [cid:image001.png@01D4031F.07B88630] Report by Category – only AUTO& SUBS categories selected [cid:image002.png@01D4031F.CB3D2AC0] Ralph Sent from Mail for Windows 10 ________________________________ From: Ralf Habacker <bugzilla_noreply@kde.org> Sent: Wednesday, June 13, 2018 1:02:31 PM To: ralph.goddard@outlook.com Subject: [kmymoney] [Bug 395327] Split entry reports https://bugs.kde.org/show_bug.cgi?id=395327 Ralf Habacker <ralf.habacker@freenet.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |ralf.habacker@freenet.de Status|UNCONFIRMED |NEEDSINFO Resolution|--- |WAITINGFORINFO --- Comment #1 from Ralf Habacker <ralf.habacker@freenet.de> --- Hi Ralph, thanks for this bug report. I took a look at the mentioned bugs and found: > Same as BUG reports 212097, 330944 these bugs were caused by user configuration and are not a kmymoney app issue >and 382245 4.8.2 is not affected by this bug If your issue is still present, please provide more information to be able to reproduce this issue either by appending a simple kmymoney test file showing the issue (please create a related report and mark it as favorite) or by reporting the detailed steps starting from a newly created file. -- You are receiving this mail because: You reported the bug.
Created attachment 113258 [details] 4549E1C87D754ACE8DE0553E03AAFA78.png
Created attachment 113260 [details] kmymoney test file I tried to reproduce the account/category structure and the transactions from the screenshots with the attached test file, but I did not see how the split transaction is structured. It would be helpful if you could adjust that in the test file
Created attachment 113261 [details] AED9FD841B9A4EC0BE2E6A4C3AF86FAD.png Hopefully, this is what you wanted? [cid:image001.png@01D4032C.266A6BE0] Ralph Sent from Mail for Windows 10 ________________________________ From: Ralf Habacker <bugzilla_noreply@kde.org> Sent: Wednesday, June 13, 2018 3:28:49 PM To: ralph.goddard@outlook.com Subject: [kmymoney] [Bug 395327] Split entry reports https://bugs.kde.org/show_bug.cgi?id=395327 --- Comment #4 from Ralf Habacker <ralf.habacker@freenet.de> --- Created attachment 113260 [details] --> https://bugs.kde.org/attachment.cgi?id=113260&action=edit kmymoney test file I tried to reproduce the account/category structure and the transactions from the screenshots with the attached test file, but I did not see how the split transaction is structured. It would be helpful if you could adjust that in the test file -- You are receiving this mail because: You reported the bug.
Created attachment 113262 [details] kmymoney test file (update) Thanks for this pointer - the updated test file now contains all required transactions. The transaction by category report generated with this file and disabled trailer and subs category only shows the transaction split for auto:maintenance, which looks like what is expected. In contrast, the monthly report with disabled trailer:maintenance category and sub categories contains all tranactions splits, which means that the category filter is ignored if a transaction contains a split. Compared to the first report this behavior looks inconsequent and therefore seems to be a bug.
see https://phabricator.kde.org/D13803 for a related review
Created attachment 113654 [details] test case (update 2) setup reports "category(without trailer)" and "month (without trailer)" showing the difference
KMymoney 5.0.2 is also affected by this bug and show the same issue
Git commit cc670b1a8733ab3f2b4c5640122d661390575c76 by Ralf Habacker. Committed on 20/09/2018 at 20:09. Pushed by habacker into branch '5.0'. Add test for MyMoneyTransactionFilter This test was created to check the basic functions of the transaction filter, but does not claim to be complete. It was written by me for 4.8 and ported from Thomas Baumgart to KF5. Test Plan: compiled and tested on linux Reviewers: wojnilowicz Signed-off-by: Ralf Habacker <ralf.habacker@freenet.de> Differential Revision: https://phabricator.kde.org/D15107 M +1 -1 kmymoney/mymoney/mymoneytransactionfilter.cpp M +4 -3 kmymoney/mymoney/mymoneytransactionfilter.h A +787 -0 kmymoney/mymoney/tests/mymoneytransactionfilter-test.cpp [License: GPL (v2+)] A +58 -0 kmymoney/mymoney/tests/mymoneytransactionfilter-test.h [License: GPL (v2+)] https://commits.kde.org/kmymoney/cc670b1a8733ab3f2b4c5640122d661390575c76
Git commit 4ed579e86be69ce5df04c03e4b9797108ec8cfa8 by Ralf Habacker. Committed on 20/09/2018 at 20:55. Pushed by habacker into branch '4.8'. Add MyMoneyTransactionFilterTest Signed-off-by: Ralf Habacker <ralf.habacker@freenet.de> Differential Revision: https://phabricator.kde.org/D15010 M +6 -0 kmymoney/mymoney/CMakeLists.txt M +1 -1 kmymoney/mymoney/mymoneytransactionfilter.cpp M +3 -2 kmymoney/mymoney/mymoneytransactionfilter.h A +805 -0 kmymoney/mymoney/mymoneytransactionfiltertest.cpp [License: GPL (v2+)] A +48 -0 kmymoney/mymoney/mymoneytransactionfiltertest.h [License: GPL (v2+)] https://commits.kde.org/kmymoney/4ed579e86be69ce5df04c03e4b9797108ec8cfa8
Git commit 55eb90e55389f7a93c77a7fcf39d0c5a171ad9b6 by Ralf Habacker. Committed on 28/09/2018 at 19:04. Pushed by habacker into branch '4.8'. kmymoneytransactionfiltertest: add 'same date' and 'bad date order' test M +9 -0 kmymoney/mymoney/mymoneytransactionfiltertest.cpp https://commits.kde.org/kmymoney/55eb90e55389f7a93c77a7fcf39d0c5a171ad9b6
A possibly relevant merge request was started @ https://invent.kde.org/office/kmymoney/-/merge_requests/230