From cf39be4de158e5c3b1b0c8913eee217b93e9fbe3 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 --- 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.30.2