[koha-commits] main Koha release repository branch 18.05.x updated. v18.05.11-90-g8e7bd44

Git repo owner gitmaster at git.koha-community.org
Thu May 30 16:10:26 CEST 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, 18.05.x has been updated
       via  8e7bd44f9446524d3d21c1d3868af2891dba812f (commit)
       via  88af22a44acb22af815cc710372f034cb4c3c91e (commit)
      from  2f3be3095c575c226b16b2caa39c5cfa8892256a (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 8e7bd44f9446524d3d21c1d3868af2891dba812f
Author: Tomas Cohen Arazi <tomascohen at theke.io>
Date:   Tue Apr 30 15:48:14 2019 -0300

    Bug 22813: (follow-up) Add missing condition
    
    This patch restores a missing condition. $patron gets renamed to make
    it obvious that it related to the logged in user.
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    
    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit 45b1ff8dfa1054590544caff200b307144bd02cf)
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    
    Signed-off-by: Lucas Gass <lucas at bywatersolutions.com>

commit 88af22a44acb22af815cc710372f034cb4c3c91e
Author: Tomas Cohen Arazi <tomascohen at theke.io>
Date:   Tue Apr 30 12:30:25 2019 -0300

    Bug 22813: remove repetitive queries inside two nested loops in searchResults
    
    This patch moves a query on Koha::Patrons and then the related
    Koha::Patron::Category that needlessly happens inside two nested loops
    (all items of all MARC records in the resultset).
    
    The Koha::Patron and Koha::Patron::Category are always the same as it is
    fetched from C4::Context->userenv each time.
    
    To test:
    - Run:
      $ kshell
     k$ prove t/db_dependent/Search.t
    => SUCCESS: Tests pass
    - Apply this patch
    - Run:
     k$ prove t/db_dependent/Search.t
    => SUCCESS: Tests still pass!
    - Sign off :-D
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
    
    Signed-off-by: Liz Rea <wizzyrea at gmail.com>
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    
    Signed-off-by: Nick Clemens <nick at bywatersolutions.com>
    (cherry picked from commit 10efe4e3f239c47b847c53f06f82245e07d54759)
    Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>
    (cherry picked from commit 738aafb92dcfb73f51f9302fa9506040a8cc2930)
    
    Signed-off-by: Lucas Gass <lucas at bywatersolutions.com>

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

Summary of changes:
 C4/Search.pm |   13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)


hooks/post-receive
-- 
main Koha release repository


More information about the koha-commits mailing list