[Bug 20496] New: Consolidate the checks to see if plugins are enabled in Koha::Plugin::new
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20496 Bug ID: 20496 Summary: Consolidate the checks to see if plugins are enabled in Koha::Plugin::new Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: koha-bugs@lists.koha-community.org Reporter: kyle@bywatersolutions.com QA Contact: testopia@bugs.koha-community.org The separate checks to see if plugins are enabled are placed separately throughout the Koha codebase. We should consolidate all those checks within Koha::Plugins. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20496 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |20181 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20181 [Bug 20181] Allow plugins to add CSS and Javascript to OPAC -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20496 --- Comment #1 from Kyle M Hall <kyle@bywatersolutions.com> --- 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. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20496 M. Tompsett <mtompset@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mtompset@hotmail.com --- Comment #2 from M. Tompsett <mtompset@hotmail.com> --- (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.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20496 Bug 20496 depends on bug 20181, which changed state. Bug 20181 Summary: Allow plugins to add CSS and Javascript to OPAC https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20181 What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org