[koha-commits] main Koha release repository branch 19.05.x updated. v19.05.10-52-g4ce9cf2d50

Thu May 14 17:26:43 CEST 2020

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, 19.05.x has been updated
       via  4ce9cf2d500129d3ea3b9833ddbb2e5284bec871 (commit)
       via  de3dcc4904379fd4ac8ece1840ab0cee50441201 (commit)
       via  63914840c620fda1b7edf5e831602db1bce179c1 (commit)
       via  655958ca124d9969abfde40876a496f8f5a173f9 (commit)
      from  b2a046f25d0a48d1262778b76958f8c9382752f1 (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 4ce9cf2d500129d3ea3b9833ddbb2e5284bec871
Author: Fridolin Somers <fridolin.somers at biblibre.com>
Date:   Wed Mar 11 17:39:18 2020 +0300

    Bug 21865: allow test arg alone

    Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>

    Signed-off-by: Joy Nelson <joy at bywatersolutions.com>
    (cherry picked from commit 8523817dcc4d2219138b1c8600f289cbe9a009ee)

    Signed-off-by: Lucas Gass <lucas at bywatersolutions.com>

commit de3dcc4904379fd4ac8ece1840ab0cee50441201
Author: Fridolin Somers <fridolin.somers at biblibre.com>
Date:   Mon Nov 19 15:45:49 2018 +0100

    Bug 21865: add confirm to remove_unused_authorities.pl script

    Like in most scripts in misc, add confirm argument
    to ensure script is not run without knowing what it does.

    Test plan:
    1) Run misc/migration_tools/remove_unused_authorities.pl -h
    2) You see help line for confirm
    3) Run misc/migration_tools/remove_unused_authorities.pl
    4) You see help and script does nothing
    5) Run misc/migration_tools/remove_unused_authorities.pl -c
    6) Script runs like wanted

    Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>

    Signed-off-by: Joy Nelson <joy at bywatersolutions.com>
    (cherry picked from commit ace1f665dfdcb7dd2fe1e864191556432e9c5ef6)

    Signed-off-by: Lucas Gass <lucas at bywatersolutions.com>

commit 63914840c620fda1b7edf5e831602db1bce179c1
Author: Fridolin Somers <fridolin.somers at biblibre.com>
Date:   Mon Nov 19 15:19:46 2018 +0100

    Bug 21865: ES compatiblity of remove_unused_authorities.pl

    The script misc/migration_tools/remove_unused_authorities.pl directly checks if Zebra search is OK.

    This patch changes so that this test is only if Zebra is the search engine.
    It also adds a test on the search off any authority number indexed (index 'an').
    With Zebra its : an,alwaysmatches=''
    With ES its : an:*
    This test ensure that biblios records are indexed and that not all autorities will be deleted.

    Test plan:
    1) On a catalog create a new authority
    2) Use Zebra in systempreference SearchEngine
    3) Stop Zebra server
    4) Run misc/migration_tools/remove_unused_authorities.pl -c
    5) The script does nothing and says :
       Zebra server seems not to be available. This script needs Zebra runs.
    6) Restart Zebra server
    7) Delete biblio index base
    8) Run misc/migration_tools/remove_unused_authorities.pl -c
    9) The script does nothing and says :
       Searching authority number in biblio records seems not to be available : an,alwaysmatches=''
    10) Use ElasticSearch in systempreference SearchEngine
    11) Delete biblio index base
    12) Run misc/migration_tools/remove_unused_authorities.pl -c
    13) The script does nothing and says :
        Searching authority number in biblio records seems not to be available : an:*

    Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>

    Signed-off-by: Joy Nelson <joy at bywatersolutions.com>
    (cherry picked from commit ee25606e8a85fb8a95284b6ad678dd18e97dfd02)

    Signed-off-by: Lucas Gass <lucas at bywatersolutions.com>

commit 655958ca124d9969abfde40876a496f8f5a173f9
Author: Fridolin Somers <fridolin.somers at biblibre.com>
Date:   Mon Nov 19 14:46:05 2018 +0100

    Bug 21865: improve remove_unused_authorities.pl script

    remove_unused_authorities.pl script can be improved.

    This patch changes changes verbosity so than test mode can be used
    to know the autorities that are used and those that can be deleted.
    It also writes a line in output if limited authority type(s).

    This patch also removes the unused vars $thresholdmin and $thresholdmax.

    It also changes the query to use SQL with parameters for authority types.

    Test plan :
    1) On a catalog create a new authority
    2) Be sure catalog is well indexed
    3) Run misc/migration_tools/remove_unused_authorities.pl -t
    4) You will see the line :
       *** Testing only, authorities will not be deleted. ***
    5) You will see lines of :
       authid=x type=y : used X time(s)
    6) You will see the line for the authority created in 1) :
       authid=x type=y : can be deleted
    7) You will see at the end :
       x authorities parsed
       y can be deleted because unused
       z unchanged because used
    8) Run misc/migration_tools/remove_unused_authorities.pl
    9) You don't see the line :
       *** Testing only, authorities will not be deleted. ***
    10) You will see lines of :
       authid=x type=y : used X time(s)
    11) You will see the line for the authority created in 1) :
       authid=x type=y : deleted
    12) You will see at the end :
       x authorities parsed
       y deleted because unused
       z unchanged because used
    13) Run misc/migration_tools/remove_unused_authorities.pl --auth NP --auth CO
    14) You see the line :
        Restricted to authority type(s) : NP,CO.

    Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>

    Signed-off-by: Joy Nelson <joy at bywatersolutions.com>
    (cherry picked from commit 36a84f4264e5c03b900eb850c93cc4eb0befd9e7)

    Signed-off-by: Lucas Gass <lucas at bywatersolutions.com>

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

Summary of changes:
 misc/migration_tools/remove_unused_authorities.pl | 87 +++++++++++++++--------
 1 file changed, 57 insertions(+), 30 deletions(-)

hooks/post-receive
-- 
main Koha release repository