]> git.koha-community.org Git - koha.git/commit
Bug 30687: Allow pickup location to be forced when override is allowed
authorNick Clemens <nick@bywatersolutions.com>
Fri, 23 Dec 2022 15:04:26 +0000 (15:04 +0000)
committerTomas Cohen Arazi <tomascohen@theke.io>
Fri, 12 May 2023 14:22:49 +0000 (11:22 -0300)
commitab93008da7eb61b697e4586c679b88c2eebaacd0
tree34ae229ba75c165460602afbda3ecb3cc8f13cf8
parentf97b448661cb2f214fccaa4ae5c3ad21fbfea22f
Bug 30687: Allow pickup location to be forced when override is allowed

This is Julian's patch with some extra cleanup to reduce repeated code

If AllowHoldPolicyOverride is enabled and only some pickup locations are
available, you still have the possibility to force one of the others
pickup locations.
But when there are zero pickup locations available, that is not
possible.

This patch change that by always displaying the list of pickup locations
when AllowHoldPolicyOverride is enabled.

Test plan:
1. Apply patch
2. Disable AllowHoldPolicyOverride
3. Create a biblio B with an item I at library A.
4. Configure this library A to not be a pickup location
5. Add a "Default holds policy by item type" for item I type where "Hold
   pickup library match" is "item's home library"
6. Try to place a hold on biblio B
   You should not be able to place a hold because there is no valid
   pickup locations
7. Enable AllowHoldPolicyOverride
8. Try to place a hold on biblio B
   You should now see all valid pickup locations in a dropdown list
   (with an exclamation mark in front of each option) with none selected
   by default
9. Verify you can place a title-level hold and an item-level hold

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Signed-off-by: Aleisha Amohia <aleishaamohia@hotmail.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
reserve/request.pl