[Koha-patches] [PATCH] [SIGNED-OFF] Bug 7057: CheckReserves uses GetMemberDetails unnecessarily

Nicole C. Engard nengard at bywatersolutions.com
Wed Oct 19 21:40:25 CEST 2011


From: Ian Walls <ian.walls at bywatersolutions.com>

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 at bywatersolutions.com>
---
 C4/Reserves.pm |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/C4/Reserves.pm b/C4/Reserves.pm
index f7a2e72..eff4e9f 100644
--- a/C4/Reserves.pm
+++ b/C4/Reserves.pm
@@ -828,7 +828,7 @@ sub CheckReserves {
             } else {
                 # See if this item is more important than what we've got so far
                 if ( $res->{'priority'} && $res->{'priority'} < $priority ) {
-                    my $borrowerinfo=C4::Members::GetMemberDetails($res->{'borrowernumber'});
+                    my $borrowerinfo=C4::Members::GetMember(borrowernumber => $res->{'borrowernumber'});
                     my $iteminfo=C4::Items::GetItem($itemnumber);
                     my $branch=C4::Circulation::_GetCircControlBranch($iteminfo,$borrowerinfo);
                     my $branchitemrule = C4::Circulation::GetBranchItemRule($branch,$iteminfo->{'itype'});
-- 
1.7.2.3



More information about the Koha-patches mailing list