Merge branch 'main' into copilot/add-cron-event-actions-display

Author: swissspidy

.github/workflows/code-quality.yml

@@ -6,6 +6,8 @@ on:
     branches:
       - main
       - master
+  schedule:
+     - cron: '17 2 * * *' # Run every day on a seemly random time.
 
 jobs:
   code-quality:

features/cron-event.feature

@@ -229,3 +229,18 @@ Feature: Manage WP Cron events
       """
       None
       """
+
+  Scenario: Confirm that cron event run in debug mode shows the start of events
+    When I try `wp cron event run wp_version_check --debug=cron`
+    Then STDOUT should contain:
+      """
+      Executed the cron event 'wp_version_check'
+      """
+    And STDOUT should contain:
+      """
+      Executed a total of 1 cron event
+      """
+    And STDERR should contain:
+      """
+      Debug: Beginning execution of cron event 'wp_version_check'
+      """

src/Cron_Event_Command.php

@@ -195,19 +195,19 @@ public function schedule( $args, $assoc_args ) {
 		}
 
 		if ( ! empty( $recurrence ) ) {
-
 			$schedules = wp_get_schedules();
 
 			if ( ! isset( $schedules[ $recurrence ] ) ) {
 				WP_CLI::error( sprintf( "'%s' is not a valid schedule name for recurrence.", $recurrence ) );
 			}
 
+			// WordPress expects a list bug we knowingly pass an associative array.
+			// @phpstan-ignore argument.type
 			$event = wp_schedule_event( $timestamp, $recurrence, $hook, $assoc_args );
-
 		} else {
-
+			// Ditto.
+			// @phpstan-ignore argument.type
 			$event = wp_schedule_single_event( $timestamp, $hook, $assoc_args );
-
 		}
 
 		if ( false !== $event ) {
@@ -252,6 +252,7 @@ public function run( $args, $assoc_args ) {
 
 		$executed = 0;
 		foreach ( $events as $event ) {
+			WP_CLI::debug( sprintf( "Beginning execution of cron event '%s'.", $event->hook ), 'cron' );
 			$start  = microtime( true );
 			$result = self::run_event( $event );
 			$total  = round( microtime( true ) - $start, 3 );