Bug 19718: Show message if patron has existing holds on a title

To test:
 1 - Set a rule to allow multiple holds on a title
 2 - Place a single hold in the opac for a patron
 3 - Return to record and attempt place a second hold
 4 - Note there is no message but that you must match the type placed before (item or record level)
 5 - Don't place the hold
 6 - Apply patch
 7 - Attempt to place second hold again
 8 - Note there is a message informing you of which level hold you may place
 9 - Test with both item/record level holds and placing a single request or multiple requests

Signed-off-by: Donna Bachowski <donna@bywatersolutions.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
This commit is contained in:
Nick Clemens 2019-08-28 11:51:18 +00:00 committed by Martin Renvoize
parent ce14ab2bf8
commit 8f33efbc29
No known key found for this signature in database
GPG Key ID: 422B469130441A0F
2 changed files with 9 additions and 2 deletions

View File

@ -158,9 +158,15 @@
<div id="bigloop">
[% FOREACH bibitemloo IN bibitemloop %]
<div class="holdrow">
<p>
[% IF ( bibitemloo.holdable ) %]
<div class="holdrow">
[% IF bibitemloo.forced_hold_level %]
<span class="forced_hold_level"><h5>
You already have at least one [% bibitemloo.forced_hold_level %] level hold on this title.
All further holds must be [% bibitemloo.forced_hold_level %] level.
</h5></span>
[% END %]
<p>
<input class="reserve_mode" name="reserve_mode" type="hidden" value="single"/>
<input class="single_bib" name="single_bib" type="hidden" value="[% bibitemloo.biblionumber | html %]"/>
<span class="confirmjs_hold" title="[% bibitemloo.biblionumber | html %]" style="padding:.3em"></span>

View File

@ -638,6 +638,7 @@ foreach my $biblioNum (@biblionumbers) {
if ($forced_hold_level) {
$biblioLoopIter{force_hold} = 1 if $forced_hold_level eq 'item';
$biblioLoopIter{itemholdable} = 0 if $forced_hold_level eq 'record';
$biblioLoopIter{forced_hold_level} = $forced_hold_level;
}