Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/2624846 event system cron finished #296

Closed
wants to merge 9 commits into from
Closed

Feature/2624846 event system cron finished #296

wants to merge 9 commits into from

Conversation

ndf
Copy link
Contributor

@ndf ndf commented Nov 28, 2015

Hi Fago,

This is the Event for System : Cron maintenance tasks are performed

First I named it SystemCronEventFinished, but it is actually called before processing the cron-job.
hook_cron() does not give us any parameters to pass to the GenericEvent constructor.
There is 1 kernel-test, that call the rules_cron(). This is based on user_login event.

https://www.drupal.org/node/2624846

$this->logger = $this->container->get('logger.channel.rules');

// Invoke the hook manually which should trigger the rule.
rules_cron();
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

talked with klausi: if available could load Cron service and then run it.
aka invoke a real cron-job

$this->container->get('cron')->run();

@klausi
Copy link
Collaborator

klausi commented Nov 29, 2015

merged, thanks!

@klausi klausi closed this Nov 29, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants