From f6bf71ef01cdbe63031d46fbbd8e9f81b2058594 Mon Sep 17 00:00:00 2001 From: Pedro Amorim Date: Fri, 20 Dec 2024 12:05:20 +0000 Subject: [PATCH] Bug 38761: Don't pass the 'all' param when getting plugins Test plan: 1) Apply tests patch. Run. Notice it fails 2) Apply this patch. Run tests patch. Notice if passes. UI: 1) Enable ILLModule and install a backend plugin e.g.: https://github.com/PTFS-Europe/koha-ill-backend-plugin/releases/tag/v2.0.5 2) Create an ILL request: http://localhost:8081/cgi-bin/koha/ill/ill-requests.pl 3) Notice when you click 'New ILL request', 'PluginBackend' is listed even if the plugin is disabled. This patch fixes that. If there are existing requests and then their respective backend is disabled, no error occurs as plugins are loaded in the background even if disabled Signed-off-by: David Nind --- Koha/ILL/Request/Config.pm | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/Koha/ILL/Request/Config.pm b/Koha/ILL/Request/Config.pm index 687c343cc6..0c73d42371 100644 --- a/Koha/ILL/Request/Config.pm +++ b/Koha/ILL/Request/Config.pm @@ -108,7 +108,7 @@ sub backend_dir { my @backend_names = $config->get_backend_plugin_names(); -Returns a list of names for all the installed ILL backend plugins. +Returns a list of names for all the installed (and enabled) ILL backend plugins. =cut @@ -119,8 +119,7 @@ sub get_backend_plugin_names { my @backend_plugins = $koha_plugins ? $koha_plugins->GetPlugins( { - method => 'ill_backend', - all => 1 + method => 'ill_backend' } ) : (); -- 2.39.5