Bug 27014: Enable C4::SIP::ILS::Patron::new to accept a hash
authorKyle M Hall <kyle@bywatersolutions.com>
Fri, 13 Nov 2020 13:24:43 +0000 (08:24 -0500)
committerAndrew Fuerste-Henry <andrew@bywatersolutions.com>
Fri, 5 Mar 2021 16:16:06 +0000 (16:16 +0000)
commitfc7b9a96ec9f5e46d4284f9065ace6301cca9664
tree0b901466ac535678342a38f51756f4ec56ec2b3f
parent80f665ece4dec9a105883788c4e0e30889abeadf
Bug 27014: Enable C4::SIP::ILS::Patron::new to accept a hash

Caused by bug 23403 - when performing a checkin we lookup the SIP patron
using the borrowernumber, however, SIP only knows how to find a patron
via cardnumber or userid

The change on 23403 was to avoid using an 'id' that didn't always exist
(as some users don't have a userid or cardnumber

When checking in, however, we are not passed a user cardnumber or
borrowernumber, so we don't have those on hand to get the patron.

Test Plan:
1) Check in an item via SIP, note patron is not found
2) Apply this patch
3) Restart all the things!
4) Check in an item via SIP, patron should be found!

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
(cherry picked from commit 05e16389efa6e66476484ee12c3a5c5fdd48757b)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
(cherry picked from commit ea74c996834fd5124fe73c893785950b28d932d1)

Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com>
C4/SIP/ILS.pm
C4/SIP/ILS/Patron.pm