| tbaumgart |
| KMyMoney |
This patch makes reports view optional instead of mandatory. User can switch on and off this view and have more space on views pane.
Developers will benefit by saving on build time of this plugin.
This patch introduces kmm_menus shared library for QActions to work seamlessly across plugin interface. It's worth considering merge with settings and icons in the future, as these libraries are often required together.
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
Works for me with the following patch applied:
diff --git a/kmymoney/menus/menuenums.h b/kmymoney/menus/menuenums.h index ac275378..bc59e434 100644 --- a/kmymoney/menus/menuenums.h +++ b/kmymoney/menus/menuenums.h @@ -19,6 +19,8 @@ #include "kmm_menus_export.h" #include <QHashFunctions> +#include <QHash> + class QAction; class QMenu;
Also, you should add a reports/.krazy file to avoid krazy complaints about the empty .kcfg file. See forecast/.krazy as reference.
In D11833#237563, @tbaumgart wrote:Also, you should add a reports/.krazy file to avoid krazy complaints about the empty .kcfg file. See forecast/.krazy as reference.
I did it without that, because I see that krazy complaints nevertheless and I think it may complain about commented stuff, which isn't present in .kcfg file from this patch.