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

--- Comment #19 from Raphael Straub <[email protected]> ---
(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.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://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