[koha-commits] main Koha release repository branch 16.11.x updated. v16.11.09-55-ga592755

Git repo owner gitmaster at git.koha-community.org
Wed Jul 19 23:53:40 CEST 2017


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, 16.11.x has been updated
       via  a5927551a0f0f882a0a8f0c5a2a9a360af095def (commit)
       via  2a8d6a886575e32bafa5f1bf06e619a48fae58a9 (commit)
       via  f67608a636cb3b20a8824fc85b86532692e808d2 (commit)
       via  bc17760bc2ac665ee6697c953d43b025b68ea2a6 (commit)
       via  e0bf7dc8d6fde72678d6011609b2b09cae5978b6 (commit)
       via  fb17b3c9c6e5d8dab44b8187c76d1fec04fbba14 (commit)
       via  36898c538bac3d861025635d96939fed8f927e10 (commit)
       via  210f7cdcf340596542fd99e1ab603020c972a3b7 (commit)
       via  487d7c55d60cb1ae45d13a8bd34ef4ca207c7dcb (commit)
       via  90ae43d289d195ab36f21bf9ffc5a4926e95413f (commit)
       via  181a8d85b2aed14a76e7839c6ec23b1e2bd9237d (commit)
       via  605f18664625e914c4e1569d96a8a9b8dbfccfe7 (commit)
       via  a41920669aeafc70d7edbd2c32fbfeeaa4f8f55d (commit)
      from  37c8278d73d98a2d9d16c129b4b1836cdce9a440 (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 a5927551a0f0f882a0a8f0c5a2a9a360af095def
Author: Baptiste Wojtkowski <baptiste.wojtkowski at biblibre.com>
Date:   Tue Jul 11 09:12:37 2017 +0000

    Bug 18613: Add unit tests for getletter
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    (cherry picked from commit ad367a47003f341e1c517c6fbcc96246cbf21609)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>
    (cherry picked from commit c0021af422b7e524718c2e2fdcee552e6166c7ef)
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>

commit 2a8d6a886575e32bafa5f1bf06e619a48fae58a9
Author: Baptiste Wojtkowski <baptiste.wojtkowski at biblibre.com>
Date:   Fri May 19 09:23:17 2017 +0000

    Bug 18613: Remove letter rule correctly as superlibrarian
    
    TEST PLAN
    
    (reproducing bug)
    1 - Set 'IndependantBranches' to enable
    2 - Log in as Superlibrarian.
    3 - Create 3 rules
        - 1 for all librairies (1)
        - 1 for the library of the superlibrarian (2)
        - 1 for another library (3)
    4 - Try to delete them and see them and see that (3) isn't the good one
    
    (checking patch)
    5 - Apply patch
    6 - Try again 3-
    7 - Connect as not superlibrarian
    8 - Try again 3-
    
    Signed-off-by: Alex Buckley <alexbuckley at catalyst.net.nz>
    Signed-off-by: Julian Maurice <julian.maurice at biblibre.com>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    (cherry picked from commit 648e2a2b808efac22a8b88545d53b05112a1f6f0)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>
    (cherry picked from commit 513205b5770e8ebf8d489c9ea7d671e136fc4a25)
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>

commit f67608a636cb3b20a8824fc85b86532692e808d2
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date:   Wed Jul 12 13:33:51 2017 +0200

    Bug 18901: [QA Follow-up] Do not translate zzpref either
    
    Adding one backslash makes a difference :)
    We need to escape the dot in the regex to exclude a file like zzpref
    from translation too. Perfect_regexes++
    
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    (cherry picked from commit a2c9513b1c3449a34f942d8fa53f8ce1fe374e3f)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>
    (cherry picked from commit 6896fdc5b31178ad9cde381d2b3b3edb2e8e37f9)
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>

commit bc17760bc2ac665ee6697c953d43b025b68ea2a6
Author: Julian Maurice <julian.maurice at biblibre.com>
Date:   Thu Jul 6 14:39:20 2017 +0200

    Bug 18901: Sysprefs translation: translate only *.pref files (not *.pref*)
    
    Text editors can create temporary files in this folder and this can
    confuse the translator.
    For instance, vim can create a file named '.opac.pref.swp' which will
    make the translator dies with the following error message:
    
    Can't use string ("b0VIM 8.0") as a HASH ref while "strict refs" in use
    at LangInstaller.pm line 248.
    
    Test plan:
    1. echo 'Oops' > .../en/modules/admin/preferences/whatever.pref.whatever
    2. cd misc/translator && ./translate update fr-FR
    3. Verify that you have the error message mentioned above
    4. Apply patch
    5. cd misc/translator && ./translate update fr-FR
    6. No more errors!
    
    Signed-off-by: Frédéric Demians <f.demians at tamil.fr>
    
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    (cherry picked from commit ec51335e9cc4e090ef80f592cc0cc7a0786661f8)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>
    (cherry picked from commit 4f8fac147633c7272959e058dc7ea56ae7ecc403)
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>

