WPIntell

Source evidence

Cude suggestion

WP-Cron Control · support · 2017-05-19T13:56:00+00:00

complaintsentiment
mediumseverity
0.86relevance
2replies
Evidence linked to opportunitycommercial context

Proof Health

Open evidence

Commercial opportunities need traceable source links before they are treated as build-worthy.

7 / 35 rows with source links

20.0% of this page's analysis has direct source links.

0 build-decision rows missing links

0 rows here require auditable proof before promotion.

28 rows with no attached evidence

0 rows have source counts but still need direct links.

Conversation

support
Nextendweb unresolved
Hi, you shouldn’t mark every non-cron requests with DOING_CRON constant as it causes issue with an other plugin which check if this constant is set or not. I think the following code should be removed: // for all other cases disable wp-cron.php and spawn_cron() by telling the system it’s already running if ( !defined( ‘DOING_CRON’ ) ) define( ‘DOING_CRON’, true ); After a quick search – I found something which might be helpful to the creators of WP-Cron Control for solving their conflict issues with other plugins checking the DOING_CRON constant. Try replacing your disable wp-cron.php code with this: // disable WP_CRON if ( ! defined('DISABLE_WP_CRON') ) { define( ‘DISABLE_WP_CRON’, true ); } Reference: https://github.com/WordPress/WordPress/blob/master/wp-includes/cron.php#L350 This reply was modified 9 years ago by Kevin Vess . It’s already in their code as I remember.

Comments

2 shown
Kevin Vess 2017-05-19T15:18:00+00:00

After a quick search – I found something which might be helpful to the creators of WP-Cron Control for solving their conflict issues with other plugins checking the DOING_CRON constant. Try replacing your disable wp-cron.php code with this: // disable WP_CRON if ( ! defined('DISABLE_WP_CRON') ) { define( ‘DISABLE_WP_CRON’, true ); } Reference: https://github.com/WordPress/WordPress/blob/master/wp-includes/cron.php#L350 This reply was modified 9 years ago by Kevin Vess .

Nextendweb 2017-05-19T15:33:00+00:00

It’s already in their code as I remember.