Bug 30845 - OMNIBUS: Move holds checks into CanBookBeReserved/CanItemBeReserved
Summary: OMNIBUS: Move holds checks into CanBookBeReserved/CanItemBeReserved
Status: NEW
Alias: None
Product: Koha
Classification: Unclassified
Component: Hold requests (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal (vote)
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords:
Depends on: 20985 25408 30846 32702 33087
Blocks:
  Show dependency treegraph
 
Reported: 2022-05-25 12:13 UTC by Nick Clemens
Modified: 2023-02-28 15:04 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:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Nick Clemens 2022-05-25 12:13:27 UTC
We have a number of holds checks:
not for loan
opacitemholds
OPACHoldsIfAvailableAtPickup

which are only checked in the script, and not our modules

The api uses CanBook/CanItem BeReserved for checking validity of holds and is missing these checks

We need to move the checks and cover them with unit tests