Changeset 62452

Timestamp:

06/03/2026 05:18:21 AM (2 months ago)

Author:

westonruter

Message:

Blocks: Include the offending block name in registration error notices.

Three of the four _doing_it_wrong() calls in WP_Block_Type_Registry::register() emitted a generic message that did not identify which block name triggered it, making it hard to locate the offending registration. The notices for non-string names, uppercase characters, and a missing namespace prefix now include the received value (the type via gettype() for non-strings, and the escaped name otherwise), matching the existing "already registered" notice that did already report the name.

Developed in ​https://github.com/WordPress/wordpress-develop/pull/11478.
Follow-up to r44108.

Props manishxdp, desrosj, benjgrolleau, westonruter.
Fixes #65039.

File:

• trunk/src/wp-includes/class-wp-block-type-registry.php (modified) (3 diffs)

trunk/src/wp-includes/class-wp-block-type-registry.php

@@ -56 +56 @@
                         _doing_it_wrong(
                                 __METHOD__,
-                                __( 'Block type names must be strings.' ),
+                                /* translators: %s: The received block type name type. */
+                                sprintf( __( 'Block type names must be strings, received %s.' ), gettype( $name ) ),
                                 '5.0.0'
                         );
@@ -65 +66 @@
                         _doing_it_wrong(
                                 __METHOD__,
-                                __( 'Block type names must not contain uppercase characters.' ),
+                                /* translators: %s: Block name. */
+                                sprintf( __( 'Block type names must not contain uppercase characters. "%s" was given.' ), esc_html( $name ) ),
                                 '5.0.0'
                         );
@@ -75 +77 @@
                         _doing_it_wrong(
                                 __METHOD__,
-                                __( 'Block type names must contain a namespace prefix. Example: my-plugin/my-custom-block-type' ),
+                                /* translators: %s: Block name. */
+                                sprintf( __( 'Block type names must contain a namespace prefix. Example: my-plugin/my-custom-block-type. "%s" was given.' ), esc_html( $name ) ),
                                 '5.0.0'
                         );