[Koha-bugs] [Bug 15565] Place multiple holds at once for the same record in OPAC

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15565

Alex Buckley <alexbuckley at catalyst.net.nz> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Failed QA                   |Needs Signoff

--- Comment #167 from Alex Buckley <alexbuckley at catalyst.net.nz> ---
(In reply to Katrin Fischer in comment 154 and Nick Clemens in comment 152):

Hi both,

I have written a follow-up patch which does the following:

OPAC: 
The MaxReserves syspref, holds per day and holds per record circ setting values
are all respected in determining how many items can be reserved. 

* Once the lowest of these 3 values is reached all un-checked item checkboxes
on opac-reserve are disabled.

Staff client:

Similar to the OPAC in that once the lowest of the 3 values is reached all
un-checked item checkboxes on request.tt are disabled except is the
AllowHoldPolicyOverride is set to 'Allow'.

If AllowHoldPolicyOverride is enabled then additional checkboxes can be
selected with the warning modal boxes loading on each checkbox click. 

Also added function to C4::Reserves named GetAllowedHoldsForPatronToday() which
returns the holds per day value for the patron and item type combination. 

I have added unit tests for this new function in
t/db_dependent/Reserves/MultiplePerRecord.t

Can you please test this?

-- 
You are receiving this mail because:
You are watching all bug changes.