Bug 4231 - Placing holds from cart fails silently

This patch adds a more meaningfull message if multiple biblios with no
items attached are in the cart.

To test:
- Add biblios with and without items to cart
- Go to cart, select all and click 'Place hold'

Without patch, message reads:
Cannot place hold: this record has no items attached.

With patch, message reads:
Cannot place hold: one or more records without items attached.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
Trivial string fix. Message is better than before.
This commit is contained in:
Marc Véron 2014-07-27 20:30:47 +02:00 committed by Tomas Cohen Arazi
parent 279afe660e
commit 437489347c
1 changed files with 7 additions and 3 deletions

View File

@ -182,9 +182,13 @@ function checkMultiHold() {
<div id="yui-main">
<div class="yui-b">
[% IF ( noitems ) %]
<div class="dialog alert">
<strong>Cannot place hold:</strong> this record has no items attached.
</div>
<div class="dialog alert">
[%IF (multi_hold) %]
<strong>Cannot place hold:</strong> one or more records without items attached.
[% ELSE %]
<strong>Cannot place hold:</strong> this record has no items attached.
[% END %]
</ div>
[% ELSE %]
[% IF ( messagetransfert ) %]
<div class="dialog message">