From 748ad2355ea3dd4e404ec664b0d50b544db179e6 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Mon, 25 Jul 2016 16:10:34 +0100 Subject: [PATCH] Bug 16586: [QA Follow-up] Simplify code Signed-off-by: Jonathan Druart Signed-off-by: Marcel de Rooy Amended this patch: Replaced none by any. Exists test adjusted. This effectively makes the test pass. Signed-off-by: Kyle M Hall --- Koha/Plugins.pm | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/Koha/Plugins.pm b/Koha/Plugins.pm index da336adec6..c0598b1292 100644 --- a/Koha/Plugins.pm +++ b/Koha/Plugins.pm @@ -21,6 +21,7 @@ use Modern::Perl; use Module::Load::Conditional qw(can_load); use Module::Pluggable search_path => ['Koha::Plugin'], except => qr/::Edifact(|::Line|::Message|::Order|::Segment|::Transport)$/; +use List::MoreUtils qw( any ); use C4::Context; use C4::Output; @@ -77,17 +78,12 @@ sub GetPlugins { my $plugin = $plugin_class->new({ enable_plugins => $self->{'enable_plugins'} }); # Limit results by method or metadata - my $ok = 1; next if $method && !$plugin->can($method); my $plugin_metadata = $plugin->get_metadata; - foreach my $key ( keys %$req_metadata ) { - if( !$plugin_metadata->{$key} || - $plugin_metadata->{$key} ne $req_metadata->{$key} ) { - $ok = 0; - last; - } - } - push( @plugins, $plugin ) if $ok; + next if $plugin_metadata + and %$req_metadata + and any { !$plugin_metadata->{$_} || $plugin_metadata->{$_} ne $req_metadata->{$_} } keys %$req_metadata; + push @plugins, $plugin; } } return @plugins; -- 2.39.5