[koha-commits] main Koha release repository branch new/bug_6801 created. v3.04.00-618-g26acaf3

[Git repo owner]

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, new/bug_6801 has been created
        at  26acaf3dfa24598f9cce1d647079f779e714f874 (commit)

- Log -----------------------------------------------------------------
commit 26acaf3dfa24598f9cce1d647079f779e714f874
Author: Ian Walls <ian.walls at bywatersolutions.com>
Date:   Wed Aug 31 14:43:09 2011 -0400

    Bug 6801: checkoverdues returns unnecessary fields, causing slowness
    
    Explicitly specifies which fields to return in C4::Overdues::checkoverdues
    SQL:  all of biblio, items, and issues, and everything in biblioitems
    EXCEPT marc, marcxml and timestamp.
    
    Bug 6801: member details page taking long time to load when many checkouts present
    
    This patch removes the call to GetMemberDetails in build_issue_data; this heavy-weight
    subroutine was being run for every single item a patron (or their relatives) have checked out.
    Instead, the borrowers first name, surname and cardnumber are added to the GetPendingIssues query.
    I believe this is reasonable since GetPendingIssues can now return issues for multiple borrowers.
    
    Also corrects the $borrowernumber used for GetIssuesCharges and CanItemBeRenewed; was using the borrower whose
    page we were on, NOT the borrower of that specific item (which would be different in the Relatives Checkouts tab).
    
    Template calls to [% scope.borrowername %] are now broken up into [% scope.firstname %] [% scope.surname %].
    
    Signed-off-by: Liz Rea <lrea at nekls.org>
    On my test data, a patron with 180 checkouts (without this patch) would take more than a minute to bring back the circulation.pl and moremember.pl pages.
    With this patch, the time is reduced to 5 or so seconds.
    
    Big ups to Ian for tenaciously hunting this one down.
    
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Signed-off-by: Chris Cormack <chrisc at catalyst.net.nz>

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


hooks/post-receive
-- 
main Koha release repository