[koha-commits] main Koha release repository branch 17.11.x updated. v17.11.13-9-gb74f4e9

Wed Jan 2 11:12:05 CET 2019

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  b74f4e9cce345801a062d81ea7e0dc5a697f975a (commit)
       via  43e626493ba758bcd46b5e70d8065d0d33c1b8fb (commit)
       via  e55b5f9bace77e3ec4250cedc7baca5e30efd6f2 (commit)
       via  d2d640a55a78c5edc9eb1230f626525f0a775afa (commit)
       via  e2e7dc62202957428fc0170573af8e347cbf5e58 (commit)
       via  172fed8521f939480d00a68bb620a5e634b13022 (commit)
       via  a704b28c495ee115961daf034d87b065edaf8fe7 (commit)
       via  a0653c28f0477772c2752c0026ce56cb34bd2e22 (commit)
       via  cdeb4c8051412184f22eb5665c66da2850941f42 (commit)
      from  eac60108318813b30f52c1a65bf8c6e8fecab30a (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 b74f4e9cce345801a062d81ea7e0dc5a697f975a
Author: Julian Maurice <julian.maurice at biblibre.com>
Date:   Fri Dec 7 10:19:44 2018 +0100

    Bug 21908: (QA follow-up) Remove useless parentheses in query

    Signed-off-by: Julian Maurice <julian.maurice at biblibre.com>

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit 875f3162c8008ef288adfcd18ada490cf06ecc15)
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    (cherry picked from commit 0c99c3a30a4ebe2ef18db1a6d1e48a99ef7ea0b6)

    Signed-off-by: Lucas Gass <lucas at bywatersolutions.com>
    (cherry picked from commit 247b57388a168b358878c1acd50b8591a3335808)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit 43e626493ba758bcd46b5e70d8065d0d33c1b8fb
Author: Fridolin Somers <fridolin.somers at biblibre.com>
Date:   Thu Nov 29 10:50:06 2018 +0100

    Bug 21908: Add DISTINCT biblionumber to rebuild_zebra.pl

    Zebra indexing script misc/migration_tools/rebuild_zebra.pl as a table arg to allow filtering.
    When using table=items we should use DISTINCT(biblionumber) to avoid indexing several times the same biblio record when it has several items.
    This patch adds DISTINCT(biblionumber) in all cases it does not harm if its already unique.

    Test plan :
    1) Be sur you have a biblio record with biblionumber 1 with 3 items
    2) Run misc/migration_tools/rebuild_zebra.pl -v -b --table items --where="biblionumber=1"
    3) Without patch you see "Records exported: 3", with patch only one
    4) Check indexing works well

    Signed-off-by: Pierre-Marc Thibault <pierre-marc.thibault at inLibro.com>
    Signed-off-by: Julian Maurice <julian.maurice at biblibre.com>

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit a1b08269a7031417312e6265ace9ef5b995933fa)
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    (cherry picked from commit 397f2e850ced3acaeb8d0f103694262ba339d63f)

    Signed-off-by: Lucas Gass <lucas at bywatersolutions.com>
    (cherry picked from commit 7008d6ae03a1f5dac6b6c3b7fd42a13d3393abab)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit e55b5f9bace77e3ec4250cedc7baca5e30efd6f2
Author: Fridolin Somers <fridolin.somers at biblibre.com>
Date:   Thu Nov 29 10:15:00 2018 +0100

    Bug 21908: Add biblio_metadata to rebuild_zebra.pl tables

    Zebra indexing script misc/migration_tools/rebuild_zebra.pl as a table arg to allow filtering.
    It is missing biblio_metadata to allow filtering on MARCXML with ExtractValue.

    Test plan :
    1) Be sur you have a biblio record with biblionumber 1
    2) Run misc/migration_tools/rebuild_zebra.pl -h
    3) You see : --table specify a table (can be items, biblioitems, biblio, biblio_metadata) to retrieve biblionumber to index.
    4) Run misc/migration_tools/rebuild_zebra.pl -v -b --table biblio_metadata --where="biblio_metadata.biblionumber=1"
    5) Check you dont have SQL errors

    Signed-off-by: Pierre-Marc Thibault <pierre-marc.thibault at inLibro.com>
    Signed-off-by: Julian Maurice <julian.maurice at biblibre.com>

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit c36597bc0c72cc31249c02af5aaaf1c1f977ee50)
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    (cherry picked from commit 33b8f3ff1df670d4512695213d004b1a189f9c25)

    Signed-off-by: Lucas Gass <lucas at bywatersolutions.com>
    (cherry picked from commit 2f7868e9c6d3a1646b9c0d9e8736301b317750e9)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit d2d640a55a78c5edc9eb1230f626525f0a775afa
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Fri Nov 16 07:01:39 2018 -0500

    Bug 21849: Two useless Koha::Account::Offset->new calls need attention

    Came across those calls in bug 20598 in _FixOverduesOnReturn

            Koha::Account::Offset->new(
                {
                    debit_id => $accountline->id,
                    type => 'Forgiven',
                    amount => $amountoutstanding * -1,
                }
            );

    This does nothing if you don't store data.

    Test Plan:
    1) Apply this patch
    2) Set up 2 items with overdue fines
    3) Return one with dropbox mode
    4) Note the dropbox account offset is created
    5) Return one with full fine forgiveness
    6) Note the forgiven account offset is created

    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>

    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit a2b5407ba5c74f826c1665c1a2ef1ff6d52689fd)
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    (cherry picked from commit ff3d3776a4d9b52da49b6ad4b2a1dde1da9425ee)

    Signed-off-by: Lucas Gass <lucas at bywatersolutions.com>
    (cherry picked from commit 56cbc940acba33003cb49f3edc964976dd2e0340)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit e2e7dc62202957428fc0170573af8e347cbf5e58
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Fri Nov 16 09:54:39 2018 -0500

    Bug 21849: Add unit tests

    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>

    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit 2dbaaa300877ff6d96ae84b5a703753d97b6c340)
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    (cherry picked from commit aa565b8e9919ee414d0693f391b141985ebacb93)

    Signed-off-by: Lucas Gass <lucas at bywatersolutions.com>
    (cherry picked from commit 0a6c0c32604ebbd89ccff8d2dc0ff4a11bd204dd)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit 172fed8521f939480d00a68bb620a5e634b13022
