From f234d1586bff62d2c4e58807cd62ec3187d94f69 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Sun, 24 Apr 2016 16:04:16 +0100 Subject: [PATCH] Bug 16276: Update borrowers.lastseen when a patron is active In order to add the ability to delete patrons who have been inactive for more than a given time, we need to track down the last time they were active. To do that, we need a new DB column in the borrowers table (lastseen). Note that the borrowers.lastseen column will not be initialised for existing installations (set to NULL) so inactive existing patrons will never be deleted. A workaround would be to init them at the date when the new column will be added. Test plan: 0/ Set the new pref TrackLastPatronActivity on 1/ Log in as a patron (staff or OPAC) 2/ Check that the borrowers.lastseen field has been updated with the current time Sponsored-by: BULAC - http://www.bulac.fr/ Signed-off-by: Nicolas Legrand https://bugs.koha-community.org/show_bug.cgi?id=12276 Signed-off-by: Marcel de Rooy Signed-off-by: Kyle M Hall --- C4/Auth.pm | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/C4/Auth.pm b/C4/Auth.pm index fcf7c48673..7171c2b499 100644 --- a/C4/Auth.pm +++ b/C4/Auth.pm @@ -1181,6 +1181,15 @@ sub checkauth { -HttpOnly => 1 ); } + + if ( $userid ) { + $dbh->do(q| + UPDATE borrowers + SET lastseen = NOW() + WHERE userid = ? + |, undef, $userid); + } + return ( $userid, $cookie, $sessionID, $flags ); } -- 2.39.5