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

--- Comment #1 from Pedro Amorim <[email protected]> ---
Would simply having "Place an ILL request" as an option satisfy this use case?
I suggest this be handled in core sub backend_create. It would update a
patron's lastseen regardless of the type of request, or if it results in a loan
or not.
It would also be in line with current options (very specific, single, actions),
and future use cases can also be considered to be implemented the same way e.g.
"Cancelling an ILL request".

(In reply to Magnus Enger from comment #0)
> - Would it be OK to have "ill" listed as an activity for
> TrackLastPatronActivityTriggers even if the ILL module is not activated?

It would be okay imo (not critical), but I think ideally it shouldn't list the
option if ILLModule is disabled. Although I can see the following happening:
1) "Place an ILL request" option is enabled in TrackLastPatronActivityTriggers
2) ILLModule gets disabled
3) TrackLastPatronActivityTriggers sys pref is updated ("Place an ILL request"
is now gone because it wasn't listed thus not submitted)
4) ILLModule gets enabled
5) TrackLastPatronActivityTriggers does not have the "Place an ILL request"
enabled.

Not very relevant or critical, but something to keep in mind if this is a first
(having conditional list options in a sys pref)

-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
_______________________________________________
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