Skip to content

Commit 8900f0a

Browse files
Coding Standards: Remove extra check in wp_get_theme_data_template_parts().
The `false === $metadata` check would always be true, as a non-false value causes an early return right before. To clarify this behavior, the needless priming of `$metadata = false` was removed as well, as there is no scenario where it will not be immediately overwritten. Follow-up to [56385]. Props justlevine. See #63268. git-svn-id: https://develop.svn.wordpress.org/trunk@60459 602fd350-edb4-49c9-b593-d223f7449a82
1 parent 8671dfe commit 8900f0a

File tree

1 file changed

+3
-6
lines changed

1 file changed

+3
-6
lines changed

src/wp-includes/global-styles-and-settings.php

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -483,19 +483,16 @@ function wp_get_theme_data_template_parts() {
483483
$cache_key = 'wp_get_theme_data_template_parts';
484484
$can_use_cached = ! wp_is_development_mode( 'theme' );
485485

486-
$metadata = false;
487486
if ( $can_use_cached ) {
488487
$metadata = wp_cache_get( $cache_key, $cache_group );
489488
if ( false !== $metadata ) {
490489
return $metadata;
491490
}
492491
}
493492

494-
if ( false === $metadata ) {
495-
$metadata = WP_Theme_JSON_Resolver::get_theme_data( array(), array( 'with_supports' => false ) )->get_template_parts();
496-
if ( $can_use_cached ) {
497-
wp_cache_set( $cache_key, $metadata, $cache_group );
498-
}
493+
$metadata = WP_Theme_JSON_Resolver::get_theme_data( array(), array( 'with_supports' => false ) )->get_template_parts();
494+
if ( $can_use_cached ) {
495+
wp_cache_set( $cache_key, $metadata, $cache_group );
499496
}
500497

501498
return $metadata;

0 commit comments

Comments
 (0)