CheckReserves, which is a workhorse subroutine, calls GetMemberDetails possibly MANY times during it's execution (for each reserve on a biblio). All it uses from this call is branchcode. This is massively inefficient at large scales. Replacing this with GetMember would be much, much more efficient.
Created attachment 5967 [details] [review] Proposed patch Replaces CheckReserves call to GetMemberDetails with a call to GetMember. Much more efficient, since only branchcode is needed.
Created attachment 5993 [details] [review] [SIGNED-OFF] Bug 7057: CheckReserves uses GetMemberDetails unnecessarily Replaces call to GetMemberDetails with a call to GetMember. Much more efficient, and since only branchcode is used, no required data is lost. Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Also marking as Passed QA, for RM's discretion to integrate
Pushed, please test
There have been no further reports of problems so I am marking this bug resolved.