From e483f065a36a49bbb1ff9d49e8dff5f2fcb3e999 Mon Sep 17 00:00:00 2001 From: Ryan Higgins Date: Fri, 20 Jun 2008 16:36:01 -0500 Subject: [PATCH] bug 2201 - autoBarcode code in acq is old. Modified to only work with 'increment', and added FIXME Signed-off-by: Joshua Ferraro --- acqui/orderreceive.pl | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/acqui/orderreceive.pl b/acqui/orderreceive.pl index 13a0e88472..eaab7ee2dc 100755 --- a/acqui/orderreceive.pl +++ b/acqui/orderreceive.pl @@ -147,12 +147,11 @@ if ( $count == 1 ) { push @branchloop, \%row; } - my $auto_barcode = C4::Context->boolean_preference("autoBarcode") || 0; - - # See whether barcodes should be automatically allocated. - # Defaults to 0, meaning "no". my $barcode; - if ( $auto_barcode ) { + # See whether barcodes should be automatically allocated. + # FIXME : only incremental is implemented here, and it creates a race condition. + # + if ( C4::Context->preference('autoBarcode') eq 'incremental' ) { my $sth = $dbh->prepare("Select max(barcode) from items"); $sth->execute; my $data = $sth->fetchrow_hashref; -- 2.20.1