Problem/Motivation

Cron jobs for the different hook_cron implementations are created automatically, but without scheduler/launcher/logger config. This config is set automatically when creating through the UI form.

This became apparent when converting our install profile to recipes. Unless you provide all cron configs in the recipe as well, they are not created with the correct scheduler/launcher/logger config.

Proposed resolution

Update CronJobDiscovery::ensureCronJobExists().

Issue fork ultimate_cron-3565035

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

dieterholvoet created an issue. See original summary.

dieterholvoet opened merge request !80

dieterholvoet’s picture

dieterholvoet commented
Status: Active » Needs review

This uncovered a small bug in a test, I fixed it.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Needs work

Needs a rebase. Makes sense. I'm unsure what to do about the default settings in general, they're currently not exposed and this hardcodes specific config keys. I guess that's fine and keeps the option to adjust the defaults if you know what you're doing.

dieterholvoet’s picture

dieterholvoet commented
Status: Needs work » Needs review

Rebased

  • dieterholvoet committed d3247ca8 on 8.x-2.x 
    fix: #3565035 Set default scheduler/launcher/logger config when creating...
dieterholvoet’s picture

dieterholvoet commented
Status: Needs review » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.