[koha-commits] main Koha release repository branch master updated. v16.05.00-981-gbc1e23e

[Git repo owner]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "main Koha release repository".

The branch, master has been updated
       via  bc1e23e1f11e146e282d0e2efa7fd77e2c26d631 (commit)
       via  0eec191ed3ab1ae1bd24972a559d12627b70f681 (commit)
       via  44b51ed80224d0361796c1a70cb66bb5096a427c (commit)
       via  e72a8ab260abed11b02266d77bc17d831884ed40 (commit)
      from  f2196a2e4f21a9a294c970a1ad067f5c3d1cb4eb (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit bc1e23e1f11e146e282d0e2efa7fd77e2c26d631
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Tue Oct 4 14:49:19 2016 +0100

    Bug 17088: Add tests to cover the changes
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    
    Signed-off-by: Brendan Gallagher <brendan at bywatersolutions.com>

commit 0eec191ed3ab1ae1bd24972a559d12627b70f681
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date:   Fri Sep 16 11:27:05 2016 +0200

    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 at 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 at bugs.koha-community.org>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    
    Signed-off-by: Brendan Gallagher <brendan at bywatersolutions.com>

commit 44b51ed80224d0361796c1a70cb66bb5096a427c
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date:   Fri Sep 16 10:44:47 2016 +0200

    Bug 17088: [Follow-up] Adjust POD in Logger.pm
    
    Not important enough to go on its own.
    
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    
    Signed-off-by: Brendan Gallagher <brendan at bywatersolutions.com>

commit e72a8ab260abed11b02266d77bc17d831884ed40
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Tue Aug 9 13:08:17 2016 +0000

    Bug 17088 - Bad MARC XML can halt export_records.pl
    
    If export_records.pl encounters a record that fails to load correctly as
    a MARC::Record the export process will fail and only the records
    encountered up to that point will be exported.
    
    Test Plan:
    1) Pick a record in your database, and replace the biblioitems.marcxml field with some random characters
    2) Export your records using the command:
       misc/export_records.pl  --format xml --record-type bibs --filename /tmp/biblios.xml
    3) Note you get an error and the script ends prematurely
    4) Apply this patch
    5) Repeat step 2
    6) Note the full record set is exported ( except for your bad record )
    7) Note an error is still generated for your bad record ( useful for
        tracking down bad records ).
    
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    
    Signed-off-by: Brendan Gallagher <brendan at bywatersolutions.com>

-----------------------------------------------------------------------

Summary of changes:
 Koha/Exporter/Record.pm          |   21 ++++++---
 Koha/Logger.pm                   |    8 +++-
 t/db_dependent/Exporter/Record.t |   89 +++++++++++++++++++++++++-------------
 3 files changed, 81 insertions(+), 37 deletions(-)


hooks/post-receive
-- 
main Koha release repository