Skip to content
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

Improve caching and logic in get_user_data_from_wp_global_styles. #2414

Closed
wants to merge 15 commits into from
Closed

Improve caching and logic in get_user_data_from_wp_global_styles. #2414

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
075d00a
Improve caching of global styles.
spacedmonkey Jun 13, 2022
2da31ee
Merge branch 'trunk' into fix/global-query
spacedmonkey Jun 17, 2022
83f9579
Add some unit tests.
spacedmonkey Jun 17, 2022
82ffd45
Improve caching.
spacedmonkey Jun 17, 2022
95667ce
Handle more edge cases.
spacedmonkey Jun 17, 2022
71bcb6c
Improve tests.
spacedmonkey Jun 17, 2022
478fa51
Merge branch 'trunk' into fix/global-query
spacedmonkey Jul 5, 2022
8bb1cd7
Add expiration back again.
spacedmonkey Jul 5, 2022
32d7306
Fix style.
spacedmonkey Jul 5, 2022
7482d90
Merge branch 'trunk' into fix/global-query
spacedmonkey Sep 9, 2022
12e6322
Fix test.
spacedmonkey Sep 9, 2022
67537a2
Apply suggestions from code review
spacedmonkey Sep 15, 2022
0389421
Merge branch 'trunk' into fix/global-query
spacedmonkey Sep 15, 2022
76fd86c
Formatting of tests.
spacedmonkey Sep 15, 2022
8e567de
Fix styling.
spacedmonkey Sep 15, 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
30 changes: 16 additions & 14 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 @@ -255,25 +255,27 @@ public static function get_user_data_from_wp_global_styles( $theme, $create_post
}
$user_cpt = array();
$post_type_filter = 'wp_global_styles';
$stylesheet = $theme->get_stylesheet();
$args = array(
'numberposts' => 1,
draganescu marked this conversation as resolved.
Show resolved Hide resolved
'orderby' => 'date',
'order' => 'desc',
'post_type' => $post_type_filter,
'post_status' => $post_status_filter,
'tax_query' => array(
'posts_per_page' => 1,
'orderby' => 'post_date',
'order' => 'desc',
'post_type' => $post_type_filter,
'post_status' => $post_status_filter,
'ignore_sticky_posts' => true,
'no_found_rows' => true,
'tax_query' => array(
array(
'taxonomy' => 'wp_theme',
'field' => 'name',
'terms' => $theme->get_stylesheet(),
'terms' => $stylesheet,
),
),
);

$cache_key = sprintf( 'wp_global_styles_%s', md5( serialize( $args ) ) );
$post_id = wp_cache_get( $cache_key );

if ( (int) $post_id > 0 ) {
$post_id = (int) get_transient( $cache_key );
spacedmonkey marked this conversation as resolved.
Show resolved Hide resolved
if ( $post_id > 0 ) {
return get_post( $post_id, ARRAY_A );
}

Expand All @@ -282,7 +284,7 @@ public static function get_user_data_from_wp_global_styles( $theme, $create_post
return $user_cpt;
}

$recent_posts = wp_get_recent_posts( $args );
$recent_posts = new WP_Query( $args );
if ( is_array( $recent_posts ) && ( count( $recent_posts ) === 1 ) ) {
$user_cpt = $recent_posts[0];
} elseif ( $create_post ) {
Expand All @@ -292,17 +294,17 @@ public static function get_user_data_from_wp_global_styles( $theme, $create_post
'post_status' => 'publish',
'post_title' => 'Custom Styles',
'post_type' => $post_type_filter,
'post_name' => 'wp-global-styles-' . urlencode( wp_get_theme()->get_stylesheet() ),
'post_name' => 'wp-global-styles-' . urlencode( $stylesheet ),
'tax_input' => array(
'wp_theme' => array( wp_get_theme()->get_stylesheet() ),
'wp_theme' => array( $stylesheet ),
),
),
true
);
$user_cpt = get_post( $cpt_post_id, ARRAY_A );
}
$cache_expiration = $user_cpt ? DAY_IN_SECONDS : HOUR_IN_SECONDS;
wp_cache_set( $cache_key, $user_cpt ? $user_cpt['ID'] : -1, '', $cache_expiration );
set_transient( $cache_key, $user_cpt ? $user_cpt['ID'] : -1, $cache_expiration );

return $user_cpt;
}
Expand Down