Skip to content

Fix wp_head performance regression for classic themes #3536

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
felixarntz wants to merge 15 commits into from
Closed

Fix wp_head performance regression for classic themes #3536

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
787b2c2
Avoid parsing core's theme.json and theme-i18n.json when using a clas…
felixarntz Oct 27, 2022
aff6347
Avoid DB query for global styles when using a classic theme without s…
felixarntz Oct 27, 2022
196c215
Merge branch 'trunk' into fix/classic-themes-wp_head-regression
felixarntz Nov 2, 2022
0c768d3
Further optimize WP_Theme_JSON_Resolver logic for classic themes, use…
felixarntz Nov 2, 2022
b9ff774
Add demo version of theme-json.php from WP core theme.json (via var_e…
felixarntz Nov 2, 2022
f34fe53
Merge branch 'trunk' into fix/classic-themes-wp_head-regression
felixarntz Nov 8, 2022
6a835a4
Remove temporary theme.json in favor of https://github.com/WordPress/…
felixarntz Nov 8, 2022
1ab511c
Include parsing parent theme.json logic in cached logic block.
felixarntz Nov 8, 2022
bf4e711
Merge branch 'trunk' into fix/classic-themes-wp_head-regression
felixarntz Nov 9, 2022
4966109
Add test coverage and fix relevant tests to require a block theme.
felixarntz Nov 9, 2022
3a24f75
Merge branch 'trunk' into fix/classic-themes-wp_head-regression
felixarntz Nov 9, 2022
34c1957
Address code review feedback.
felixarntz Nov 9, 2022
392391a
Limit comment to 80 characters per line.
felixarntz Nov 10, 2022
782c49d
Merge branch 'trunk' into fix/classic-themes-wp_head-regression
felixarntz Nov 10, 2022
3ab53bf
Merge branch 'fix/classic-themes-wp_head-regression' of github.com:fe…
felixarntz Nov 10, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Remove temporary theme.json in favor of WordPress/gutenberg#45616.
  • Loading branch information
@felixarntz
felixarntz committed Nov 8, 2022
commit 6a835a4084b4f990cfe30724c5afa1185d09aeac
10 changes: 2 additions & 8 deletions src/wp-includes/class-wp-theme-json-resolver.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -170,14 +170,8 @@ public static function get_core_data() {
return static::$core;
}

// TODO: Generate 'theme-json.php' at build time which returns fully parsed and translation-ready 'theme.json'
// as a PHP associative array.
if ( file_exists( __DIR__ . '/theme-json.php' ) ) {
$config = require __DIR__ . '/theme-json.php';
} else {
$config = static::read_json_file( __DIR__ . '/theme.json' );
$config = static::translate( $config );
}
$config = static::read_json_file( __DIR__ . '/theme.json' );
$config = static::translate( $config );

/**
* Filters the default data provided by WordPress for global styles & settings.
Expand Down
12 changes: 0 additions & 12 deletions src/wp-includes/theme-json.php
View file
Open in desktop

This file was deleted.