Bug 13687: Move hold policy check into CanItemBeReserved
authorJulian Maurice <julian.maurice@biblibre.com>
Tue, 10 Feb 2015 10:34:01 +0000 (11:34 +0100)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Tue, 19 May 2015 15:05:50 +0000 (12:05 -0300)
commit1970b245f5426506a0eed92d74f530de2e909d37
tree5f95146e72a2d1d63e5a288fc18055c1e842e046
parent2efbd3794ecb5a2fc3c9f4ea3e2cf0e65fc20385
Bug 13687: Move hold policy check into CanItemBeReserved

This way ILS-DI HoldItem and HoldTitle services also benefit from this
check

Test plan:

1/ Define some default holds policies by item type in
/admin/smart-rules.pl
2/ Use ILS-DI HoldItem service and check that those rules are respected
3/ Check that staff and opac hold behaviour is unchanged regarding
these rules.

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Passes tests and QA script. No regressions found,
improves the ILS-DI HoldItem response.
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
C4/Reserves.pm
opac/opac-reserve.pl
reserve/request.pl
t/db_dependent/Holds.t