[koha-commits] main Koha release repository branch 17.05.x updated. v17.05.01-38-g15e6d33

[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, 17.05.x has been updated
       via  15e6d33fc02f2c11e24c359dbc52da6b2f20fbf9 (commit)
       via  794bb066aaedbbd73cd66c08744c52f25b725d96 (commit)
       via  7feb5654ebe7c0445eead29ac14cb47374d1783c (commit)
       via  b1737846404716094b42e19fd2f5774bf15e2662 (commit)
       via  437d7f62fb98ec04c4ec9753aae1e3d04bb56f51 (commit)
      from  9800895cabade7ad8d78ff986a9a156f3962efa2 (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 15e6d33fc02f2c11e24c359dbc52da6b2f20fbf9
Author: Nick Clemens <nick at bywatersolutions.com>
Date:   Mon Jul 3 09:42:21 2017 +0000

    Bug 18434 - Followup fix tests for sorting and factes
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    (cherry picked from commit 540d488e980b2521b424bed0f0267002c6dbd03e)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit 794bb066aaedbbd73cd66c08744c52f25b725d96
Author: Nick Clemens <nick at bywatersolutions.com>
Date:   Fri Jun 16 10:01:12 2017 -0400

    Bug 18434 - Followup - same changes for sort and facet fields
    
    To test:
    1 - Index some stuff with multiple fields defined for sorting
      i.e. Authorites - make heading sortable - default is 110a and 111a for
      heading - a record with 111a empty will make the sort field empty
    2 - view the record:
    curl http://localhost:9200/koha_kohadev_authorities/data/30?pretty=true
    3 - Note the blank field
    4 - Apply patch
    5 - Reindex
    6 - Fields should be correctly populated
    
    Unit tests to follow (once I have the originals working for all)
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    (cherry picked from commit 241be8ff1ae24fff6165c8d6877945637467ee9a)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit 7feb5654ebe7c0445eead29ac14cb47374d1783c
Author: Tomas Cohen Arazi <tomascohen at theke.io>
Date:   Mon May 15 11:46:00 2017 -0300

    Bug 18434: (QA followup) Move _convert_marc_to_json tests into Indexer.t
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    (cherry picked from commit 3b5529c3abca87afb4f66f5bc7d3aa0fe005c1aa)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit b1737846404716094b42e19fd2f5774bf15e2662
Author: Nick Clemens <nick at bywatersolutions.com>
Date:   Fri Apr 14 10:03:11 2017 -0400

    Bug 18434: Elasticsearch indexing broken with newer catmandu version
    
    To test:
    1 - Make sure you have latest koha deps, catmandu versions should be:
        libcatmandu-marc-perl   1.09-1~kohadev1
        libcatmandu-perl        1.0304-2~kohadev1
    2 - Reindex elastic
    3 - Try searching and likely notice odd results
    4 - Try:
    curl -XGET
    'http://localhost:9200/koha_kohadev_biblios/data/792?pretty=true'
    with a known biblionumber and notice some null fields
    5 - Apply patch
    6 - Reindex
    7 - Note fields are populated and search works as expected
    
    Signed-off-by: Chris Cormack <chrisc at catalyst.net.nz>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    (cherry picked from commit 9eb88f588bd13ddf25135c26acd905abe71f326a)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit 437d7f62fb98ec04c4ec9753aae1e3d04bb56f51
Author: Tomas Cohen Arazi <tomascohen at theke.io>
Date:   Fri May 12 15:39:25 2017 -0300

    Bug 18434: Add tests for K:SE:E::get_fixer_rules
    
    This patch tries to introduce exhaustive tests for this class method.
    I didn't try to provide a regression test for the current bug per-se, but
    cover the current method behaviour as much as I could.
    
    (kidclamp) I added a quick test of _convert_marc_to_json to use the mocking here
    and illuminate what the change does, before the patches this should
    fail (fields are indexed in place of one another), after it should succeed (new indexed fields are appended).
    
    A minor bug is highlighted by this new tests, I'll provide a followup for it.
    
    To test:
    - Run:
      $ sudo koha-shell kohadev
     k$ de kohaclone
     k$ prove t/db_dependent/Koha_Elasticsearch.t
    => FAIL: The returned fixer rules are not the expected ones
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    (cherry picked from commit 2d19a34529bc313922a085a99288e55d8634c8b6)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

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

Summary of changes:
 Koha/SearchEngine/Elasticsearch.pm          |    6 +-
 t/db_dependent/Koha_Elasticsearch.t         |  138 ++++++++++++++++++++++++++-
 t/db_dependent/Koha_Elasticsearch_Indexer.t |   73 +++++++++++++-
 3 files changed, 210 insertions(+), 7 deletions(-)


hooks/post-receive
-- 
main Koha release repository