http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6811 Ian Walls <ian.walls@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |ian.walls@bywatersolutions. | |com Patch Status|Needs Signoff |Failed QA --- Comment #5 from Ian Walls <ian.walls@bywatersolutions.com> 2011-10-07 16:54:37 UTC --- This patch uses GetMemberDetails() universally on every renewal, whether or not a notice will actually be generated. The only bit of information used from this expensive call is the categorycode. It would be better, therefore, to use GetMember() instead, which is a much lighter-weight subroutine and will result in less of a performance hit. For example, GetMemberDetails() includes a call to patronflags(), which calls checkoverdues(), which pulls in all the biblio, biblioitems (minus marc), items and issues data for every overdue that patron has... -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.