Make it easier to determine if we are 'inside' a queue processor

[remcotolsma]

Make it easier to determine if we are 'inside' a queue processor

[barryhughes]

It would be nice if this library introduced it's own DOING_ACTION_SCHEDULER or something similar.

Re-opening so we don't lose sight of this idea (a little unsure if defining a constant is the best approach for us, but the general idea makes sense).

[barryhughes]

Hi!

However, it seems that wp_doing_cron() does not always return true when running "Action Scheduler" tasks.

That's right. By default, there are up to three different contexts through which Action Scheduler may be triggered:

• WP Cron
• Async HTTP requests
• WP CLI

In most cases, it's a mix of the first two: it doesn't always run via WP Cron (and so DOING_CRON will not always be set).

However, we wonder whether this is wise and/or correct, so we are curious about the opinion of the developers of this library.

If you are finding you need something like this, then possibly? Can I just highlight though that we already have some related functionality within the library:

https://github.com/woocommerce/action-scheduler/blob/3.7.1/classes/ActionScheduler_Compatibility.php#L87-L104

This is generally called immediately prior to processing waiting actions. However, there isn't really any way to ensure this takes effect: restrictions in a given hosting environment take precedence, in many cases.

[remcotolsma]

Thanks @barryhughes, good to know about these 3 contexts. I think the PHP timeouts you were referring to are different from the WordPress HTTP API timeouts. WordPress uses a standard timeout of 5 seconds for HTTP requests: https://developer.wordpress.org/reference/classes/wp_http/request/#parameters. For HTTP requests that are executed via background tasks, this may be slightly higher. However, there is currently no easy way to check whether a script is running in one of the 3 contexts mentioned? For the WordPress cron wp_doing_cron() for WP-CLI defined( 'WP_CLI' ) && WP_CLI and for the Action Scheduler "Async HTTP requests"?

[barryhughes]

I'm sorry, I misunderstood.

However, there is currently no easy way to check whether a script is running in one of the 3 contexts mentioned?

I'm not sure how useful this would be in your case, but the action_scheduler_begin_execute hook is triggered immediately before each scheduled action is fetched and executed, and supplies two parameters to its callbacks:

• $action_id
• $context

The second of those provides the information you are looking for (and even without it, you could still use the action itself as a signal that you are 'inside' the scheduled action queue).

Might that work? If not, we could explore alternatives. For instance, it would seem reasonable to also share the context via the action_scheduler_before_process_queue action hook, and there are probably various other possibilities.

[remcotolsma]

Thanks, for now we handled it this way:

• https://github.com/pronamic/wp-pay-core/blob/b1c28b0d2f7e4772ce4e600ff6215e613cc87996/src/ActionSchedulerController.php#L1-L41
• https://github.com/pronamic/wp-mollie/blob/13bfb8f77bb4333c975e45093b19ffb2771d0c82/src/Client.php#L102-L147

It would be nice if this library introduced it's own DOING_ACTION_SCHEDULER or something similar.