Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
job_scheduler_rebuild_scheduler() is only called in one place. The $info param is passed in.
So there's no need at all for this:
function job_scheduler_rebuild_scheduler($name, $info = NULL) {
$info = $info ? $info : job_scheduler_info($name);
It's only one line of code, but the calls to job_scheduler_info() everywhere make it really hard to understand the flow of the code. This will simplify things a lot.
Comment | File | Size | Author |
---|---|---|---|
#1 | 2180757.job_scheduler.job_scheduler_rebuild_scheduler-optional-param.patch | 533 bytes | joachim |
Comments
Comment #1
joachim CreditAttribution: joachim commentedComment #2
renatogHi people.
I applied the patch and works good for me.
Good Catch. Thank you very much for contribution.
Regards
Comment #4
renatogFixed.
Commited in dev branch.
Thanks.
Good Work and Good Weekend.
Regards.