commit e0bf7dc8d6fde72678d6011609b2b09cae5978b6
Author: Mark Tompsett <mtompset at hotmail.com>
Date:   Wed Jun 7 12:53:52 2017 -0400

    Bug 18748: Noisy t/db_dependent/AuthorisedValues.t
    
    This noise is from a failure. This patch expands the delete
    to 952$c for the ACQ framework as per comment #5.
    
    TEST PLAN
    ---------
    
    insert into marc_subfield_structure (tagfield,tagsubfield,liblibrarian, libopac, repeatable, mandatory, kohafield,tab,authorised_value,authtypecode,value_builder,isurl,hidden,frameworkcode,seealso,link,defaultvalue,maxlength) values (952,'c','Shelving location','Shelving location',0,0,'items.location',10,'LOC','','',0,0,'ACQ','','',null,9999);
    -- this makes sure you have a pre-existing 952$c ACQ record.
    
    prove t/db_dependent/AuthorisedValues.t
    -- should have ugly message like in comment #0
    apply patch
    prove t/db_dependent/AuthorisedValues.t
    -- should be green
    run koha qa test tools
    
    Signed-off-by: Lee Jamison <ldjamison at marywood.edu>
    
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    (cherry picked from commit 441f6fac33975f7e0f7c96b262bc0c0ee3b0c82e)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>
    (cherry picked from commit 97498a6a7b2ab43fc1d0a42bb93b91f3020e9ffd)
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>

commit fb17b3c9c6e5d8dab44b8187c76d1fec04fbba14
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Mon Jul 10 11:56:46 2017 -0300

    Bug 18854: Protect few other occurrences of offset
    
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    (cherry picked from commit 9f47bd88252e201f9f06fd734dae154d9bc41543)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>
    (cherry picked from commit 50431a49849089fd1960accc26b934f29763cd99)
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>

commit 36898c538bac3d861025635d96939fed8f927e10
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Wed Jul 5 10:44:55 2017 -0300

    Bug 18854: Make sure offset will not be < 0 - protect from DoS
    
    There was a bug that meant a very large offset in the search params
    will cause the search script to run forever (or long enough to crash
    the machine)
    
    To test
    
    1/ Get ready with sudo top so you can kill the thread before it causes
    your machine to OOM
    2/ Hit a page like yourdomain.com/cgi-bin/koha/opac-search.pl?q=1&offset=-9999999999999999999
    3/ Notice the process runs for a long time
    4/ Kill the process
    5/ Apply the patch
    6/ Hit the page again, notice the it loads (offset is set to zero)
    7/ Do the same to search in the staff client
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
    Signed-off-by: Julian Maurice <julian.maurice at biblibre.com>
    
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    (cherry picked from commit aabf001232fc46ee04cecb3a43fe8c0366a091df)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>
    (cherry picked from commit 9939c7f3bf961e5b7789f4d64e0fc074408b71b1)
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>

commit 210f7cdcf340596542fd99e1ab603020c972a3b7
Author: Chris Cormack <chris at bigballofwax.co.nz>
Date:   Sun Jun 25 17:34:12 2017 +1200

    Bug 18854 - Protect from DOS
    
    There was a bug that meant a very large offset in the search params
    will cause the search script to run forever (or long enough to crash
    the machine)
    
    To test
    
    1/ Get ready with sudo top so you can kill the thread before it causes
    your machine to OOM
    2/ Hit a page like yourdomain.com/cgi-bin/koha/opac-search.pl?q=1&offset=-9999999999999999999
    3/ Notice the process runs for a long time
    4/ Kill the process
    5/ Apply the patch
    6/ Hit the page again, notice the it loads (offset is set to zero)
    7/ Do the same to search in the staff client
    
    Signed-off-by: Mirko Tietgen <mirko at abunchofthings.net>
    
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    Amended: changed -2 to 0 in opac-search.pl.
    Signed-off-by: Julian Maurice <julian.maurice at biblibre.com>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    (cherry picked from commit 99b32717cd2596ce89a2d46b8cb4ddbba2dea5ad)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>
    (cherry picked from commit a64d14db20ac55fe4bcc8de06207c516d2237788)
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>

