[koha-commits] main Koha release repository branch 17.05.x updated. v17.05.03-47-g4c76583

Git repo owner gitmaster at git.koha-community.org
Tue Sep 12 17:14:51 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, 17.05.x has been updated
       via  4c76583e6d3079bad51048f62f43c3c433f1e9bd (commit)
      from  960015819e0c19ef85c458d0bdaec8a24cd86dc3 (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 4c76583e6d3079bad51048f62f43c3c433f1e9bd
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>
    (cherry picked from commit 95f1844e6c46ff51de9f95f99fcdf3369c756fd4)
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

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

Summary of changes:
 tools/inventory.pl |   22 ++++++++--------------
 1 file changed, 8 insertions(+), 14 deletions(-)


hooks/post-receive
-- 
main Koha release repository


More information about the koha-commits mailing list