Bug 391091

Summary: Year is not shown in plots
Product: [Applications] kmymoney Reporter: Josef Kufner <jk>
Component: reportsAssignee: KMyMoney Devel Mailing List <kmymoney-devel>
Status: CONFIRMED ---    
Severity: normal CC: antoine, ralf.habacker
Priority: NOR    
Version: 4.8.1   
Target Milestone: ---   
Platform: Debian unstable   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:
Attachments: The screenshot
test xmi file

Description Josef Kufner 2018-02-26 10:36:34 UTC
Created attachment 111005 [details]
The screenshot

(Forked from https://bugs.kde.org/show_bug.cgi?id=390045)

For certain date ranges, the time axis does not show years, but it shows dates from multiple years. Therefore, it is hard to guess on which year we are looking. See the attached screenshot.
Comment 1 Ralf Habacker 2018-02-28 17:49:07 UTC
Years are displayed are available with monthly and greater report intervals. May be that help in your case.

One related problem of diagrams compared to the table report variant may be that there is no date range displayed.
Comment 2 Ralf Habacker 2018-02-28 17:55:00 UTC
Created attachment 111094 [details]
test xmi file

see reports named "Eigenkapital aktuell (xxx)"
Comment 3 Josef Kufner 2018-02-28 19:53:08 UTC
The plot on the screenshot spans over six years.
Comment 4 Ralf Habacker 2018-03-01 07:17:19 UTC
(In reply to Josef Kufner from comment #3)
> The plot on the screenshot spans over six years.
Why can not you use a monthly interval?
Comment 5 Josef Kufner 2018-03-01 09:18:44 UTC
I cannot use monthly interval because then I loose many important details of what had happened. Many of my transactions are on boundaries of months, sometimes falling into previous month, sometimes into the next month. With monthly interval I get significantly less acurate plot.

Using daily interval (I would prefer no aggregation at all) I get more noisy line, but also more useful line. Amplitude of the noise around the mean value also shows how much the money moved around.
Comment 6 Ralf Habacker 2018-03-07 16:05:18 UTC
Git commit 0179eea65d078fe273ab79cbf96bf32698302ac1 by Ralf Habacker.
Committed on 07/03/2018 at 09:51.
Pushed by habacker into branch '4.8'.

Convert enumeration MyMoneyReport::ERowType into a class to be more type safe

Summary:

Test Plan: compiled on linux

Reviewers: #kmymoney

Reviewed By: #kmymoney, tbaumgart

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

M  +1    -1    kmymoney/dialogs/kbalancechartdlg.cpp
M  +19   -19   kmymoney/dialogs/kreportconfigurationfilterdlg.cpp
M  +1    -1    kmymoney/kmymoney.cpp
M  +50   -52   kmymoney/mymoney/mymoneyreport.cpp
M  +18   -15   kmymoney/mymoney/mymoneyreport.h
M  +1    -1    kmymoney/reports/listtable.cpp
M  +9    -9    kmymoney/reports/objectinfotable.cpp
M  +4    -4    kmymoney/reports/pivottable.cpp
M  +38   -38   kmymoney/reports/pivottabletest.cpp
M  +38   -38   kmymoney/reports/querytable.cpp
M  +14   -14   kmymoney/reports/querytabletest.cpp
M  +1    -1    kmymoney/views/kforecastview.cpp
M  +2    -2    kmymoney/views/khomeview.cpp
M  +51   -51   kmymoney/views/kreportsview.cpp

https://commits.kde.org/kmymoney/0179eea65d078fe273ab79cbf96bf32698302ac1
Comment 7 Ralf Habacker 2018-03-07 16:05:18 UTC
Git commit 1e5384157ca6f33c49ebd3bed1fed7967a7edbd9 by Ralf Habacker.
Committed on 07/03/2018 at 09:46.
Pushed by habacker into branch '4.8'.

Convert enumeration MyMoneyReport::EColumnType into a class to be more type safe

Type safety is increased by moving the enum into a separate name space (here
a class is used because namespaces are not allowed inside classes).

Q_ENUM(s) or QFlags could also not be used because the related enums need
to be defined somewhere outside, which does not solve the issue having all
constants in MyMoneyReport namespace.

Test Plan: compiled on linux

Reviewers: #kmymoney

Reviewed By: #kmymoney, tbaumgart

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

Signed-off-by: Ralf Habacker <ralf.habacker@freenet.de>

M  +1    -1    kmymoney/dialogs/kbalancechartdlg.cpp
M  +10   -10   kmymoney/dialogs/kreportconfigurationfilterdlg.cpp
M  +8    -7    kmymoney/mymoney/mymoneyreport.cpp
M  +8    -5    kmymoney/mymoney/mymoneyreport.h
M  +14   -14   kmymoney/reports/pivottable.cpp
M  +10   -10   kmymoney/reports/pivottabletest.cpp
M  +1    -1    kmymoney/views/kforecastview.cpp
M  +2    -2    kmymoney/views/khomeview.cpp
M  +31   -31   kmymoney/views/kreportsview.cpp

https://commits.kde.org/kmymoney/1e5384157ca6f33c49ebd3bed1fed7967a7edbd9
Comment 8 Ralf Habacker 2018-03-07 16:05:18 UTC
Git commit ad66b57f7397b15e7e8d1667101fdaf23b8601d0 by Ralf Habacker.
Committed on 07/03/2018 at 09:46.
Pushed by habacker into branch '4.8'.

Convert enumeration MyMoneyReport::EQueryColumns into a class to be more type safe

Test Plan: compiled on linux

Reviewers: #kmymoney

Reviewed By: #kmymoney, tbaumgart

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

M  +26   -26   kmymoney/dialogs/kreportconfigurationfilterdlg.cpp
M  +1    -1    kmymoney/kmymoney.cpp
M  +11   -12   kmymoney/mymoney/mymoneyreport.cpp
M  +10   -7    kmymoney/mymoney/mymoneyreport.h
M  +15   -15   kmymoney/reports/querytable.cpp
M  +20   -20   kmymoney/reports/querytabletest.cpp
M  +20   -20   kmymoney/views/kreportsview.cpp

https://commits.kde.org/kmymoney/ad66b57f7397b15e7e8d1667101fdaf23b8601d0
Comment 9 Ralf Habacker 2018-03-07 16:05:18 UTC
Git commit 2f575bbe69214798c67b8b47e0c2f6baa67584a4 by Ralf Habacker.
Committed on 07/03/2018 at 09:46.
Pushed by habacker into branch '4.8'.

Convert enumeration MyMoneyReport::EReportType into a class to be more type safe

Test Plan: compiled on linux

Reviewers: #kmymoney

Reviewed By: #kmymoney, tbaumgart

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

Signed-off-by: Ralf Habacker <ralf.habacker@freenet.de>

M  +2    -2    kmymoney/dialogs/kreportconfigurationfilterdlg.cpp
M  +28   -28   kmymoney/mymoney/mymoneyreport.cpp
M  +15   -12   kmymoney/mymoney/mymoneyreport.h
M  +3    -3    kmymoney/views/kreportsview.cpp

https://commits.kde.org/kmymoney/2f575bbe69214798c67b8b47e0c2f6baa67584a4
Comment 10 Ralf Habacker 2018-03-10 16:09:10 UTC
Git commit e4f200d23be7a4c2f50139ccba9998649007b3ce by Ralf Habacker.
Committed on 10/03/2018 at 16:08.
Pushed by habacker into branch '4.8'.

Convert enumeration MyMoneyReport::EChartType into a class to be more type safe

Test Plan: compiled on linux

Reviewers: #kmymoney, tbaumgart

Reviewed By: tbaumgart

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

M  +1    -1    kmymoney/dialogs/kbalancechartdlg.cpp
M  +9    -9    kmymoney/dialogs/kreportconfigurationfilterdlg.cpp
M  +11   -11   kmymoney/mymoney/mymoneyreport.cpp
M  +8    -6    kmymoney/mymoney/mymoneyreport.h
M  +13   -13   kmymoney/reports/kreportchartview.cpp
M  +1    -1    kmymoney/views/kforecastview.cpp
M  +1    -1    kmymoney/views/khomeview.cpp
M  +11   -11   kmymoney/views/kreportsview.cpp
M  +5    -5    kmymoney/widgets/kmymoneyreportconfigtabimpl.cpp

https://commits.kde.org/kmymoney/e4f200d23be7a4c2f50139ccba9998649007b3ce
Comment 11 Ralf Habacker 2018-03-10 16:09:10 UTC
Git commit 1819d276fbee3b2d59ef0f26ad893dd68e6926fa by Ralf Habacker.
Committed on 10/03/2018 at 16:08.
Pushed by habacker into branch '4.8'.

Convert enumeration MyMoneyReport::eDetailLevel into a class to be more type safe

Test Plan: compiled on linux

Reviewers: #kmymoney, tbaumgart

Reviewed By: tbaumgart

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

M  +1    -1    kmymoney/dialogs/kbalancechartdlg.cpp
M  +9    -9    kmymoney/dialogs/kreportconfigurationfilterdlg.cpp
M  +1    -1    kmymoney/kmymoney.cpp
M  +9    -9    kmymoney/mymoney/mymoneyreport.cpp
M  +10   -6    kmymoney/mymoney/mymoneyreport.h
M  +6    -6    kmymoney/reports/kreportchartview.cpp
M  +2    -2    kmymoney/reports/objectinfotable.cpp
M  +5    -5    kmymoney/reports/pivottable.cpp
M  +8    -8    kmymoney/reports/pivottabletest.cpp
M  +3    -3    kmymoney/reports/querytable.cpp
M  +2    -2    kmymoney/reports/querytabletest.cpp
M  +1    -1    kmymoney/views/kforecastview.cpp
M  +2    -2    kmymoney/views/khomeview.cpp
M  +53   -53   kmymoney/views/kreportsview.cpp

https://commits.kde.org/kmymoney/1819d276fbee3b2d59ef0f26ad893dd68e6926fa
Comment 12 antoine 2020-07-13 19:31:11 UTC
Hi

I confirm this issue.
I checked with the test file and KMyMoney-5.1.0-74a8f0d-x86_64.AppImage, Xubuntu 20.04.