Bug 17088: [Follow-up] Use Logger for failed exports
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Fri, 16 Sep 2016 09:27:05 +0000 (11:27 +0200)
committerFrédéric Demians <f.demians@tamil.fr>
Sat, 22 Oct 2016 09:08:13 +0000 (11:08 +0200)
commit40a2b8c12150d6fe193ba10c808b47069494157a
tree2ad85ed70675cb9add06e511ff354bb4988edd79
parent051661024ca92a343ad74e2664627c070bb5c66e
Bug 17088: [Follow-up] Use Logger for failed exports

Fixes a TODO for logging unsupported record_type in _get_record_for_export.
Also logs a warning when the record_type parameter is not supplied at all in sub export.
Replaces a warn by a log message about an invalid record for format iso2709.
Also adds a log message about an invalid record for format xml.
Adds a trivial unit test for passing no record_type to export.

Test plan:
See also first patch.
Run t/db_dependent/Exporter/Record.t.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Also tested the log messages for iso2709 and xml by manipulating
the record_type parameter with:
    $params->{record_type}='xx';
before calling _get_record_for_export in Record.pm.

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Brendan Gallagher <brendan@bywatersolutions.com>
(cherry picked from commit 0eec191ed3ab1ae1bd24972a559d12627b70f681)
Koha/Exporter/Record.pm
t/db_dependent/Exporter/Record.t