Bug 9761: Make it possible to confirm future hold requests at checkin time
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Mon, 4 Mar 2013 13:47:57 +0000 (14:47 +0100)
committerGalen Charlton <gmc@esilibrary.com>
Wed, 25 Sep 2013 00:26:34 +0000 (00:26 +0000)
commitded520afdc1b3191d3ec204e3ad10443e7c0925e
tree5994f9d852536f188eb9066182cd7cbe8efc0d44
parent39c435e36fec65dbba795214bdde79932bb4edcf
Bug 9761: Make it possible to confirm future hold requests at checkin time

Description:

A new pref ConfirmFutureHolds is added. When confirming a hold at checkin time,
the number of days in this pref is taken into account when looking for reserves.
Note that this pref does not interfere with renewing, issuing or transferring
a book. For report Holds to pull, the default end date is calculated with this
new preference.
The use of ConfirmFutureHolds is useful only when future holds are allowed.

Test plan:
1) Enable future holds. Add a number of days into ConfirmFutureHolds.
2) Place a future hold within this number of days.
3) Run holds to pull report. Check default startdate and enddate.
4) Check this book in. Can you confirm the hold? Do not confirm.
5) Issue the book to another patron. You should not see a warning.
6) Renew the book for this patron via opac or staff. No warning either.
7) Check in again. Warning pops up again.
8) Transfer book. Switch branch. Check in. Hold found pops up. Do not confirm.
9) Back to first branch. Check in (with popup). Remove the hold. Add new future
hold past the number of days. Check in (no warn).

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
C4/Circulation.pm
C4/Reserves.pm
circ/pendingreserves.pl
koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref
koha-tmpl/intranet-tmpl/prog/en/modules/circ/pendingreserves.tt