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

Timestamp:

06/27/2023 05:10:28 PM (19 months ago)

Author:

spacedmonkey

Message:

Themes: Use get_theme_file_path() in wp_theme_has_theme_json().

Ensure that all places where theme.json is included, use get_theme_file_path or WP_Theme->get_file_path, so that the path is run through theme_file_path filter. This change also means that the method get_file_path_from_theme can be deprecated, as it is no longer used in core.

Props flixos90, spacedmonkey, costdev, johnbillion, oglekler, hellofromtonya, mukesh27, audrasjb, oandregal.
Fixes #57629.

File:

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

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

@@ -239 +239 @@
 
         if ( null === static::$theme || ! static::has_same_registered_blocks( 'theme' ) ) {
-            $theme_json_file = static::get_file_path_from_theme( 'theme.json' );
             $wp_theme        = wp_get_theme();
-            if ( '' !== $theme_json_file ) {
+            $theme_json_file = $wp_theme->get_file_path( 'theme.json' );
+            if ( is_readable( $theme_json_file ) ) {
                 $theme_json_data = static::read_json_file( $theme_json_file );
                 $theme_json_data = static::translate( $theme_json_data, $wp_theme->get( 'TextDomain' ) );
@@ -261 +261 @@
             if ( $wp_theme->parent() ) {
                 // Get parent theme.json.
-                $parent_theme_json_file = static::get_file_path_from_theme( 'theme.json', true );
-                if ( '' !== $parent_theme_json_file ) {
+                $parent_theme_json_file = $wp_theme->parent()->get_file_path( 'theme.json' );
+                if ( $theme_json_file !== $parent_theme_json_file && is_readable( $parent_theme_json_file ) ) {
                     $parent_theme_json_data = static::read_json_file( $parent_theme_json_file );
                     $parent_theme_json_data = static::translate( $parent_theme_json_data, $wp_theme->parent()->get( 'TextDomain' ) );