https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36503 --- Comment #19 from Raphael Straub <raphael.straub@kit.edu> --- (In reply to Martin Renvoize from comment #18) Hi Martin,
It looks like you missed Marks comment about removing the 'enable_plugins' section... no, as Clemens wrote in comment #5 the if statement is necessary.
in fact you can and should just use the ->call method now.
my @responses = Koha::Plugins->call($method, @args);
The above should suffice for a large part of your code block introduced.. it does the check for enable_plugins, find the right plugins and checked they're enabled/disabled individually and then runs their methods.. it's also allows passing by reference so you should still be able to use the passed arguments to fill the data on return. Yes, I considered using Koha::Plugins->call but, as far as I understand, "call" calls the given method of all plugins that implement it in an arbitrary order. Because all such plugins could read/modify the mapped data and/or the patron, the order is important here. This is why I added the possibility to define a priority ($plugin->retrieve_data('priority')) on these plugins.
-- You are receiving this mail because: You are watching all bug changes.