From 99298f3d924f69d271296d5134d21d3a8a6dda39 Mon Sep 17 00:00:00 2001
From: arensb <arensb>
Date: Wed, 9 Oct 2002 14:13:31 +0000
Subject: [PATCH] Added some FIXMEs.

---
 C4/Reserves2.pm | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/C4/Reserves2.pm b/C4/Reserves2.pm
index c08a887179..7dcdd5eb2f 100755
--- a/C4/Reserves2.pm
+++ b/C4/Reserves2.pm
@@ -118,6 +118,7 @@ sub CheckReserves {
                             WHERE items.biblioitemnumber = biblioitems.biblioitemnumber
                               AND biblioitems.itemtype = itemtypes.itemtype
                               AND barcode=$qbc");
+	# FIXME - This function uses $item later on. Ought to set it here.
     }
     $sth->execute;
     my ($biblio, $bibitem, $notforloan) = $sth->fetchrow_array;
@@ -130,6 +131,8 @@ sub CheckReserves {
     my $highest;
     if ($count) {
 	foreach my $res (@reserves) {
+	    # FIXME - $item might be undefined or empty: the caller
+	    # might be searching by barcode.
 	    if ($res->{'itemnumber'} == $item) {
 		return ("Waiting", $res);
 	    } else {
@@ -140,7 +143,7 @@ sub CheckReserves {
 	    }
 	}
     }
-    if ($highest) {
+    if ($highest) {	# FIXME - $highest might be undefined
 	$highest->{'itemnumber'} = $item;
 	return ("Reserved", $highest);
     } else {
-- 
2.39.5