@@ -, +, @@ confusing - Only display the first message, when somoene tried unsuccessfully to place holds on multiple records. - Change first message to: Sorry, none of these titles can be placed on hold. - Change the second message to read: No items available to be placed on hold. - Remove around Sorry for better translatability. - Try to place a hold on single record, where no hold is possible. - Try to place a hold on a single record, where a hold is possible. - Try to place holds on multiple records where no hold is possible. - Try to place holds on multiple records where at least one hold is possible. Verify the screen messages make sense in all cases. --- koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-reserve.tt | 8 ++++---- opac/opac-reserve.pl | 4 ++++ 2 files changed, 8 insertions(+), 4 deletions(-) --- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-reserve.tt +++ a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-reserve.tt @@ -22,7 +22,7 @@
[% IF ( message ) %]
-

Sorry, you cannot place holds.

+

Sorry, you cannot place holds.

@@ -174,7 +174,7 @@
You have already requested this title.
[% ELSE %] [% UNLESS ( bibitemloo.bib_available ) %] -
No available items.
+
There are no items that can be placed on hold.
[% ELSE %] [% IF ( bibitemloo.already_patron_possession ) %]
This title cannot be requested because it's already in your possession.
--- a/opac/opac-reserve.pl +++ a/opac/opac-reserve.pl @@ -623,6 +623,10 @@ if ( $numBibsAvailable == 0 || $anyholdable == 0) { $template->param( none_available => 1 ); } +if (scalar @biblionumbers > 1) { + $template->param( multi_hold => 1); +} + my $show_notes=C4::Context->preference('OpacHoldNotes'); $template->param(OpacHoldNotes=>$show_notes); --