[Koha-bugs] [Bug 25711] Add ability to define expiredpickup charge for patron category and item type
bugzilla-daemon at bugs.koha-community.org
bugzilla-daemon at bugs.koha-community.org
Mon Apr 26 10:01:04 CEST 2021
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25711
Peter Vashchuk <stalkernoid at gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #119912|0 |1
is obsolete| |
--- Comment #20 from Peter Vashchuk <stalkernoid at gmail.com> ---
Created attachment 120164
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120164&action=edit
Bug 25711: (QA follow-up) Refactor to clarify logic
At the point where 'ExpireReservesMaxPickUpDelayCharge' can/should kick
in, we should already have an item waiting (else what's triggered a pickup
delay). As such we can drop the fall through default of using the syspref
when we don't yet have an item.
We can also save us some calculations and database hits by only
calculating the charge value when we've actually been asked to charge.
So we also move the block inside the 'charge_cancel_fee' block.
Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
Signed-off-by: Petro Vashchuk <stalkernoid at gmail.com>
--
You are receiving this mail because:
You are watching all bug changes.
More information about the Koha-bugs
mailing list