Bug 6448 - carp on invalid barcode
authorDobrica Pavlinusic <dpavlin@rot13.org>
Fri, 3 Aug 2012 22:03:11 +0000 (00:03 +0200)
committerPaul Poulain <paul.poulain@biblibre.com>
Wed, 19 Sep 2012 14:11:41 +0000 (16:11 +0200)
This is consistent with implementation of other barcode types
which all carp instead of die on invalid barcodes.

C4/Barcodes/EAN13.pm

index da27c41..adc1283 100644 (file)
@@ -24,6 +24,7 @@ use C4::Context;
 use C4::Debug;
 
 use Algorithm::CheckDigits;
+use Carp;
 
 use vars qw($VERSION @ISA);
 use vars qw($debug $cgi_debug);        # from C4::Debug, of course
@@ -40,7 +41,8 @@ sub parse {
     if ( $ean->is_valid($barcode) ) {
         return ( '', $ean->basenumber($barcode), $ean->checkdigit($barcode) );
     } else {
-        die "$barcode not valid EAN-13 barcode";
+        carp "$barcode not valid EAN-13 barcode";
+        return ( '', $barcode, '' );
     }
 }