Bug 20972: If ISBN has 10 numbers only the first 9 numbers are used
authorxljoha <johan.larsson@ub.gu.se>
Thu, 21 Jun 2018 07:58:55 +0000 (09:58 +0200)
committerNick Clemens <nick@bywatersolutions.com>
Fri, 22 Jun 2018 20:12:16 +0000 (20:12 +0000)
If ISBN has 10 numbers only the first 9 numbers are being added to the Edifact-message (PIA+5+3540556753:IB')

This is caused by wrong positioning of capturing parentheses in a regular expression.

Sponsored-by: Gothenburg University Library
Signed-off-by: Colin Campbell <colin.campbell@ptfs-europe.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Koha/Edifact/Order.pm

index ac63a94..dcab51d 100644 (file)
@@ -596,7 +596,7 @@ sub additional_product_id {
         $product_id   = $1;
         $product_code = 'EN';
     }
-    elsif ( $isbn_field =~ m/(\d{9})[Xx\d]/ ) {
+    elsif ( $isbn_field =~ m/(\d{9}[Xx\d])/ ) {
         $product_id   = $1;
         $product_code = 'IB';
     }