[koha-commits] main Koha release repository branch 17.11.x updated. v17.11.10-48-g6830a61

[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.11.x has been updated
       via  6830a61918a8ece7a3254cfe3473c48a6630f284 (commit)
       via  25e62577902fe7b426d545d5f17e74452980f7cf (commit)
      from  94446917946de314302345f0b28503fcbea38c70 (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 6830a61918a8ece7a3254cfe3473c48a6630f284
Author: Fridolin Somers <fridolin.somers at biblibre.com>
Date:   Wed Feb 7 15:06:56 2018 +0100

    Bug 20151: always use current language for stemming
    
    When stemming is enabled, in catalog searching "C4::Search::_build_stemmed_operand" will transform query operand into stemmed operand using stemmer Lingua::Stem::Snowball with a specified language.
    This stemmer returns undef stemmed operand if no language is defined.
    
    In main catalog search (catalogue/search.pl) current language is used.
    But in other pages "acqui/neworderbiblio.pl" and "cataloguing/addbooks.pl" no language is defined so stemmed operand is empty and so stemming is not applied.
    
    This patch corrects by returning in "C4::Search::_build_stemmed_operand" operand without change if no langage is defined.
    And uses current langage in pages "acqui/neworderbiblio.pl" and "cataloguing/addbooks.pl" so all catalog search uses stemming.
    
    Test plan :
    1) Enable system preferences QueryStemming and QueryWeightFields
    2) Disable system preferences QueryAutoTruncate, QueryFuzzy and UseQueryParser
    3) Go to intranet main page and click on "Search the catalog" tab
    4) Perform a search (without index) that uses the stemming, for example searching for "years" will also match "year"
    5) Note how many results you get, for example "year" gets 24 results and "years" gets 24 results
    6) Go to "Cataloging" module
    7) Perform a search on same word in "Cataloging search" tab
    8) Note how many results you get
    9) Without patch you get fewer results than first search (step 5) because stemming is not applied, for example "year" gets 11 results and "years" gets 15 results
    10) With patch you get the same results as first search (step 5) because stemming is applied, for example "year" and "years" gets 24 results
    11) Same tests in aquisition module
    12) On a basket, click "Add to basket" and perform a search in "From an existing record"
    
    Signed-off-by: David Nind <david at davidnind.com>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    
    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit 78a692dee4523d04986ec394442c3a6d80ea0771)
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    (cherry picked from commit 2170698ab9da24d306286991b55257c4c3317e65)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit 25e62577902fe7b426d545d5f17e74452980f7cf
Author: Magnus Enger <magnus at libriotech.no>
Date:   Thu Sep 6 13:23:17 2018 +0000

    Bug 21322: process_message_queue.pl --type should take an argument
    
    The help for this script says:
    -t --type: If supplied, only processes this type of message ( email, sms )
    
    Currently, the type argument is set up wrong, so it does not look
    for an argument. This patch fixes that.
    
    To test, run this command (should work in kohadevbox) or something
    similar:
    
    $ sudo koha-shell -c "perl \
    /home/vagrant/kohaclone/misc/cronjobs/process_message_queue.pl -v \
    --type=sms" kohadev
    
    This should give the following error: "Option type does not take an argument".
    Apply the patch and run the same command again. This should not give an
    error.
    
    Signed-off-by: Mark Tompsett <mtompset at hotmail.com>
    
    Signed-off-by: Josef Moravec <josef.moravec at gmail.com>
    
    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit d74a05d4fe81a599c030024f7944d2bdcca43d12)
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    (cherry picked from commit a3f097958cae68e4f3f16c8eb5aa5411d2488bc2)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

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

Summary of changes:
 C4/Search.pm                           |    3 +++
 acqui/neworderbiblio.pl                |    5 ++++-
 cataloguing/addbooks.pl                |    5 ++++-
 misc/cronjobs/process_message_queue.pl |    2 +-
 4 files changed, 12 insertions(+), 3 deletions(-)


hooks/post-receive
-- 
main Koha release repository