Context Navigation

← Previous Changeset
Next Changeset →

Changeset 62408

Timestamp:

05/22/2026 06:59:15 AM (2 months ago)

Author:

westonruter

Message:

Plugins: Improve hook performance by using spl_object_id() instead of spl_object_hash() to construct unique IDs.

Also use spl_object_id() similarly when registering and unregistering classic widgets.
Improve typing and phpdoc in _wp_filter_build_unique_id(). Return null for malformed callbacks.
Add tests for _wp_filter_build_unique_id().
Improve type safety of WP_Hook::add_filter() in case an invalid callback is provided for parity with ::has_filter() and ::remove_filter().

Developed in https://github.com/WordPress/wordpress-develop/pull/11865
Follow-up to r46220, r46801, r60179.

Props bor0, westonruter, SergeyBiryukov, schlessera, arshidkv12, knutsp, spacedmonkey, swissspidy.
See #64898.
Fixes #58291.

Location:

trunk

Files:

: 1 added
: 3 edited

src/wp-includes/class-wp-hook.php (modified) (1 diff)
src/wp-includes/class-wp-widget-factory.php (modified) (2 diffs)
src/wp-includes/plugin.php (modified) (2 diffs)
tests/phpunit/tests/hooks/buildUniqueId.php (added)

Legend:

: Unmodified
: Added
: Removed

trunk/src/wp-includes/class-wp-hook.php

-              r61457
+              r62408
                 $idx = _wp_filter_build_unique_id( $hook_name, $callback, $priority );
+                if ( null === $idx ) {
+                        return;
+                }
                 $priority_existed = isset( $this->callbacks[ $priority ] );

trunk/src/wp-includes/class-wp-widget-factory.php

-              r54133
+              r62408
         public function register( $widget ) {
                 if ( $widget instanceof WP_Widget ) {
                         $this->widgets[ spl_object_hash( $widget ) ] = $widget;
+                        $this->widgets[ spl_object_id( $widget ) ] = $widget;
                 } else {
                         $this->widgets[ $widget ] = new $widget();
 …
         public function unregister( $widget ) {
                 if ( $widget instanceof WP_Widget ) {
                         unset( $this->widgets[ spl_object_hash( $widget ) ] );
+                        unset( $this->widgets[ spl_object_id( $widget ) ] );
                 } else {
                         unset( $this->widgets[ $widget ] );

trunk/src/wp-includes/plugin.php

-              r62397
+              r62408
  * @since 5.3.0 Removed workarounds for spl_object_hash().
  *              `$hook_name` and `$priority` are no longer used,
+ *              and the function always returns a string.
+ *              and no longer returns false, but can still return void for invalid callbacks.
+ * @since 6.9.0 Returns explicit null if an invalid callback is supplied.
+ * @since 7.1.0 Uses spl_object_id() instead of spl_object_hash() for performance.
+ *
  * @access private
+ *
+ * @param string                $hook_name Unused. The name of the filter to build ID for.
+ * @param callable|string|array $callback  The callback to generate ID for. The callback may
+ *                                         or may not exist.
+ * @param int                   $priority  Unused. The order in which the functions
+ *                                         associated with a particular action are executed.
+ * @return string|null Unique function ID for usage as array key.
+ *                     Null if a valid `$callback` is not passed.
+ */
+function _wp_filter_build_unique_id( $hook_name, $callback, $priority ) {
+ * @param string   $hook_name Unused. The name of the filter to build ID for.
+ * @param callable $callback  The callback to generate ID for. The callback may
+ *                            or may not exist.
+ * @param int      $priority  Unused. The order in which the functions
+ *                            associated with a particular action are executed.
+ * @return string|null Unique function ID for usage as array key, or null if it couldn't be determined.
+ */
+function _wp_filter_build_unique_id( $hook_name, $callback, $priority ): ?string {
         if ( is_string( $callback ) ) {
                 return $callback;
 …
         if ( is_object( $callback ) ) {
+                // Closures are currently implemented as objects.
+                $callback = array( $callback, '' );
+        } else {
+                $callback = (array) $callback;
+                return (string) spl_object_id( (object) $callback );
+        }
+        $callback = (array) $callback;
+        if ( ! isset( $callback[1] ) || ! is_string( $callback[1] ) ) {
+                return null;
+        }
         if ( is_object( $callback[0] ) ) {
                 // Object class calling.
                 return spl_object_hash( $callback[0] ) . $callback[1];
+                return ( (string) spl_object_id( $callback[0] ) ) . $callback[1];
         } elseif ( is_string( $callback[0] ) ) {
                 // Static calling.

Note: See TracChangeset for help on using the changeset viewer.

Make WordPress Core