https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38857 Martin Renvoize (ashimema) <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #176369|0 |1 is obsolete| | --- Comment #3 from Martin Renvoize (ashimema) <martin.renvoize@ptfs-europe.com> --- Created attachment 176370 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=176370&action=edit Bug 38857: Update cronjobs to log run before processing parameters A few notes: - I grabbed this list by looking for things in misc/ that had cronlogaction - I didn't touch scripts that aren't in the cronjobs directory, except fix_invalid_dates where I follow import_patrons and only log if confirmed since they aren't crons, I figure this seems reasonable - purge_suggestions I moved the logggin up and changed the addition of effective days to a verbose message - this seems more consistent To test: 1 - Enable CronjobLog 2 - On command line: perl misc/cronjobs/cleanup_database.pl --old_reservers 550 3 - Error on command line 4 - In staff client, go to Tools->Log viewer, check 'Cronjobs' and submit 5 - No entries 6 - Apply patch 7 - Repeat and see run is logged with wrong parameter 8 - Fix parameter and confirm 9 - Spot check other files until you are satisfied 10 - Sign it off! Signed-off-by: William Lavoie <william.lavoie@inLibro.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> -- You are receiving this mail because: You are watching all bug changes.