From b5d94cb9d7d07f5ee13d4ef2a6de6bd3e6c0185e Mon Sep 17 00:00:00 2001 From: Chris Cormack Date: Tue, 17 May 2011 09:23:52 +1200 Subject: [PATCH] [SIGNED-OFF] Bug 6347 - Fix for itemlevel holds in OPAC Content-Type: text/plain; charset="utf-8" Edits by Owen: - More variable scope corrections - Minor markup and style tweak - Table column span correction Signed-off-by: Katrin Fischer - Item level holds can be placed and hold saved - Table and layout are ok now --- .../opac-tmpl/prog/en/modules/opac-reserve.tt | 29 +++++++++---------- opac/opac-reserve.pl | 6 ++-- 2 files changed, 17 insertions(+), 18 deletions(-) diff --git a/koha-tmpl/opac-tmpl/prog/en/modules/opac-reserve.tt b/koha-tmpl/opac-tmpl/prog/en/modules/opac-reserve.tt index bc46b28..6a7e973 100644 --- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-reserve.tt +++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-reserve.tt @@ -152,6 +152,7 @@ }); // ]]> + @@ -371,11 +372,13 @@ Clear Date

[% END %] - [% IF ( bibitemloo.holdable ) %][% IF ( OPACItemHolds ) %] + [% IF ( bibitemloo.holdable ) %] + + [% IF ( OPACItemHolds ) %] + - - - - - - - -
+
    +
  • [% UNLESS ( bibitemloo.holdable ) %] [% END %] -
+ +
  • [% UNLESS ( bibitemloo.holdable ) %] [% END %] -
  • + + [% END %][% END %] @@ -440,9 +440,8 @@ [% IF ( OPACItemHolds ) %] [% IF ( bibitemloo.holdable ) %] - - - +   + @@ -466,7 +465,7 @@
    Select a specific copy:
    [% IF ( itemLoo.available ) %] - [% ELSE %] diff --git a/opac/opac-reserve.pl b/opac/opac-reserve.pl index d50480a..a92a976 100755 --- a/opac/opac-reserve.pl +++ b/opac/opac-reserve.pl @@ -477,11 +477,11 @@ if ( $numBibsAvailable == 0 || !$anyholdable) { $template->param( none_available => 1 ); } -my $itemTableColspan = 5; -if (!$template->param('OPACItemHolds')) { +my $itemTableColspan = 7; +if (! $template->{VARS}->{'OPACItemHolds'}) { $itemTableColspan--; } -if ($template->param('singleBranchMode')) { +if (! $template->{VARS}->{'singleBranchMode'}) { $itemTableColspan--; } $template->param(itemtable_colspan => $itemTableColspan); -- 1.7.4.1