Attachment 33130 Details for Bug 13167 – Bug 13167 Stage MARC for Import hangs for biblio containing invalid ISBN-13

[patch] Bug 13167 Stage MARC for Import hangs for biblio containing invalid ISBN-13

Bug-13167-Stage-MARC-for-Import-hangs-for-biblio-c.patch (text/plain), 1.36 KB, created by Frédéric Demians on 2014-11-02 16:34:38 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Frédéric Demians

Created: 2014-11-02 16:34:38 UTC

Size: 1.36 KB

patch

obsolete

>From cb155d6fbc643a7bfca2c5599d367dfb7a489ceb Mon Sep 17 00:00:00 2001
>From: =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20Demians?= <f.demians@tamil.fr>
>Date: Sun, 2 Nov 2014 17:41:34 +0100
>Subject: [PATCH] Bug 13167 Stage MARC for Import hangs for biblio containing
> invalid ISBN-13
>
>If the ISBN of a UNIMARC record begins with 979 then the 'Stage MARC for
>import' hangs.  If I use the same UNIMARC record and change 979 to 978 in the
>ISBN, 'Stage MARC for import' works perfectly.
>
>The patch deals with the fact that converting an ISBN-13 to ISBN-10 with
>Business::ISBN as_isbn10() method fails if the ISBN doesn't begin with 978.
>
>TEST PLAN:
>
>(1) Download, and decompress the ZIP file attached to BZ.
>(2) On a UNIMARC Koha instance, go in Tools > Stage MARC for import.
>(3) Choose the MARC file containing the record with an ISBN begining with 979.
>    Click on Upload file, then Stage to import.
>(4) The Job progress bar stay at 0%.
>(5) Apply the patch. Repeat steps 2-3. The upload works.
>---
> C4/Koha.pm | 1 +
> 1 file changed, 1 insertion(+)
>
>diff --git a/C4/Koha.pm b/C4/Koha.pm
>index 2e19bc0..18b7f24 100644
>--- a/C4/Koha.pm
>+++ b/C4/Koha.pm
>@@ -1680,6 +1680,7 @@ sub NormalizeISBN {
>         elsif ( $format eq 'ISBN-13' ) {
>             $isbn = $isbn->as_isbn13();
>         }
>+        return unless $isbn;
> 
>         if ($strip_hyphens) {
>             $string = $isbn->as_string( [] );
>-- 
>2.1.2

Actions: View | Diff | Splinter Review

Attachments on bug 13167: 32969 | 33130 | 34414 | 34699 | 34916 | 34917