[Koha-bugs] [Bug 25245] Add a plugin hook to allow running code on a nightly basis

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Fri Apr 24 08:22:25 CEST 2020


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

Marcel de Rooy <m.de.rooy at rijksmuseum.nl> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |m.de.rooy at rijksmuseum.nl
             Status|Signed Off                  |Failed QA

--- Comment #7 from Marcel de Rooy <m.de.rooy at rijksmuseum.nl> ---
Some QA comments:
use Modern::Perl
Where do you need FindBin for here?
I understand the test on enable_plugins but I feel that it should be part of
GetPlugins somehow. If you did not enable them, GetPlugins should take care of
it and return nothing.
The if(@plugins) is not needed. You are looping thru @plugins inside.
Why do you warn $_ ?

Second patch
If you are adjusting the debian koha-conf, you should also touch the other one.

-- 
You are receiving this mail because:
You are watching all bug changes.


More information about the Koha-bugs mailing list