Bug 12868: Wrong variable used for borrower number
authorMark Tompsett <mtompset@hotmail.com>
Wed, 3 Sep 2014 23:15:12 +0000 (19:15 -0400)
committerMason James <mtj@kohaaloha.com>
Fri, 16 Jan 2015 23:39:52 +0000 (12:39 +1300)
commit05a9deaacd381a3be18690762ed6d547b6819b22
tree413fb1a380ef341f45118a681b646c6f6735e08f
parentb8ef9e288548ea5247c120df4c8012f0df7a7be4
Bug 12868: Wrong variable used for borrower number

When only the card number is passed to GetMemberDetail, the
value of $borrowernumber is undefined. Even after finding the
correct borrower and providing a nice hash ($borrower), the
GetMemberAccountRecords is called with the wrong borrower number,
even though it is in the hash ($borrower).

This was fixed by changing $borrowernumber to
$borrower->{borrowernumber}, so that the hash's value will always
be used, since it is correct regardless of whether borrowernumber
or cardnumber were used to find the borrower.

TEST PLAN
---------
1) Apply both patches
2) prove -v t/db_dependent/Member.t
   -- This time the previously failing test will pass.
3) run koha QA test tools.

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
C4/Members.pm