From c56b74ae9c970c75f314667001f96417a1a68ccb Mon Sep 17 00:00:00 2001
From: Tomas Cohen Arazi <tomascohen@theke.io>
Date: Fri, 17 Dec 2021 12:53:28 -0300
Subject: [PATCH] Bug 29553: (QA follow-up) Check defined instead of evaluating
 as boolean

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
---
 C4/Reserves.pm | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/C4/Reserves.pm b/C4/Reserves.pm
index 706721acab..1490d5aa88 100644
--- a/C4/Reserves.pm
+++ b/C4/Reserves.pm
@@ -1345,7 +1345,9 @@ sub IsAvailableForItemLevelRequest {
     # FIXME - a lot of places in the code do this
     #         or something similar - need to be
     #         consolidated
-    my $itemtype = $item->effective_itemtype or return 0;
+    my $itemtype = $item->effective_itemtype;
+    return 0
+      unless defined $itemtype;
     my $notforloan_per_itemtype = Koha::ItemTypes->find($itemtype)->notforloan;
 
     return 0 if
-- 
2.39.5