Changeset 51472

Timestamp:

07/22/2021 10:05:33 AM (3 years ago)

Author:

gziolo

Message:

I18n: Fix broken loop in WP_Theme_JSON_Resolver

Related issue in Gutenberg: ​https://github.com/WordPress/gutenberg/issues/33552

The loop in WP_Theme_JSON_Resolver to extract translatable paths was broken, as it contained an immediate and unconditional return. This caused the loop to immediately exit again after the first iteration, thus never actually looping.

Follow-up to [50959].

Props schlessera.

File:

• trunk/src/wp-includes/class-wp-theme-json-resolver.php (modified) (1 diff)

trunk/src/wp-includes/class-wp-theme-json-resolver.php

@@ -122 +122 @@
             if ( is_numeric( $property ) ) {
                 foreach ( $partial_child as $key => $context ) {
-                    return array(
-                        array(
-                            'path'    => $current_path,
-                            'key'     => $key,
-                            'context' => $context,
-                        ),
+                    $result[] = array(
+                        'path'    => $current_path,
+                        'key'     => $key,
+                        'context' => $context,
                     );
                 }
+                return $result;
             }
             $result = array_merge(