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

Timestamp:

01/26/2023 06:37:47 PM (2 years ago)

Author:

flixos90

Message:

Themes: Avoid unnecessary database queries from get_default_block_editor_settings() in WP_Theme_JSON_Resolver::get_theme_data().

The get_default_block_editor_settings() function included several pieces of data that are irrelevant for the purpose that WP_Theme_JSON_Resolver was calling it for, yet resulted in three database queries on page load that can be avoided.

This changeset introduces a new function get_classic_theme_supports_block_editor_settings() to takes responsibility of only the data needed in WP_Theme_JSON_Resolver, which now uses that function. This leads to a reduction of database queries and accordingly a performance improvement.

Props mamaduka, spacedmonkey, oandregal, flixos90, audrasjb, mukesh27.
Fixes #57547.

File:

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

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

@@ -287 +287 @@
          * and merge the static::$theme upon that.
          */
-        $theme_support_data = WP_Theme_JSON::get_from_editor_settings( get_default_block_editor_settings() );
+        $theme_support_data = WP_Theme_JSON::get_from_editor_settings( get_classic_theme_supports_block_editor_settings() );
         if ( ! wp_theme_has_theme_json() ) {
             if ( ! isset( $theme_support_data['settings']['color'] ) ) {