[koha-commits] main Koha release repository branch master updated. v17.05.00-327-g1da4721

Git repo owner gitmaster at git.koha-community.org
Wed Aug 9 21:56:43 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, master has been updated
       via  1da47215516206d51545e5bc662a34e335127ff6 (commit)
       via  95f1844e6c46ff51de9f95f99fcdf3369c756fd4 (commit)
       via  2fdfbaf0ddbf214c0efb9a3a3c2595a54517f795 (commit)
       via  19a7b498e1f4355ab1e4ee0adb5db299613f3f5c (commit)
      from  5bb56bf336187cde38e1a3b89c792ee6dc231acf (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 1da47215516206d51545e5bc662a34e335127ff6
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Wed Aug 9 16:52:21 2017 -0300

    Bug 17829: (follow-up) Move GetMember to Koha::Patron - routing-lists.pl
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>

commit 95f1844e6c46ff51de9f95f99fcdf3369c756fd4
Author: Fridolin Somers <fridolin.somers at biblibre.com>
Date:   Wed Aug 2 11:21:58 2017 +0200

    Bug 19023 - inventory tool performance
    
    Inventory tool performance seems to be worst since 16.11.
    I think it is because of authorized values computing changed by Bug 17249.
    
    For each subfield of each item, we try to get the authorized value description with Koha::AuthorisedValues->search_by_marc_field.
    But this method does not use cache like Koha::AuthorisedValues->get_description_by_koha_field.
    
    I propose to use Koha::AuthorisedValues->get_description_by_koha_field and also to look for authorized value description only for item fields used in TT : location, notforloan, itemlost, damaged, withdrawn.
    
    I have experimented inventory time on 100 items from 5s to 1s.
    
    Test plan :
    - Without patch
    - Perform inventory with barcode file
    - Check results and mesure execution time
    - Apply patch
    - Reperform inventory with same barcode file
    - Check results is the same
    - Compare execution time
    - Run prove t/db_dependent/Items/GetItemsForInventory.t
    
    Signed-off-by: Owen Leonard <oleonard at myacpl.org>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>

commit 2fdfbaf0ddbf214c0efb9a3a3c2595a54517f795
Author: David Cook <dcook at prosentient.com.au>
Date:   Thu Jul 27 11:58:28 2017 +1000

    Bug 18898 - Some permissions for Reports can be bypassed
    
    If you manually visit the following links when you only have
    permission to run reports, you'll still be able to access the ability
    to create and edit reports:
    
    /cgi-bin/koha/reports/guided_reports.pl?phase=Create%20report%20from%20SQL
    /cgi-bin/koha/reports/guided_reports.pl?phase=Edit%20SQL
    
    This patch ties these 2 unaccounted for phases to the create_reports
    permission.
    
    With patch, issue no longer can be reproduced.
    Signed-off-by: Marc Véron <veron at veron.ch>
    
    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>

commit 19a7b498e1f4355ab1e4ee0adb5db299613f3f5c
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date:   Thu Jul 6 18:10:10 2017 -0300

    Bug 18906: Display all funds the logged in user can use
    
    There are complex conditions to know if a librarian can use a fund, we
    cannot do that with a SQL query, we need to use C4::Budgets::CanUserUseBudget
    
    When this patch is applied then all funds are visible to both
    superlibrarian patrons and patrons with all acq permissions
    
    Signed-off-by: Alex Buckley <alexbuckley at catalyst.net.nz>
    
    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    
    Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>

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

Summary of changes:
 admin/aqbudgets.pl        |    3 ++-
 members/routing-lists.pl  |    2 +-
 reports/guided_reports.pl |    2 +-
 tools/inventory.pl        |   24 +++++++++---------------
 4 files changed, 13 insertions(+), 18 deletions(-)


hooks/post-receive
-- 
main Koha release repository


More information about the koha-commits mailing list