Make WordPress Core

Opened 6 years ago

Closed 6 years ago

#42758 closed defect (bug) (invalid)

Wp Cron, major issue by removing the apply_filter when wp_cron() is trigged.

Reported by: ajami's profile ajami Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: Cron API Keywords: needs-patch
Focuses: Cc:

Description

I have seen on Wordpress 4.9, they remove apply_filter($hook,$args) form wp_cron() function which will affect a lot of crons which depends on this action hook. When we schedule a event we give hook to trig while the cron is executing, now with the latest branch ( wp-4.9-branch ) this i will not happen.I hope Wordpress to make it a urgent priority and provide the patch.

Attachments (2)

cron.php (17.2 KB) - added by ajami 6 years ago.
Wp Cron patch
ajami.42758.diff (1.2 KB) - added by dd32 6 years ago.
cron.php in diff form

Download all attachments as: .zip

Change History (4)

@ajami
6 years ago

Wp Cron patch

#1 @ajami
6 years ago

  • Keywords needs-patch added
  • Severity changed from normal to blocker

#2 @dd32
6 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to invalid
  • Severity changed from blocker to normal
  • Status changed from new to closed
  • Version 4.9.1 deleted

Hi @ajami

I'm not sure I follow your ticket.
No filters have been removed from WordPress in the cron system recently, infact, here's all the changes to the file since WordPress 4.0.

Based on your addition of a hook to be run in cron.php, I'm going to assume you're having issues with cron running - Cron runs through the use of a loopback HTTP request which kicks off a new PHP process which the crons run in.
The code you've altered calls spawn_cron() which then loads https://core.trac.wordpress.org/browser/trunk/src/wp-cron.php#L98 and the action gets triggered there.

If you're having cron spawning problems, Trac is not a support avenue and I'd recommend the Support Forums.
I'm closing this report as cron hasn't had any major changes in the last 3 years, there's been no other reports of cron issues, and I've personally been using it since the release.

@dd32
6 years ago

cron.php in diff form

Note: See TracTickets for help on using tickets.