Attachment 152294 Details for Bug 33997 – Bug 33997: Stop track_login from updating borrowers.updated_on

[patch] Bug 33997: Stop track_login from updating borrowers.updated_on

Bug-33997-Stop-tracklogin-from-updating-borrowersu.patch (text/plain), 1.06 KB, created by Matt Blenkinsop on 2023-06-13 13:43:36 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Matt Blenkinsop

Created: 2023-06-13 13:43:36 UTC

Size: 1.06 KB

patch

obsolete

>From 805dcbfc95b6850a14b5ddc6023cabdb8697ecd4 Mon Sep 17 00:00:00 2001
>From: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
>Date: Tue, 13 Jun 2023 13:36:13 +0000
>Subject: [PATCH] Bug 33997: Stop track_login from updating
> borrowers.updated_on
>
>This patch blocks the update to borrowers.updated_on when track_login is called
>
>Test plan:
>1) Enable syspref 'TrackLastPatronActivity'
>2) Log into the OPAC
>3) Check the database for the patron/account used and observe that the timestamps for updated_on and lastseen match
>4) reset_all
>5) Apply patch
>6) Repeat steps one to three, the updated_on timestamp should not have changed
>---
> Koha/Patron.pm | 1 +
> 1 file changed, 1 insertion(+)
>
>diff --git a/Koha/Patron.pm b/Koha/Patron.pm
>index 2a8eecf9f6..e20eb6f966 100644
>--- a/Koha/Patron.pm
>+++ b/Koha/Patron.pm
>@@ -1014,6 +1014,7 @@ sub track_login {
>     return if
>         !$params->{force} &&
>         !C4::Context->preference('TrackLastPatronActivity');
>+    $self->make_column_dirty('updated_on');
>     $self->lastseen( dt_from_string() )->store;
> }
> 
>-- 
>2.37.1 (Apple Git-137.1)

Actions: View | Diff | Splinter Review

Attachments on bug 33997: 152294 | 152321 | 152342