Allowing arguments for then() function

[Adrienb91]

Hello,

I'm using your good cron php manager, connected to my database. I want to log my cron executions to do a dashboard of all cronjobs and be able to manage the tasks in an dedicated interface with last execution result.

To store the result of a cron job I have to use your then() function but (perhaps as the issue 51) I've a problem with passing arguments to the then() function.

I've made some changes to your Job.php file to do that :

In Job.php

Juste after the line private $after; add : /** * Arguments to function executed after the job is executed. * * @var array */ private $afterArgs;

Add argument $args to then() declaration function to be : public function then(callable $fn, array $args, $runInBackground = false)

In then function, after $this->after = $fn; add : $this->afterArgs = $args;

In finalize function change : call_user_func($this->after, $this->output, $this->returnCode); by : call_user_func_array($this->after, array($this->afterArgs, $this->output, $this->returnCode));

It's done ! We can now call then function like then($afterExecutionFunction,array($arg1,$arg2)); And use this arguments from array in the function like : $afterExecutionFunction=function($args, $output) { print_r($args); $arg1=$args[0]; $arg2=$args[1];

Do you think it's a good way to solve the problem ?

Thanks for this good php cron manager and hope this can interest somebody.

[peppeocchi]

@Adrienb91 it's a great suggestion, I am a bit worried about this being a breaking change. If you want to work on it and send me a PR I'll be more than happy to review it, if you do that just make sure not to introduce breaking changes