]> git.koha-community.org Git - koha.git/commit
Bug 16492 - Checkouts ( and possibly checkins and other actions ) will use the patron...
authorKyle M Hall <kyle@bywatersolutions.com>
Tue, 10 May 2016 16:36:03 +0000 (16:36 +0000)
committerJulian Maurice <julian.maurice@biblibre.com>
Thu, 16 Jun 2016 09:12:34 +0000 (11:12 +0200)
commit853d297415c20a6a4fa42362185572006ee64192
tree8a119496e5563b3a34c5a46d89b73efd5ef213b9
parent32bf30da9d5cf182171861b5b2dadcab60d1c4b3
Bug 16492 - Checkouts ( and possibly checkins and other actions ) will use the patron home branch as the logged in library

Bug 14507 introduced the use of checkpw in C4::SIP::ILS::Patron so that
non-Koha internal authentication processes would be able to function via
SIP ( LDAP et al ).

The problem is that checkpw changes the userenv to that of the patron!
This is not usually an issue in Koha because most of the time that
patron running through checkpw is the one to be logged in.

Aside from SIP2 the only other area where this may be an issue is in SCO
when using SelfCheckoutByLogin.

Test Plan:
1) On master, check out an item to a patron via SIP2
2) Note the checkout lists the item as having been checked out
   from the patron's home library not matter which library is was
   supposed to be checked out from.
3) Apply this patch
4) Re-checkout the item
5) The item should now be checked out as if it was checked out from
   the library as defined in the SIP configuration file.

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
(cherry picked from commit 90307bcbfaf4c2c8907be94eabca249ac173442e)
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
(cherry picked from commit 57086a241e141e157d42b8af3f56ae9283d46ae9)
Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
C4/Auth.pm
C4/SIP/ILS/Patron.pm
t/db_dependent/Auth.t