From 392b84e841e5f989ef633737afda066b5db0f238 Mon Sep 17 00:00:00 2001 From: Colin Campbell Date: Thu, 18 May 2017 15:59:49 +0100 Subject: [PATCH 1/3] 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. --- 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 17034c0d0f..0d96dd23fe 100644 --- a/C4/SIP/ILS/Patron.pm +++ b/C4/SIP/ILS/Patron.pm @@ -433,6 +433,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 db451224a8..5c9fadf2e0 100644 --- a/C4/SIP/Sip/MsgType.pm +++ b/C4/SIP/Sip/MsgType.pm @@ -932,6 +932,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.14.3