Bug 31575: Missing warning for holds where AllowHoldPolicyOverride can be used to...
authorKyle Hall <kyle@bywatersolutions.com>
Fri, 16 Sep 2022 14:47:48 +0000 (10:47 -0400)
committerTomas Cohen Arazi <tomascohen@theke.io>
Tue, 11 Oct 2022 13:12:35 +0000 (10:12 -0300)
commit30eb30b3c1b3d71aed4e63f9f230cfb36f52abe2
tree86aa2e3d806cb475903a64ff01eeb382988411a3
parent7af9c1c109fd191cc638614e32e5aecd5cbb525a
Bug 31575: Missing warning for holds where AllowHoldPolicyOverride can be used to force a hold to be placed

If some holds are overridable, and AllowHoldPolicyOverride is enabled, Koha
used to "flag" each holdable item to let librarians know the hold could not
normally be placed. In 22.05 up to current master as of this writing, no
warning is displayed.

Test Plan:
1) Set one circ rule to allow 0 holds
2) For Library A, set "Hold pickup library match" to "item's hold group"
3) Enable AllowHoldPolicyOverride
4) Set ReservesControlBranch to "item's home branch"
5) Create a record with an item for Library A, one for Library B, and one for Library C
   This should create a scenario where A's item cannot be held, and B and C's items must be overridden
6) Note you get the "Cannot place hold" but can place those holds anyway without a "warning" sign
7) Apply this patch
8) Restart all the things!
9) Reload the page, you should now see the warning icons!

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
reserve/request.pl