From 80af124b064714b35f85fa47101385a57716be41 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. --- 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.20.1