From 86ec07d50b4d4b47872317e822f16fac15c0e62b Mon Sep 17 00:00:00 2001
From: David Birmingham <dbirmingham@ptfs.com>
Date: Fri, 5 Jun 2009 16:40:55 -0400
Subject: [PATCH] Removed a buggy conditional if that resulted in an incorrect
 item status

Previous conditional check with if ($restype) is not needed because a true
result for one item will result in subsqeuent items defaulting to this true
value.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
---
 C4/Items.pm | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/C4/Items.pm b/C4/Items.pm
index f47f9c88b9..595812c46e 100644
--- a/C4/Items.pm
+++ b/C4/Items.pm
@@ -1267,9 +1267,10 @@ sub GetItemsInfo {
 		if ( $datedue eq '' ) {
             my ( $restype, $reserves ) =
               C4::Reserves::CheckReserves( $data->{'itemnumber'} );
-            if ($restype) {
-                $count_reserves = $restype;
-            }
+# Previous conditional check with if ($restype) is not needed because a true
+# result for one item will result in subsequent items defaulting to this true
+# value.
+            $count_reserves = $restype;
         }
         $isth->finish;
         $ssth->finish;
-- 
2.39.2