]> git.koha-community.org Git - koha.git/commit
Bug 35536: Refine verbose handling in some Koha::Plugins calls
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Mon, 18 Dec 2023 08:52:20 +0000 (08:52 +0000)
committerFridolin Somers <fridolin.somers@biblibre.com>
Mon, 22 Jul 2024 12:00:11 +0000 (14:00 +0200)
commitb00f04a7609a958352142fc50e20a34d431e6d64
tree2d0b6bac9149622b6b49a629f923d53cbe259454
parentf11c05bbdd313082c4a97a5410a6e6ce606170d7
Bug 35536: Refine verbose handling in some Koha::Plugins calls

Three routines in Plugins got the verbose parameter on 35507.
We can refine this a bit further.
The idea here is report when you are installing plugins but not
report when just calling plugins (flooding logs).

[1] GetPlugins: Most callers do not expect (or check) results for
    failing plugins. This patch makes GetPlugins only return
    errors when passing the *errors* flag (in 2 cases).
    [a] The misc/devel script prints warnings now using verbose,
        so does not need the errors flag anymore.
    [b] plugins/plugins-home is the only case left. Tiny adjustment
        to keep current behavior. Fixed colspan in template.
        Does not need verbose in favor of 'errors' (passed to
        template).
    [c] For most calls we do not want verbose. New default is 0.

[2] InstallPlugins
    [a] Disabled verbose in plugin-upload. Not really needed.
        Added a FIXME; we need to improve individual install.
    [b] misc/devel: No warnings anymore when calling InstallPlugins
        after GetPlugins.

[3] get_enabled_plugins
    [a] Plugins->call does not need verbose.
    [b] Plugins->feature_enabled does not need it too.

Test plan:

[1] See previous plan. With TestMR data but without patch, run
misc script and go to plugins-home. Do you see load errors on
commandline or form?

[2] Run plugins/plugins-upload (uploading just some file is good
enough); verify that you do not see TestMR lines in logfile.

[3] Run t/db_dependent/Koha/Plugins/Plugins.t for the additional
test on verbose and errors flag.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
(cherry picked from commit d8e04545b80869821057d5b2c3ac46f6e18b1b78)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
Koha/Plugins.pm
koha-tmpl/intranet-tmpl/prog/en/modules/plugins/plugins-home.tt
misc/devel/install_plugins.pl
plugins/plugins-home.pl
plugins/plugins-upload.pl
t/db_dependent/Koha/Plugins/Plugins.t