Bug 15357: Display existing holds even if no items exist
If all the items have been deleted for a record and holds exist, the holds are not displayed. You are not able to delete the items from the record detail page, but you can from the items page. Test plan: 1 - Place 1 or more title level holds on a record. 2 - Delete each item individually. 3 - Note that you see the number of holds on the record details page With this patch, that holds are accessible via the holds tab. Signed-off-by: Nick Clemens <nick@bywatersolutions.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Brendan A Gallagher <brendan@bywatersolutions.com> (cherry picked from commit0e4b703d23) Signed-off-by: Julian Maurice <julian.maurice@biblibre.com> (cherry picked from commit2903bc2909) Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
This commit is contained in:
parent
dead28ef19
commit
05ef470ecb
|
|
@ -227,8 +227,9 @@ function checkMultiHold() {
|
|||
[% ELSE %]
|
||||
<strong>Cannot place hold:</strong> this record has no items attached.
|
||||
[% END %]
|
||||
</ div>
|
||||
[% ELSE %]
|
||||
</div>
|
||||
[% END %]
|
||||
|
||||
[% IF ( messagetransfert ) %]
|
||||
<div class="dialog message">
|
||||
<h2>Hold found for ([% nextreservtitle %]), please transfer</h2>
|
||||
|
|
@ -246,7 +247,7 @@ function checkMultiHold() {
|
|||
<h1>Confirm holds</h1>
|
||||
[% END %]
|
||||
|
||||
[% UNLESS ( borrowernumber ) %]
|
||||
[% UNLESS borrowernumber OR noitems %]
|
||||
[% IF ( messageborrower ) %]
|
||||
<div class="dialog alert"><h3>Patron not found</h3><p>No patron with this name, please, try another</p> </div>
|
||||
[% END %]
|
||||
|
|
@ -295,7 +296,7 @@ function checkMultiHold() {
|
|||
<input type="hidden" name="biblionumbers" value="[% biblionumbers %]"/>
|
||||
[% END %]
|
||||
</form>
|
||||
[% ELSE %]
|
||||
[% ELSIF NOT noitems %]
|
||||
|
||||
[% IF ( maxreserves || alreadyreserved || none_available || alreadypossession || ageRestricted ) %]
|
||||
<div class="dialog alert">
|
||||
|
|
@ -841,7 +842,6 @@ function checkMultiHold() {
|
|||
</form>
|
||||
[% END %]
|
||||
[% END %]
|
||||
[% END %]
|
||||
</div>
|
||||
</div>
|
||||
|
||||
|
|
|
|||
Loading…
Reference in New Issue