From de6ca8aa3d291edc054028fb4f16bc6540ab47fb Mon Sep 17 00:00:00 2001 From: Martin Renvoize Date: Wed, 20 Jul 2022 09:29:39 +0100 Subject: [PATCH] Bug 31192: Check for item before calling is_bundle The code in here is weird.. we really aught to check for the item before anything else and throw an error to screen if we don't find one... but my patch takes the simple option, and the one taken elsewhere in the script.. to just check for item being defined before calling a method upon it. Signed-off-by: Sally Signed-off-by: Nick Clemens Signed-off-by: Tomas Cohen Arazi --- circ/returns.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/circ/returns.pl b/circ/returns.pl index 7399568cf5..c118260cb4 100755 --- a/circ/returns.pl +++ b/circ/returns.pl @@ -410,7 +410,7 @@ if ($barcode) { } # Mark missing bundle items as lost and report unexpected items - if ( $item->is_bundle && $query->param('confirm_items_bundle_return') ) { + if ( $item && $item->is_bundle && $query->param('confirm_items_bundle_return') ) { my $BundleLostValue = C4::Context->preference('BundleLostValue'); my $barcodes = $query->param('verify-items-bundle-contents-barcodes'); my @barcodes = map { s/^\s+|\s+$//gr } ( split /\n/, $barcodes ); -- 2.39.5