Bug 33470: Don't calculate overridden hold limits
authorNick Clemens <nick@bywatersolutions.com>
Sun, 9 Apr 2023 18:25:41 +0000 (18:25 +0000)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Fri, 12 May 2023 14:04:22 +0000 (15:04 +0100)
commit309e1b0cb8df29265919080e8c772fe3997a69e3
treed50df0756ab121860a636f4cc7b5ec29082702ab
parent3d75bf866bb05e0a8ab457ed3f48a2e84243bfb9
Bug 33470: Don't calculate overridden hold limits

This patch can largely be checked in the code to ensure it makes sense, it
moves a conditional up one level for the two different checks that could prevent holds

To test:
1 - Attempt to place a hold via the REST API with an invalid pickup location
    Set pickup location as no in branches to make this easy
2 - Confirm you get an error
3 - Attempt again with header (bug 27760):
    x-koha-override: any
4 - Hold is placed
5 - Repeat 1-4 above but with an item that cannot be held
6 - Apply patch
7 - Repeat 1-4 above
8 - Results should be the same

Signed-off-by: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
(cherry picked from commit e3c31212df4d1917004a20998f47777ef0aea35d)
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Koha/REST/V1/Holds.pm