Bug 20811: (QA follow-up) Make the isbn check work
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Fri, 20 Jul 2018 09:21:11 +0000 (11:21 +0200)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Thu, 2 Aug 2018 14:17:52 +0000 (15:17 +0100)
commit5094909140081b02c9688faa07fc630feab91f24
tree20be8ae888c2e86aeaf3a82e68396bb9a1b90d6b
parentc76d5b4b4a531e408fbaa5a9a7a523a749638764
Bug 20811: (QA follow-up) Make the isbn check work

The following code was never reached, since $isbn was not filled.
    if (!$biblionumber && $isbn_check && $isbn) {
        $sth_isbn->execute($isbn);
        ($biblionumber,$biblioitemnumber) = $sth_isbn->fetchrow;
    }
Solution: Fix the code with two $isbn declarations. Move the checkisbn
condition a level deeper.

Test plan:
Run misc/migration_tools/bulkmarcimport.pl -file bib726.utf8 --update -isbn
Since you do not match on biblionumber, the ISBN should match.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
(cherry picked from commit a49a7f08d6dbaf992d70420dae744c75c2ad8aae)
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
misc/migration_tools/bulkmarcimport.pl