]> git.koha-community.org Git - koha.git/commit
Bug 27529: Choose patron's branch or item's homebranch if following group rules and...
authorNick Clemens <nick@bywatersolutions.com>
Mon, 25 Jan 2021 15:34:31 +0000 (15:34 +0000)
committerAndrew Fuerste-Henry <andrew@bywatersolutions.com>
Sun, 25 Apr 2021 16:36:13 +0000 (16:36 +0000)
commita363413a4e49cf80136a806fac424a7c8e9ebe3a
treeb145a94510d412efcdaf26d08400778e548b9666
parentd52969810fe597197bd6313d5446b649c0011ee2
Bug 27529: Choose patron's branch or item's homebranch if following group rules and patron cannot choose branch

This patch defaults the holds pickup location to the items homebranch or the patron's branch when a group option is selected
for the hold fulfillment policy and the patron is not allowed to choose the branch on the OPAC

To test:
1 - Set 'Default checkout, hold and return policy' -> 'Hold pickup library match' to item's hold group or patron's hold group
2 - Set  OPACAllowUserToChooseBranch to 'Don't allow'
3 - Try to place an item level hold on the opac
4 - Ka-boom, etiher:
    The method Koha::Item->patrongroup is not covered by tests!
    The method Koha::Item->itemgroup is not covered by tests!
5 - Apply patch
6 - Repeat
7 - Note the pickup location is set to either the patron's branch or the items homebranch
8 - Repeat plan with the other group setting

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
(cherry picked from commit 99c3b9ae1f55066f4fd2bd0e37ba336f18592780)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
(cherry picked from commit b78ccb3e479919be83dc027eaf23fc1c44f62fba)

Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com>
opac/opac-reserve.pl