Bug 12197: enforce the maxreserves preference when staff members place hold requests
authorKyle M Hall <kyle@bywatersolutions.com>
Tue, 6 May 2014 15:32:02 +0000 (11:32 -0400)
committerTomas Cohen Arazi <tomascohen@theke.io>
Thu, 22 Oct 2015 12:38:53 +0000 (09:38 -0300)
commit27a6b8c5919230ecf845f9f23a19a433df83ddec
tree5a5cc7be3b97a3a61538fa1812202af9ddae0f78
parent95811a29993cd7718e90e4fcb37344a61e000fe4
Bug 12197: enforce the maxreserves preference when staff members place hold requests

This patch ensures that the global maxreserves preference is enforced
when staff members place hold requests.

For example:

Create 3 items to place holds on. Set the circulation rule to allow 50
holds for all items. Set maxreserves to 2. Place a hold on 3 different
items. On the third item, it will give a warning, but you can still
place the hold. Despite what the circulation rule is set for (which is
only a specific case rule), maxreserves is a global rule and
should stop this from happening, not just give a warning.

Test Plan:
1) Reproduce the bug by following the steps above
2) Verify the bug exists
3) Apply this patch
4) Verify the librarian cannot place the hold now
5) Enable AllowHoldPolicyOverride
6) Verify the librarian can forcefully place the hold

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

Bug 12197: (follow-up) rename variable for greater clarity

"maxreserves" was referring both to the system preference and to the
condition of having exceeded the number of hold requests allowed.

This patch renames a variable to remove the ambguity.

Test plan:

* Same as the main patch.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
reserve/request.pl