From 6b6bd14d5c180641430b68a300cc5e63b504f59f Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Tue, 20 Jul 2021 11:45:25 +0200 Subject: [PATCH] Bug 27526: Fix incorrect condition The barcode was always prefilled! Signed-off-by: Martin Renvoize Signed-off-by: Tomas Cohen Arazi --- cataloguing/additem.pl | 17 ++++++++++++++--- 1 file changed, 14 insertions(+), 3 deletions(-) diff --git a/cataloguing/additem.pl b/cataloguing/additem.pl index 14e88d1599..f4b059f379 100755 --- a/cataloguing/additem.pl +++ b/cataloguing/additem.pl @@ -457,7 +457,7 @@ if ($op eq "additem") { } # if autoBarcode is set to 'incremental', calculate barcode... - if ( not $item->barcode && C4::Context->preference('autoBarcode') eq 'incremental' ) { + if ( ! defined $item->barcode && C4::Context->preference('autoBarcode') eq 'incremental' ) { my ( $barcode ) = C4::Barcodes::ValueBuilder::incremental::get_barcode; $item->barcode($barcode); } @@ -466,7 +466,9 @@ if ($op eq "additem") { if ( $add_submit || $prefillitem) { # check for item barcode # being unique - if ( Koha::Items->search({ barcode => $item->barcode })->count ) { + if ( defined $item->barcode + && Koha::Items->search( { barcode => $item->barcode } )->count ) + { # if barcode exists, don't create, but report The problem. push @errors, "barcode_not_unique"; @@ -713,7 +715,16 @@ if ($op eq "additem") { } # check that the barcode don't exist already - if ( Koha::Items->search({ barcode => $item->barcode, itemnumber => { '!=' => $item->itemnumber } })->count ) { + if ( + defined $item->barcode + && Koha::Items->search( + { + barcode => $item->barcode, + itemnumber => { '!=' => $item->itemnumber } + } + )->count + ) + { # FIXME We shouldn't need that, ->store would explode as there is a unique constraint on items.barcode push @errors,"barcode_not_unique"; $current_item = $item->unblessed; # Restore edit form for the same item -- 2.32.0