Changeset 44693 for trunk/src/wp-includes/cron.php

Timestamp:

01/23/2019 11:07:39 PM (7 years ago)

Author:

peterwilsoncc

Message:

Cron: Move logic for finding the next event's timestamp.

Moves the logic for determining when an event will next run from wp_next_scheduled() to wp_get_scheduled_event().

This improves the performance of wp_get_scheduled_event() by avoiding duplicate function calls and object cache hits.

Props peterwilsoncc.
Fixes #45976.

File:

• trunk/src/wp-includes/cron.php (modified) (6 diffs)

trunk/src/wp-includes/cron.php

@@ -482 +482 @@
  * Retrieve a scheduled event.
  *
- * Retrieve the full event object for a given event.
+ * Retrieve the full event object for a given event, if no timestamp is specified the next
+ * scheduled event is returned.
  *
  * @since 5.1.0
@@ -494 +495 @@
  */
 function wp_get_scheduled_event( $hook, $args = array(), $timestamp = null ) {
-    if ( ! $timestamp ) {
-        // Get the next scheduled event.
-        $timestamp = wp_next_scheduled( $hook, $args );
-    }
-
     /**
      * Filter to preflight or hijack retrieving a scheduled event.
@@ -515 +511 @@
      *                             Although not passed to a callback, these arguments are used to uniquely identify the
      *                             event.
-     * @param int       $timestamp Unix timestamp (UTC) of the event.
+     * @param int|null  $timestamp Unix timestamp (UTC) of the event. Null to retrieve next scheduled event.
      */
     $pre = apply_filters( 'pre_get_scheduled_event', null, $hook, $args, $timestamp );
@@ -522 +518 @@
     }
 
+    if ( null !== $timestamp && ! is_numeric( $timestamp ) ) {
+        return false;
+    }
+
     $crons = _get_cron_array();
-    $key   = md5( serialize( $args ) );
-
-    if ( ! $timestamp || ! isset( $crons[ $timestamp ] ) ) {
-        // No such event.
-        return false;
-    }
-
-    if ( ! isset( $crons[ $timestamp ][ $hook ] ) || ! isset( $crons[ $timestamp ][ $hook ][ $key ] ) ) {
+    if ( empty( $crons ) ) {
+        return false;
+    }
+
+    $key = md5( serialize( $args ) );
+
+    if ( ! $timestamp ) {
+        // Get next event.
+        $next = false;
+        foreach ( $crons as $timestamp => $cron ) {
+            if ( isset( $cron[ $hook ][ $key ] ) ) {
+                $next = $timestamp;
+                break;
+            }
+        }
+        if ( ! $next ) {
+            return false;
+        }
+
+        $timestamp = $next;
+    } elseif ( ! isset( $crons[ $timestamp ][ $hook ][ $key ] ) ) {
         return false;
     }
@@ -552 +565 @@
  *
  * @since 2.1.0
- * @since 5.1.0 {@see 'pre_next_scheduled'} and {@see 'next_scheduled'} filters added.
  *
  * @param string $hook Action hook of the event.
@@ -561 +573 @@
  */
 function wp_next_scheduled( $hook, $args = array() ) {
-    /**
-     * Filter to preflight or hijack retrieving the next scheduled event timestamp.
-     *
-     * Returning a non-null value will short-circuit the normal retrieval
-     * process, causing the function to return the filtered value instead.
-     *
-     * Pass the timestamp of the next event if it exists, false if not.
-     *
-     * @since 5.1.0
-     *
-     * @param null|bool $pre       Value to return instead. Default null to continue unscheduling the event.
-     * @param string    $hook      Action hook of the event.
-     * @param array     $args      Arguments to pass to the hook's callback function.
-     */
-    $pre = apply_filters( 'pre_next_scheduled', null, $hook, $args );
-    if ( null !== $pre ) {
-        return $pre;
-    }
-
-    $crons = _get_cron_array();
-    $key   = md5( serialize( $args ) );
-    $next  = false;
-
-    if ( ! empty( $crons ) ) {
-        foreach ( $crons as $timestamp => $cron ) {
-            if ( isset( $cron[ $hook ][ $key ] ) ) {
-                $next = $timestamp;
-                break;
-            }
-        }
-    }
-
-    /**
-     * Filter the next scheduled event timestamp.
-     *
-     * @since 5.1.0
-     *
-     * @param int|bool $next The UNIX timestamp when the scheduled event will next occur, or false if not found.
-     * @param string   $hook Action hook to execute when cron is run.
-     * @param array    $args Arguments to be passed to the callback function. Used for deduplicating events.
-     */
-    return apply_filters( 'next_scheduled', $next, $hook, $args );
+    $next_event = wp_get_scheduled_event( $hook, $args );
+    if ( ! $next_event ) {
+        return false;
+    }
+
+    return $next_event->timestamp;
 }