Bugzilla – Attachment 80739 Details for
Bug 16280
purge_suggestions.pl: Cron job log should tell number of days used
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 16280: purge_suggestions.pl: Cron job log should tell number of days used
Bug-16280-purgesuggestionspl-Cron-job-log-should-t.patch (text/plain), 1.48 KB, created by
Jonathan Druart
on 2018-10-17 16:36:38 UTC
(
hide
)
Description:
Bug 16280: purge_suggestions.pl: Cron job log should tell number of days used
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2018-10-17 16:36:38 UTC
Size:
1.48 KB
patch
obsolete
>From ba602ff4ffa6b6f4e2cb71aa849bb3c064ed2ae8 Mon Sep 17 00:00:00 2001 >From: =?UTF-8?q?Marc=20V=C3=A9ron?= <veron@veron.ch> >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 <mtompset@hotmail.com> > >Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> >--- > 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
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 16280
:
50298
|
80669
| 80739