[koha-commits] main Koha release repository branch master updated. v16.05.00-1399-gcb76c3b

[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  cb76c3bd97665ef6bee1f9be8b0e96333f91abbe (commit)
       via  847ac4b031d7b89968743c8f1f22feb4f4906d8c (commit)
      from  6d5b056a14e5671f51735dded7609f65ca389f00 (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 cb76c3bd97665ef6bee1f9be8b0e96333f91abbe
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Thu Sep 29 16:08:36 2016 +0100

    Bug 17377: ES - Take control fields into account
    
    There is a bug in Koha::SearchEngine::Elasticsearc::Search->json2marc, it
    assumes that the MARC::Field constructor always takes >= 5 parameters.
    This assumption is wrong for control fields, to create a control field you
    need to call the constructor with:
      MARC::Field->new($tag, $value);
    
    Note that I got "Too much data for control field" in the _warning value of my
    MARC::Field because too many parameters were passed, and the value was undef.
    
    That broke the result search links (on the staff interface) for DB with
    biblio.biblionumber mapped with 001.
    
    Other bugs will certainly be fixed by this patch.
    For instance I got:
      GetMarcBiblio called with undefined biblionumber at /home/koha/src/opac/opac-search.pl line 664.
      GetCOinSBiblio called with undefined record at /home/koha/src/opac/opac-search.pl line 665.
    in my logs, with this patch I don't get them anymore.
    
    Test plan:
    You can try to recreate the different issues and confirm than this patch
    fixes them.
    Or just run the tests
    
    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

commit 847ac4b031d7b89968743c8f1f22feb4f4906d8c
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Thu Sep 29 16:41:53 2016 +0100

    Bug 17377: Add tests for json2marc
    
    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

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

Summary of changes:
 Koha/SearchEngine/Elasticsearch/Search.pm          |   12 ++++++++--
 .../Koha_SearchEngine_Elasticsearch_Search.t       |   24 +++++++++++++++++++-
 2 files changed, 33 insertions(+), 3 deletions(-)


hooks/post-receive
-- 
main Koha release repository