[koha-commits] main Koha release repository branch master updated. v16.11.00-437-gd115602

[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  d115602c9881371d38f3b1c4bda66e979984b218 (commit)
      from  116301ae473075a6e183beb0027bd3657816b5e6 (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 d115602c9881371d38f3b1c4bda66e979984b218
Author: Tomas Cohen Arazi <tomascohen at theke.io>
Date:   Mon Feb 6 16:22:51 2017 -0300

    Bug 18068: ES - Fix location and (home|holding)branch facets
    
    This patch makes the 'Locations' facet work as expected (i.e. having the
    same behaviour it has for Zebra: picking the 952$c in MARC21 and 995e
    for UNIMARC).
    
    It also adds the code to handle holding and home library settings for
    facets and makes the facets show the library name instead of the branch
    code.
    
    The mappings are updated so the labels match what facets.inc expect to
    work properly.
    
    To test:
    - On master, do a search that returns biblios with items having
    homebranch set.
    => FAIL: Under the 'Locations' label on the facets you will notice
    branchcodes are shown.
    - Apply the patch
    - Restart memcached and plack (just in case, it was tricky)
    - Reset your mappings:
      http://localhost:8081/cgi-bin/koha/admin/searchengine/elasticsearch/mappings.pl?op=reset&i_know_what_i_am_doing=1
    - Restart memcached and plack (again, not sure if needed)
    - Make sure this mappings are set:
      homebranch => HomeLibrary
      holdingbranch => HoldingLibrary
      (Note: it might not be set due to the place the yaml file is being picked)
    - Reindex your records:
      $ sudo koha-shell kohadev
     k$ cd kohaclone
     k$ perl misc/search_tools/rebuild_elastic_search.pl -d -v
    - Repeat the initial search
    => SUCCESS: 'Location' contains the right stuff, 'Home libraries' and
    'Holding libraries' too.
    - Run
     k$ prove t/db_dependent/Koha_SearchEngine_Elasticsearch_Search.t
    => SUCCESS: Tests pass!
    - Sign off :-D
    
    Note: play with the 'DisplayLibraryFacets' syspref options.
    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

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

Summary of changes:
 Koha/SearchEngine/Elasticsearch/QueryBuilder.pm    |   12 ++++++++-
 Koha/SearchEngine/Elasticsearch/Search.pm          |    6 +++++
 admin/searchengine/elasticsearch/mappings.yaml     |   27 +++++++++++++++++---
 .../Koha_SearchEngine_Elasticsearch_Search.t       |   26 ++++++++++++++++++-
 4 files changed, 65 insertions(+), 6 deletions(-)


hooks/post-receive
-- 
main Koha release repository