bug 2201 - autoBarcode code in acq is old. Modified to only work with 'increment...
authorRyan Higgins <rch@liblime.com>
Fri, 20 Jun 2008 21:36:01 +0000 (16:36 -0500)
committerJoshua Ferraro <jmf@liblime.com>
Fri, 20 Jun 2008 21:48:50 +0000 (16:48 -0500)
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
acqui/orderreceive.pl

index 13a0e88..eaab7ee 100755 (executable)
@@ -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;