Changeset 65228 for trunk/src/wp-includes/interactivity-api/class-wp-interactivity-api.php

Timestamp:

11/03/2026 02:36:41 PM (9 days ago)

Author:

jonsurrell

Message:

Interactivity API: Remove event handler bind directives.

Server processing now removes data-wp-bind--on* directives from output.

Developed in: ​https://github.com/WordPress/wordpress-develop/pull/15396

Follow-up to r61885.

Props jonsurrell, ntsekouras, luisherranz.
See #66452.

File:

• trunk/src/wp-includes/interactivity-api/class-wp-interactivity-api.php (modified) (3 diffs)

trunk/src/wp-includes/interactivity-api/class-wp-interactivity-api.php

@@ -1154 +1154 @@
          *              JSON encoder, and a value which cannot be sent to the client is rejected rather than passed to
          *              WP_HTML_Tag_Processor::set_attribute().
+         * @since 7.2.0 Event handler bind directives are removed.
          *
          * @param WP_Interactivity_API_Directives_Processor $p    The directives processor instance.
@@ -1160 +1161 @@
         private function data_wp_bind_processor( WP_Interactivity_API_Directives_Processor $p, string $mode ): void {
                 if ( 'enter' === $mode ) {
-                        $entries = $this->get_directive_entries( $p, 'bind' );
+                        $event_handler_prefix = 'data-wp-bind--on';
+                        $entries              = $this->get_directive_entries( $p, 'bind' );
                         foreach ( $entries as $entry ) {
+                                // Binding event handler attributes is not supported.
+                                if ( str_starts_with( $entry['attribute_name'], $event_handler_prefix ) ) {
+                                        $p->remove_attribute( $entry['attribute_name'] );
+                                        _doing_it_wrong(
+                                                __METHOD__,
+                                                sprintf(
+                                                        /* translators: %s: The directive, e.g. data-wp-on--click. */
+                                                        __( 'Binding event handler attributes is not supported. Please use "%s" instead.' ),
+                                                        esc_attr( 'data-wp-on--' . substr( $entry['attribute_name'], strlen( $event_handler_prefix ) ) )
+                                                ),
+                                                '6.9.2'
+                                        );
+                                        continue;
+                                }
+
                                 if ( empty( $entry['suffix'] ) || null !== $entry['unique_id'] ) {
                                         continue;
@@ -1180 +1197 @@
                                                 ),
                                                 '7.1.4'
-                                        );
-                                        continue;
-                                }
-
-                                // Skip if the suffix is an event handler.
-                                if ( str_starts_with( $entry['suffix'], 'on' ) ) {
-                                        _doing_it_wrong(
-                                                __METHOD__,
-                                                sprintf(
-                                                        /* translators: %s: The directive, e.g. data-wp-on--click. */
-                                                        __( 'Binding event handler attributes is not supported. Please use "%s" instead.' ),
-                                                        esc_attr( 'data-wp-on--' . substr( $entry['suffix'], 2 ) )
-                                                ),
-                                                '6.9.2'
                                         );
                                         continue;