]> git.koha-community.org Git - koha.git/commit
Bug 19049: Fix regression on stage-marc-import with to_marc plugin
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Mon, 7 Aug 2017 19:24:48 +0000 (21:24 +0200)
committerKatrin Fischer <katrin.fischer.83@web.de>
Sun, 10 Sep 2017 08:59:47 +0000 (10:59 +0200)
commit8ef1bf53cfa0c7ee74bd157c39adc68b5e1121c7
tree9d81c3ed334cd4318f0c53107d3c87e430e09ac3
parent4e59f94961cd39d079bcd3664a8e5f7fa479c156
Bug 19049: Fix regression on stage-marc-import with to_marc plugin

Bug 12412 added the use of to_marc plugins allowing arbitrary file formats
in stage-marc-import (as long as the plugins can handle them). The feature
was not very visible in the code, and when bug 10407 added the marcxml
format, it made some changes that broke the use of to_marc.

This patch restores the functionality by:
[1] Adding a sub RecordsFromMarcPlugin to ImportBatch.pm, specifically
    addressing the conversion from arbitrary formats to MARC::Record.
    The original to_marc interface is used: pass it the file contents,
    and it returns a string consisting of a number of MARC blobs separated
    by \x1D.
    Consequently, the call of to_marc is removed from routine
    BatchStageMarcRecords where it did not belong. The to_marc_plugin
    parameter is removed and two calls are adjusted accordingly.
[2] Instead of a separate combo with plugins, the format combo contains
    MARC, MARCXML and optionally some plugin formats.
[3] The code in stage-marc-import.pl now clearly shows the three main
    format types: MARC, MARCXML or plugin based.

Note: This patch restores more or less the situation after bug 12412, but
I would actually recommend to have the to_marc plugins return MARC::Record
objects instead of large text strings. In the second example I added a
to_marc plugin that actually converts MARC record objects to string format,
while RecordsFromMarcPlugin reconverts them to MARC::Records.

Test plan:
See second patch.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
(cherry picked from commit d24568b67256fab79b42ead54302c76d1c9653d7)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
(cherry picked from commit 4dd020a3019cd7e2b357153ec737b15eea95f671)
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
C4/ImportBatch.pm
koha-tmpl/intranet-tmpl/prog/en/modules/tools/stage-marc-import.tt
misc/stage_file.pl
tools/stage-marc-import.pl