I have a use case where it would be useful to know the last time a cron service ran. For example, getting a list of nodes that have been created since the last execution. It would be great if this module provided that functionality.

Issue fork cron_service-3393041

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

davisben created an issue. See original summary.

davisben opened merge request !2

davisben’s picture

davisben
Primary language English
Location NH, USA
commented
Status: Active » Needs review
jureth’s picture

jureth commented
Status: Needs review » Needs work

1. Why there's a check for ScheduledCronServiceInterface in getLastExecutionTime() ? As I understand the last time value is stored for every handler.
2. Technically, 0 is valid unix timestamp, so I'm not sure why getLastExecutionTime() shouldn't return null in case the value is missing.

davisben’s picture

davisben
Primary language English
Location NH, USA
commented
Version: 8.x-1.x-dev » 2.0.1
Status: Needs work » Needs review

I addressed the concerns in #4, and rebased against 2.0.x.

vasyl.kletsko made their first commit to this issue’s fork.

vasyl.kletsko’s picture

vasyl.kletsko commented
vasyl.kletsko’s picture

vasyl.kletsko commented

Looks good now.

vasyl.kletsko’s picture

vasyl.kletsko commented
Status: Needs review » Fixed
vasyl.kletsko’s picture

vasyl.kletsko commented

Status: Fixed » Closed (fixed)

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