From 01a5d302b96a3fa2913e14d00d87a8562f11cc98 Mon Sep 17 00:00:00 2001 From: Martin Renvoize Date: Wed, 11 Oct 2023 18:10:32 +0100 Subject: [PATCH] Bug 15504: (follow-up) Clarify --last_seen option in delete_patrons This patch adds clarifications to the last_seen option in delete_patrons to clarify that TrackLastPatronActivity and TrackLastPatronActivityTriggers system preferences act as a couplet. Signed-off-by: Marcel de Rooy Signed-off-by: Tomas Cohen Arazi --- misc/cronjobs/delete_patrons.pl | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/misc/cronjobs/delete_patrons.pl b/misc/cronjobs/delete_patrons.pl index c568c6d027..6293bbf442 100755 --- a/misc/cronjobs/delete_patrons.pl +++ b/misc/cronjobs/delete_patrons.pl @@ -39,7 +39,7 @@ $expired_before = dt_from_string( $expired_before, 'iso' ) if $expired_before; if ( $last_seen and not C4::Context->preference('TrackLastPatronActivity') ) { - pod2usage(q{The --last_seen option cannot be used with TrackLastPatronActivity turned off}); + pod2usage(q{The --last_seen will be ineffective if TrackLastPatronActivity turned off or there are not TrackLastPatronActivityTriggers defined}); } unless ( $not_borrowed_since or $expired_before or $last_seen or @category_code or $branchcode or $file ) { @@ -183,9 +183,10 @@ Delete patrons with an account expired before this date. =item B<--last_seen> -Delete patrons who have not been connected since this date. +Delete patrons who have not been active since this date. -The system preference TrackLastPatronActivity must be enabled to use this option. +The system preference TrackLastPatronActivity must be enabled and at least one TrackLastPatronActivityTriggers +must be set to use this option. =item B<--category_code> -- 2.39.5