|
Description
Nick Clemens (kidclamp)
2018-06-07 17:46:51 UTC
Created attachment 98803 [details] [review] Bug 20897: Add generic cronjob to run plugins with a cronjob method Created attachment 98804 [details] Plugin to test Bug 20897 This is a plugin to test bug 20897. All it does it implement the cronjob-method, and in that method, logs a warning: "PLUGIN Bug20897: CRONJOB" Created attachment 99017 [details] [review] Bug 20897: Add generic cronjob to run plugins with a cronjob method This adds a run_cronjob_plugins.pl script, which will run the cronjob-methods of the installed plugins. Test plan: 1) Enable plugins, upload the Bug20897 test plugin 2) Set up the run_cronjob_plugins.pl in cronjob 3) Check the cron logs to see the message "PLUGIN Bug20897: CRONJOB" shows up there every time the cronjob is run 4) Test that the plugins-home.pl -page can filter the cronjob plugins Signed-off-by: Pasi Kallinen <pasi.kallinen@koha-suomi.fi> Signed-off-by: Liz Rea <wizzyrea@gmail.com> Created attachment 99018 [details] [review] Bug 20897 - add commented cron example to cron.hourly To test: Have a look at kohaclone/debian/koha-common.cron.hourly, see that there is a job in there for running the plugin cron jobs for every Koha instance. Signed-off-by: Liz Rea <wizzyrea@gmail.com> Should not it be in misc/cronjobs/crontab.example as well? FAIL misc/cronjobs/run_cronjob_plugins.pl
FAIL forbidden patterns
forbidden pattern: tab char (line 48)
forbidden pattern: tab char (line 49)
forbidden pattern: tab char (line 50)
|