From d204482b5e801e47bb516ece52f0596cc14c2aea Mon Sep 17 00:00:00 2001 From: Magnus Enger Date: Tue, 18 Oct 2016 09:47:19 +0000 Subject: [PATCH] Bug 17461: Warn about plugins that can't be loaded The logs should have some info if a module has been installed that can not be loaded. To test: - Make sure you have an up to date dev install - Install v1.0 of the KitchenSink plugin from https://github.com/bywatersolutions/koha-plugin-kitchen-sink/releases - Make sure the plugin does not show up in the list of installed plugins, and that the intranet-error.log has no info about it - Apply the patch - Reload plugins-home.pl - Verify that there is now some interesting info in intranet-error.log Signed-off-by: Liz Rea Works as advertised, generates useful diagnostic information in the form of a log entry to either intranet-error or plack-error.log. Relief, thanks Magnus! Signed-off-by: Tomas Cohen Arazi Signed-off-by: Kyle M Hall (cherry picked from commit 830786aba7084698c081a3f7745fe0fe54643c18) Signed-off-by: Katrin Fischer --- Koha/Plugins.pm | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Koha/Plugins.pm b/Koha/Plugins.pm index c0598b1292..6f660984d4 100644 --- a/Koha/Plugins.pm +++ b/Koha/Plugins.pm @@ -84,6 +84,8 @@ sub GetPlugins { and %$req_metadata and any { !$plugin_metadata->{$_} || $plugin_metadata->{$_} ne $req_metadata->{$_} } keys %$req_metadata; push @plugins, $plugin; + } else { + warn $Module::Load::Conditional::ERROR; } } return @plugins; -- 2.39.5