https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20496

M. Tompsett <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]

--- Comment #2 from M. Tompsett <[email protected]> ---
(In reply to Kyle M Hall from comment #1)
> Point of note, we check two different things:
> 1) Is UseKohaPlugins enabled
> 2) Is enabled_plugins set in the Koha config file
> 
> I think we should have a method in Koha::Plugins that checks both and
> returns true/false. If there are any parts of Koha that only check one for a
> good reason, we can have another method to check just that.

Or you could return a hashref.
Configured => true/false                    # true when both syspref and config
Failures => [systempreference,config,other] # an array with what fails

Then:
my $rv = Koha::Plugins->configured();
if ( $rv->{Configured} ) { ... }
or perhaps
if ( $rv->{Configured} || ! grep { $_ eq 'systempreference' } @{$rv->Failures}
) {
 ...
}

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to