Changeset 14859

Timestamp:

05/08/2026 08:35:19 PM (2 weeks ago)

Author:

obenland

Message:

Plugin Directory: Improve block validator accuracy for script translations

Closes ​https://github.com/WordPress/wordpress.org/pull/624.

Location:

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory

Files:

• cli/class-block-plugin-checker.php (modified) (2 diffs)
• phpunit.xml (modified) (1 diff)
• shortcodes/class-block-validator.php (modified) (1 diff)
• tests/Block_Plugin_Checker_Translation_Test.php (added)
• tests/fixtures (added)
• tests/fixtures/block-plugin-register-dir (added)
• tests/fixtures/block-plugin-register-dir/block.json (added)
• tests/fixtures/block-plugin-register-dir/plugin.php (added)
• tests/fixtures/block-plugin-with-textdomain (added)
• tests/fixtures/block-plugin-with-textdomain/block.json (added)
• tests/fixtures/block-plugin-with-textdomain/plugin.php (added)

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/cli/class-block-plugin-checker.php

@@ -431 +431 @@
                 );
             } else {
-                $all_calls += $php_calls;
+                // Append, not union — per-file calls share numeric keys from 0 so `+=` would drop later files.
+                array_push( $all_calls, ...$php_calls );
             }
         }
@@ -1020 +1021 @@
 
     /**
-     * Check that the plugin uses `wp_set_script_translations`.
+     * Check that the plugin loads script translations. WordPress auto-loads them when a
+     * block is registered from its `block.json` (via `register_block_type_from_metadata()`
+     * or `register_block_type( __DIR__ )`) and `block.json` declares a `textdomain`.
      */
     function check_for_translation_function() {
         $functions = wp_list_pluck( $this->php_function_calls, 0 );
-        if ( ! in_array( 'wp_set_script_translations', $functions ) ) {
-            $this->record_result(
-                __FUNCTION__,
-                'warning',
-                __( 'No translations are loaded for the scripts.', 'wporg-plugins' ),
-                'wp_set_script_translations'
-            );
-        }
+
+        if ( in_array( 'wp_set_script_translations', $functions, true ) ) {
+            return;
+        }
+
+        $registers_block = in_array( 'register_block_type', $functions, true )
+            || in_array( 'register_block_type_from_metadata', $functions, true );
+
+        if ( $registers_block ) {
+            foreach ( (array) $this->blocks as $block ) {
+                if ( ! empty( $block->textdomain ) ) {
+                    return;
+                }
+            }
+        }
+
+        $this->record_result(
+            __FUNCTION__,
+            'warning',
+            __( 'No translations are loaded for the scripts.', 'wporg-plugins' ),
+            'wp_set_script_translations'
+        );
     }
 

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/phpunit.xml

@@ -12 +12 @@
             <exclude>tests/wporg-plugin-api-performance.php</exclude>
             <exclude>tests/api</exclude>
+            <exclude>tests/fixtures</exclude>
         </testsuite>
     </testsuites>

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/shortcodes/class-block-validator.php

@@ -436 +436 @@
                 return sprintf(
                     // translators: %s is the link to the internationalization docs.
-                    __( 'Block plugins should use <code>wp_set_script_translations</code> to load translations for each script file. <a href="%s">Learn more about internationalization.</a>', 'wporg-plugins' ),
+                    __( 'Block plugins should load translations for each script file, either by registering blocks from <code>block.json</code> with a <code>textdomain</code> set, or by calling <code>wp_set_script_translations</code> directly. <a href="%s">Learn more about internationalization.</a>', 'wporg-plugins' ),
                     'https://developer.wordpress.org/block-editor/developers/internationalization/'
                 );