Skip to content

Store merged data in memory #5024

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
joemcgill wants to merge 8 commits into from
Closed

Store merged data in memory #5024

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
8f3fbcd
Store merged data in memory
joemcgill Aug 17, 2023
b9cd9c8
Allow cached merged data to be cleaned
joemcgill Aug 17, 2023
d1319a1
Validate block registration for merged data
joemcgill Aug 18, 2023
f985db4
Merge branch 'trunk' into update/57789-store-merged-theme-json
joemcgill Sep 18, 2023
6811e6d
Cache theme supports data for use when caching merged data
joemcgill Sep 18, 2023
b477c3d
Remove unused global var
joemcgill Sep 18, 2023
a5922fe
Don't return early in get_theme_data
joemcgill Sep 18, 2023
3727709
Apply PR feedback
joemcgill Sep 22, 2023
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
Next Next commit
Store merged data in memory
  • Loading branch information
@joemcgill
joemcgill committed Aug 17, 2023
commit 8f3fbcdeff32d609f2e475d098e92d0f732cde92
23 changes: 23 additions & 0 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 @@ -65,6 +65,19 @@ class WP_Theme_JSON_Resolver {
*/
protected static $user = null;

/**
* Container for merged data.
*
* @since n.e.x.t
* @var WP_Theme_JSON
*/
protected static $merged = array(
'default' => null,
'blocks' => null,
'theme' => null,
'custom' => null,
);

/**
* Stores the ID of the custom post type
* that holds the user data.
Expand Down Expand Up @@ -576,6 +589,14 @@ public static function get_merged_data( $origin = 'custom' ) {
_deprecated_argument( __FUNCTION__, '5.9.0' );
}

if ( ! in_array( $origin, WP_Theme_JSON::VALID_ORIGINS, true ) ) {
$origin = 'custom';
}

if ( null !== static::$merged[ $origin ] ) {
return static::$merged[ $origin ];
}

$result = new WP_Theme_JSON();
$result->merge( static::get_core_data() );
if ( 'default' === $origin ) {
Expand All @@ -597,6 +618,8 @@ public static function get_merged_data( $origin = 'custom' ) {
$result->merge( static::get_user_data() );
$result->set_spacing_sizes();

static::$merged[ $origin ] = $result;
Copy link
Member

@felixarntz felixarntz Sep 21, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are early returns above, so I think this needs to be added to all those clauses as well, otherwise only the custom origin will ever have cached values.

Copy link
Member Author

@joemcgill joemcgill Sep 21, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I actually wonder if there's any value in storing anything other than the completed merged data. I'll raise that question as part of the PR to the Gutenberg repo, but this is good feedback!


return $result;
}

Expand Down