Bug 17327: Item level holds no longer enforced
authorKyle M Hall <kyle@bywatersolutions.com>
Wed, 21 Sep 2016 02:15:02 +0000 (02:15 +0000)
committerKyle M Hall <kyle@bywatersolutions.com>
Sat, 24 Sep 2016 20:55:16 +0000 (20:55 +0000)
commit91b23e09bbb57284e65578491ea226a407420a0b
tree8c7917135d2a867e8b40fda7758c9c7e8ad4393f
parentaa62448897f769f31604796494644e3171ef902b
Bug 17327: Item level holds no longer enforced

Reported by Katrin:

Example:
- on shelf holds allowed
- holds allowed 5
- holds on same record allowed 5
- FORCE item level holds

On a record with only one item, you can only place a record level hold
in OPAC now - which is against your circulation conditions.

Another example:
- record with 2 items
- circulation rules as above, 1 hold allowed on the record
- Item level holds: forced

I can only place record level holds in OPAC and staff. At the moment, I
am not able to place item level holds in any circumstances in my
installation.

Test Plan:
1) Apply the unit test patch
2) prove t/db_dependent/Reserves/MultiplePerRecord.t
   should fail
3) Apply the second patch
4) prove t/db_dependent/Reserves/MultiplePerRecord.t
   should pass
5) Attempt to replicate one of the examples above, you should be unable to

Signed-off-by: Benjamin Rokseth <benjamin.rokseth@kul.oslo.kommune.no>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Koha/Holds.pm