How to pass arguments

Hi everyone,
Please review the patch.
I suggest to use callback arguments as a string with line ending separation in the Cron Job configuration form since we don't actually know the final amount of them. It depends on the particular implementation of each callback. Then this string will be parsed, split to arguments and passed to callback.

The last submitted patch, 5: ultimate_cron-callback-arguments-2909191-5.patch, failed testing. View results →
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

I just set it up on a project and it works well. I believe that this is e must-have feature.

This has a few conflicts. I have some concerns about the argument structure, XSS filtering doesn't seem relevant here as it's not displayed.

The schema talks about comma separated values, but according to tests and code, it's really line breaks.

What about storing arguments as a sequence, then only the form needs to process them into an array and runtime can just pass it through directly as an array?

Hi @berdir,

Thanks for findings, they make sense.
Please take a look and review updated patch or MR

> What about storing arguments as a sequence, then only the form needs to process them into an array and runtime can just pass it through directly as an array?

This hasn't been addressed yet.

To be honest, I'm not sure this should even be exposed in the UI. We don't allow to set/change the callback, also for security reasons, so this should just display the arguments as well.