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]
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 <email@example.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.