From 03d62dbb8c44159e4a2fef1b4768d6cd29da7572 Mon Sep 17 00:00:00 2001 From: Colin Campbell Date: Thu, 18 May 2017 15:59:49 +0100 Subject: [PATCH] Bug 18625: Update lastseen in patron info request Many services use SIP patron req info to validate a patron against the koha database. If recording lastseen sites will also want to record that the user has accessed these library facilities although they have not logged into koha Signed-off-by: Marcel de Rooy Verified that Patron Info request (63) updates lastseen when the pref has been enabled. Signed-off-by: Kyle M Hall --- C4/SIP/ILS/Patron.pm | 19 +++++++++++++++++++ C4/SIP/Sip/MsgType.pm | 1 + 2 files changed, 20 insertions(+) diff --git a/C4/SIP/ILS/Patron.pm b/C4/SIP/ILS/Patron.pm index 81b626888d..81036c9cce 100644 --- a/C4/SIP/ILS/Patron.pm +++ b/C4/SIP/ILS/Patron.pm @@ -441,6 +441,25 @@ sub charge_denied { return "Please contact library staff"; } +=head2 update_lastseen + + $patron->update_lastseen(); + + Patron method to update lastseen field in borrower + to record that patron has been seen via sip connection + +=cut + +sub update_lastseen { + my $self = shift; + my $p; + if (C4::Context->preference('TrackLastPatronActivity') + && $p = Koha::Patrons->find({ borrowernumber => $self->{borrowernumber} })) { + $p->track_login; + } + return; +} + sub _get_address { my $patron = shift; diff --git a/C4/SIP/Sip/MsgType.pm b/C4/SIP/Sip/MsgType.pm index ac2ac16588..9ec126ed12 100644 --- a/C4/SIP/Sip/MsgType.pm +++ b/C4/SIP/Sip/MsgType.pm @@ -936,6 +936,7 @@ sub handle_patron_info { $resp = (PATRON_INFO_RESP); if ($patron) { + $patron->update_lastseen(); $resp .= patron_status_string($patron); $resp .= ( defined($lang) and length($lang) == 3 ) ? $lang : $patron->language; $resp .= timestamp(); -- 2.15.1 (Apple Git-101)