From ba602ff4ffa6b6f4e2cb71aa849bb3c064ed2ae8 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Marc=20V=C3=A9ron?= Date: Sun, 17 Apr 2016 18:52:06 +0200 Subject: [PATCH] Bug 16280: purge_suggestions.pl: Cron job log should tell number of days used To test: - Apply patch - Set syspref CronjobLog to "Log" - Set syspref PurgeSuggestionsOlderThan e.g. to 20 - From console, run misc/cronjobs/purge_suggestions.pl - Check entry in cgi-bin/koha/admin/preferences.pl?op=search&searchfield=log - Verify that log entry contains number of days (20, same as in syspref) - From console, run misc/cronjobs/purge_suggestions.pl -days 100 - Verify that log entry contains number of days (100, same as in param -days) Signed-off-by: Mark Tompsett Signed-off-by: Jonathan Druart --- misc/cronjobs/purge_suggestions.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/misc/cronjobs/purge_suggestions.pl b/misc/cronjobs/purge_suggestions.pl index 766f501db5..58a9a1dc12 100755 --- a/misc/cronjobs/purge_suggestions.pl +++ b/misc/cronjobs/purge_suggestions.pl @@ -63,7 +63,7 @@ if( !$confirm || $help || !defined($days) ) { print "No confirm parameter passed!\n\n" if !$confirm && !$help; print $usage; } elsif( $days and $days > 0 ) { - cronlogaction(); + cronlogaction( " ( days: $days )"); DelSuggestionsOlderThan($days); } else { warn "This script requires a positive number of days. Aborted.\n"; -- 2.11.0