You can pass a 'crontab' property to the set() method, but that's not documented in its docblock.

CommentFileSizeAuthor
#1 job_scheduler-crontab-docs-2113179-1.patch793 bytestwistor
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

twistor’s picture

Status: Active » Needs review
FileSize
793 bytes

Hrm, yeah. I wish we had tests for the crontab parser. I'm not terribly confident in it.

Here's a quick stab at this.

joachim’s picture

That doesn't sound good! Should we think about writing tests for it? Or using an 3rd party library? A quick search found https://github.com/mtdowling/cron-expression

joachim’s picture

Issue summary: View changes
Status: Needs review » Fixed

git commit -m "Issue #2113179 by twistor: Fixed missing documentation of crontab functionality for JobScheduler::set()." --author="twistor "

joachim’s picture

Status: Fixed » Closed (fixed)

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