Bug 29003 - Apply different reasons via cancel_expired_holds cron based on found value
Summary: Apply different reasons via cancel_expired_holds cron based on found value
Status: NEW
Alias: None
Product: Koha
Classification: Unclassified
Component: Circulation (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords:
Depends on: 12656
Blocks:
  Show dependency treegraph
 
Reported: 2021-09-13 12:48 UTC by Andrew Fuerste-Henry
Modified: 2021-09-13 12:52 UTC (History)
2 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Andrew Fuerste-Henry 2021-09-13 12:48:14 UTC
Bug 12656 allows one to define a cancellation reason when running the cancel_expired_holds cron. That reason is recorded for later reporting and also triggers a notice.

That cron cancels both holds that have simply never had an item captured *and* holds where the item has been waiting on the hold shelf past the pickup delay. Ideally, we'd be able to tell the cron to apply two different reasons in these two very distinct cases.

Maybe the cron could take a pair of [found value]=[reason]? So you could set it with something like:
'W'='TOO_LONG'
NULL='UNFILLED'
Comment 1 Andrew Fuerste-Henry 2021-09-13 12:52:35 UTC
Actually, reiterating the importance of this a bit, if a library is using ExpireReservesMaxPickUpDelayCharge then an auto-cancelled waiting hold will result in a fee but an auto-cancelled non-waiting hold will not. That's not new behavior or anything, but it definitely would be helpful to be able to easily know that in the notice.