From f8fec84317ddd5e7ac411791895b622257c8aa09 Mon Sep 17 00:00:00 2001 From: Katrin Fischer Date: Tue, 10 Apr 2012 07:17:58 +0200 Subject: [PATCH] [SIGNED OFF] Bug 7937: Fix warns in opac-reserve.pl 1) opac-reserve.pl: Use of uninitialized value in string ne at /home/katrin/kohaclone/opac/opac-reserve.pl line 475. 2) opac-reserve.pl: Use of uninitialized value $reservedfor in string eq at /home/katrin/kohaclone/opac/opac-reserve.pl line 410 Signed-off-by: Chris Cormack --- opac/opac-reserve.pl | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/opac/opac-reserve.pl b/opac/opac-reserve.pl index 4e4d440..91f3569 100755 --- a/opac/opac-reserve.pl +++ b/opac/opac-reserve.pl @@ -407,7 +407,7 @@ foreach my $biblioNum (@biblionumbers) { my $ItemBorrowerReserveInfo = GetMemberDetails( $reservedfor, 0); # the item could be reserved for this borrower vi a host record, flag this - if ($reservedfor eq $borrowernumber){ + if ($reservedfor && $reservedfor eq $borrowernumber){ $itemLoopIter->{already_reserved} = 1; } @@ -472,7 +472,7 @@ foreach my $biblioNum (@biblionumbers) { $policy_holdallowed = 0; } - if (IsAvailableForItemLevelRequest($itemNum) and $policy_holdallowed and CanItemBeReserved($borrowernumber,$itemNum) and ($itemLoopIter->{already_reserved} ne 1)) { + if (IsAvailableForItemLevelRequest($itemNum) and $policy_holdallowed and CanItemBeReserved($borrowernumber,$itemNum) and !$itemLoopIter->{already_reserved}) { $itemLoopIter->{available} = 1; $numCopiesAvailable++; } -- 1.7.5.4