Bug 15743: Allow plugins to embed Perl modules
authorJulian Maurice <julian.maurice@biblibre.com>
Thu, 4 Feb 2016 16:42:42 +0000 (17:42 +0100)
committerBrendan Gallagher <brendan@bywatersolutions.com>
Wed, 24 Feb 2016 01:49:48 +0000 (01:49 +0000)
This patch allows plugins to embed Perl modules by ignoring Perl modules
in plugins directory that don't inherit from Koha::Plugins::Base

TEST PLAN
---------
1) Edit koha-conf.xml to have enable_plugins set to 1, and
   the pluginsdir set to a particular path.
2) Create a dummy plugin file. (e.g. {that path}/Koha/Plugin/Kaboom.pm)

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Brendan Gallagher brendan@bywatersolutions.com
Koha/Plugins.pm

index 9e2d4eb..d905554 100644 (file)
@@ -63,6 +63,8 @@ sub GetPlugins {
 
     foreach my $plugin_class (@plugin_classes) {
         if ( can_load( modules => { $plugin_class => undef } ) ) {
+            next unless $plugin_class->isa('Koha::Plugins::Base');
+
             my $plugin = $plugin_class->new({ enable_plugins => $self->{'enable_plugins'} });
 
             if ($method) {