[Koha-bugs] [Bug 25855] Add hook to AddRenewal using a new _after_circ_actions method in circulation

[bugzilla-daemon at bugs.koha-community.org]

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

Jonathan Druart <jonathan.druart at bugs.koha-community.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jonathan.druart at bugs.koha-c
                   |                            |ommunity.org

--- Comment #19 from Jonathan Druart <jonathan.druart at bugs.koha-community.org> ---
A couple of questions:

1. Why do we have the hook before UpdateStats and logaction?
We could imagine a plugin that want to deal with them, and so get up-to-date
info

2. A renewal is identified by an issue_id, why don't we only pass the issue_id
to the plugin?
It could then retrieve the info it wants.
We should limit the number of params we pass, we only need to inform the plugin
"yeah, we've just renewed this!".
It will make the plugins' api more robust and maintainable IMO.

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