Author: Nick Clemens <nick at bywatersolutions.com>
Date:   Wed Nov 7 01:32:26 2018 +0000

    Bug 21778: Sort user list by surname,firstname unless user cannot manage patron lists OR edit patrons

    To test:
    1 - Create four users:
        1 superlibrarian (or with both edit patrons and manage patron
                lists)
        1 with only edit patrons and catalogue
        1 with only add to list and catalogue
        1 with catalogue but neither edit patrons or manage patron lists
    2 - Search patrons signed in as each user
    3 - Note different sorting
    4 - Apply patch
    5 - Try searching with each patron
    6 - Results should always be sorted by surname, firstname
    7 - Only in the case of neither extra permission should the checkboxes
    be absent

    Signed-off-by: Devinim <kohadevinim at devinim.com.tr>

    Signed-off-by: Josef Moravec <josef.moravec at gmail.com>

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit 96b3889390f8efe9851aaa9c2cafa9e20bda3f43)
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    (cherry picked from commit f228847155bc46433cbad4f834c2df89734b9c5c)

    Signed-off-by: Jesse Maseto <jesse at bywatersolution.com>
    (cherry picked from commit f0b6ec9e3934badb1a9b6c8a165a3a4012d83186)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit a704b28c495ee115961daf034d87b065edaf8fe7
Author: Jesse Maseto <jesse at bywatersolution.com>
Date:   Wed Dec 19 17:46:58 2018 +0000

    Bug 21962: (RMaint follow-up) Fixed bad rebase.

    Signed-off-by: Jesse Maseto <jesse at bywatersolutions.com>
    (cherry picked from commit b60fa7296b4c54ed30b568cb429a557933c876de)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit a0653c28f0477772c2752c0026ce56cb34bd2e22
Author: Nick Clemens <nick at bywatersolutions.com>
Date:   Thu Dec 6 12:42:45 2018 +0000

    Bug 21962: Fix authority search using 'all'

    To test:
    1 - Set search engine to Zebra
    2 - Have some authorities and indexes up to date
    3 - Search authorities with option 'search entire record'
    4 - No results
    5 - Apply patch
    6 - Run unit tests, they pass!
    7 - Repeat search
    8 - Results!

    Signed-off-by: Pierre-Marc Thibault <pierre-marc.thibault at inLibro.com>

    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit 3adf7708d544338250703165fe7e87f2690d8639)
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    (cherry picked from commit a110125338202f30cec0bc061141d82c80652f30)
    (cherry picked from commit f6f5de147af71d7ff6f8eb505c58c272d26173ee)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit cdeb4c8051412184f22eb5665c66da2850941f42
Author: Nick Clemens <nick at bywatersolutions.com>
Date:   Thu Dec 6 12:41:41 2018 +0000

    Bug 21962: Unit tests

    Signed-off-by: Pierre-Marc Thibault <pierre-marc.thibault at inLibro.com>

    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>

    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit 39c93e87f648978ec7907220710efaf64e556ea0)
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    (cherry picked from commit d6b7e00b9cd04e15c472b2f7b075d85886ffa3e5)
    (cherry picked from commit f9a4e3656102d982923989ce49641ff0a267322f)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

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

Summary of changes:
 C4/AuthoritiesMarc.pm                                     |    3 +++
 C4/Circulation.pm                                         |    4 ++--
 koha-tmpl/intranet-tmpl/prog/en/modules/members/member.tt |    2 +-
 misc/migration_tools/rebuild_zebra.pl                     |    6 +++---
 t/db_dependent/Circulation.t                              |    8 +++++++-
 t/db_dependent/Search.t                                   |    8 ++++++--
 6 files changed, 22 insertions(+), 9 deletions(-)

hooks/post-receive
-- 
main Koha release repository