commit 487d7c55d60cb1ae45d13a8bd34ef4ca207c7dcb
Author: Owen Leonard <oleonard at myacpl.org>
Date:   Tue Jun 6 15:50:19 2017 +0000

    Bug 17639 - Remove white filling inside of Koha logo
    
    This patch modifies the Koha logo SVG file to remove the fill from two
    letters. This patch also optimizes the file and converts the text object
    to paths for better cross-platform portability.
    
    To test you could:
    
    - Open the file in an editor and confirm that the change is correct
      - or -
    - Open the file in a browser and use the code inspector to add a
      background-color attribute to the top-level <svg> tag. The logo should
      appear transparent, with no white fills.
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    (cherry picked from commit 068db9aaabb0694b4d12404bda16525efad0fde7)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>
    (cherry picked from commit cffb4a26b3264137130d7a1a3c287d1f17c05b77)
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>

commit 90ae43d289d195ab36f21bf9ffc5a4926e95413f
Author: Aleisha Amohia <aleishaamohia at hotmail.com>
Date:   Thu Jul 13 05:16:53 2017 +0000

    Bug 18934: Preventing warns in Admin -> SMS providers
    
    This patch initialises the $op variable to prevent the warns
    
    To test:
    1) Go to admin -> sms providers
    2) Notice warns in error log
    3) Apply patch and refresh page
    4) Notice warns are gone
    
    Sponsored-by: Catalyst IT
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    (cherry picked from commit 88ab0ef5791d2b428399690c505ac42aba4616df)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>
    (cherry picked from commit adbd961d82fa6aebcc3f93f2dec6af1c8127aab9)
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>

commit 181a8d85b2aed14a76e7839c6ec23b1e2bd9237d
Author: Aleisha Amohia <aleishaamohia at hotmail.com>
Date:   Tue Jun 27 01:40:48 2017 +0000

    Bug 18858: Prevent warn when deleting a borrower debarment
    
    To test:
    1) Look at intranet log
    2) Go to delete a debarment on a borrower
    3) Notice warn
    4) Apply patch
    5) Add a new debarment
    6) Delete this debarment
    7) Notice warn is gone
    
    Sponsored-by: Catalyst IT
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    (cherry picked from commit 7ed66a6c8ae1c3c824e496c2d312e23db1dd8e41)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>
    (cherry picked from commit 7d2bd0482033ac106434aca59bab6b201a592fef)
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>

commit 605f18664625e914c4e1569d96a8a9b8dbfccfe7
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Mon Jul 10 18:09:23 2017 -0300

    Bug 18577: Add FIXME about the missing FK
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    (cherry picked from commit 5d6813e0708a54a9113cd74cc6a2558c77b5fc33)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>
    (cherry picked from commit 2c341cebba7d303226b791d762b6bd745acb51f3)
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>

commit a41920669aeafc70d7edbd2c32fbfeeaa4f8f55d
Author: Nick Clemens <nick at bywatersolutions.com>
Date:   Fri Jun 23 14:11:44 2017 +0000

    Bug 18577 - Importing a batch using a framework not fully set up causes and endless loop
    
    To test:
    1 - Create a new framework - don't set it up
    2 - Stage some records for import
    3 - Manage the import, note your new framwork is in the list
    4 - Apply patch
    5 - Reload the page
    6 - Note the framework is no longer an option
    
    Followed test plan,  worked as intended
    Signed-off-by: Alex Buckley <alexbuckley at catalyst.net.nz>
    
    Signed-off-by: Julian Maurice <julian.maurice at biblibre.com>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
    (cherry picked from commit 96b1f92ae218bc889d1490385f934deec28f630a)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>
    (cherry picked from commit 334fdcea429bb49ade3f56b3a5998624481900de)
    Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>

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

Summary of changes:
 C4/AuthoritiesMarc.pm                              |    2 +-
 C4/Letters.pm                                      |    8 +-
 C4/Search.pm                                       |    1 +
 Koha/Schema/Result/BiblioFramework.pm              |    9 ++
 Koha/SearchEngine/Elasticsearch/Search.pm          |    4 +-
 admin/auth_subfields_structure.pl                  |    3 +-
 admin/auth_tag_structure.pl                        |    3 +-
 admin/marc_subfields_structure.pl                  |    3 +-
 admin/sms_providers.pl                             |    2 +-
 catalogue/search.pl                                |    1 +
 members/mod_debarment.pl                           |    2 +-
 .../koha3-staff-client-logo.svg                    |  162 +-------------------
 misc/translator/LangInstaller.pm                   |    2 +-
 opac/opac-search.pl                                |    1 +
 t/db_dependent/AuthorisedValues.t                  |    2 +
 t/db_dependent/Letters.t                           |   43 ++++--
 tools/manage-marc-import.pl                        |    2 +-
 17 files changed, 65 insertions(+), 185 deletions(-)


hooks/post-receive
-- 
main Koha release repository


More information about the koha-commits mailing list