Attachment 155536 Details for Bug 15504 – Bug 15504: Add triggers to instances of track_login_daily

[patch] Bug 15504: Add triggers to instances of track_login_daily

Bug-15504-Add-triggers-to-instances-of-tracklogind.patch (text/plain), 3.68 KB, created by David Nind on 2023-09-12 05:56:03 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: David Nind

Created: 2023-09-12 05:56:03 UTC

Size: 3.68 KB

patch

obsolete

>From eaaa41d84a348311195bf9aea6d2eef483324795 Mon Sep 17 00:00:00 2001
>From: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
>Date: Wed, 14 Jun 2023 14:14:53 +0000
>Subject: [PATCH] Bug 15504: Add triggers to instances of track_login_daily
>
>This patch adds a trigger to every instance of track_login_daily
>
>Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
>Signed-off-by: David Nind <david@davidnind.com>
>---
> C4/Auth.pm            | 2 +-
> C4/Circulation.pm     | 9 +++++++--
> C4/ILSDI/Services.pm  | 2 +-
> C4/SIP/Sip/MsgType.pm | 2 +-
> 4 files changed, 10 insertions(+), 5 deletions(-)
>
>diff --git a/C4/Auth.pm b/C4/Auth.pm
>index ab39ef366d..44362b1c99 100644
>--- a/C4/Auth.pm
>+++ b/C4/Auth.pm
>@@ -1318,7 +1318,7 @@ sub checkauth {
>             ));
>         }
> 
>-        track_login_daily( $userid );
>+        track_login_daily( $userid, 'login' );
> 
>         # In case, that this request was a login attempt, we want to prevent that users can repost the opac login
>         # request. We therefore redirect the user to the requested page again without the login parameters.
>diff --git a/C4/Circulation.pm b/C4/Circulation.pm
>index 98c66828f5..66cf71f666 100644
>--- a/C4/Circulation.pm
>+++ b/C4/Circulation.pm
>@@ -1682,7 +1682,7 @@ sub AddIssue {
>                 )->store;
>             }
>             $issue->discard_changes;
>-            C4::Auth::track_login_daily( $borrower->{userid} );
>+            C4::Auth::track_login_daily( $borrower->{userid}, 'check_out' );
>             if ( $item_object->location && $item_object->location eq 'CART'
>                 && ( !$item_object->permanent_location || $item_object->permanent_location ne 'CART' ) ) {
>             ## Item was moved to cart via UpdateItemLocationOnCheckin, anything issued should be taken off the cart.
>@@ -2475,7 +2475,10 @@ sub AddReturn {
> 
>         logaction("CIRCULATION", "RETURN", $borrowernumber, $item->itemnumber)
>             if C4::Context->preference("ReturnLog");
>-        }
>+
>+        #Update borrowers.lastseen
>+        C4::Auth::track_login_daily( $patron->userid, 'check_in' );
>+    }
> 
>     # Check if this item belongs to a biblio record that is attached to an
>     # ILL request, if it is we need to update the ILL request's status
>@@ -3323,6 +3326,8 @@ sub AddRenewal {
>                 interface      => C4::Context->interface,
>             }
>         );
>+        #Update borrowers.lastseen
>+        C4::Auth::track_login_daily( $patron_unblessed->{userid}, 'renewal' );
> 
>         #Log the renewal
>         logaction("CIRCULATION", "RENEWAL", $borrowernumber, $itemnumber) if C4::Context->preference("RenewalLog");
>diff --git a/C4/ILSDI/Services.pm b/C4/ILSDI/Services.pm
>index 77243a882a..6cb6ff5d89 100644
>--- a/C4/ILSDI/Services.pm
>+++ b/C4/ILSDI/Services.pm
>@@ -397,7 +397,7 @@ sub AuthenticatePatron {
>     my ($status, $cardnumber, $userid) = C4::Auth::checkpw( $username, $password );
>     if ( $status == 1 ) {
>         # Track the login
>-        C4::Auth::track_login_daily( $userid );
>+        C4::Auth::track_login_daily( $userid, 'connection' );
>         # Get the borrower
>         my $patron = Koha::Patrons->find( { userid => $userid } );
>         return { id => $patron->borrowernumber };
>diff --git a/C4/SIP/Sip/MsgType.pm b/C4/SIP/Sip/MsgType.pm
>index e29d390da8..b166551e45 100644
>--- a/C4/SIP/Sip/MsgType.pm
>+++ b/C4/SIP/Sip/MsgType.pm
>@@ -995,7 +995,7 @@ sub handle_patron_info {
> 
>     $resp = (PATRON_INFO_RESP);
>     if ($patron) {
>-        $patron->update_lastseen();
>+        C4::Auth::track_login_daily( $patron->userid, 'connection' );
>         $resp .= patron_status_string( $patron, $server );
>         $resp .= ( defined($lang) and length($lang) == 3 ) ? $lang : $patron->language;
>         $resp .= timestamp();
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 15504: 152332 | 152333 | 152334 | 152335 | 152336 | 152500 | 152501 | 152502 | 152503 | 152504 | 152505 | 152506 | 155290 | 155291 | 155292 | 155293 | 155294 | 155295 | 155296 | 155534 | 155535 | 155536 | 155537 | 155538 | 155539 | 155540 | 156598 | 156599 | 156600 | 156601 | 156602 | 156603 | 156604 | 156605 | 156606 | 156607 | 156608 | 156742 | 156855 | 156856 | 156857 | 156858 | 156859 | 156860 | 156861 | 156862 | 156863 | 156864 | 156865 | 156866 | 156867 | 156868 | 156869 | 156870 | 156875 | 156879 | 156898 | 156899 | 156900 | 156901 | 156902 | 156903 | 156904 | 156905 | 156906 | 156907 | 156908 | 156909 | 156910 | 156911 | 156912 | 156913 | 156914 | 156915 | 156917 | 156918 | 156919 | 156920 | 156921 | 156922 | 156923 | 156924 | 156925 | 156926 | 156927 | 156928 | 156929 | 156930 | 156931 | 156932 | 156933 | 156934 | 156935 | 157602 | 157603 | 157604 | 157605 | 157606 | 157607 | 157608 | 157609 | 157610 | 157611 | 157612 | 157613 | 157614 | 157615 | 157616 | 157617 | 157618 | 157619 | 157620 | 157624