From 1b2448770ab826d7b3471fe93039771d4153f246 Mon Sep 17 00:00:00 2001 From: Joe McGill Date: Wed, 8 Nov 2023 18:16:06 -0600 Subject: [PATCH] Remove memoization --- src/wp-includes/theme.php | 80 ++++++++++++++------------------------- 1 file changed, 28 insertions(+), 52 deletions(-) diff --git a/src/wp-includes/theme.php b/src/wp-includes/theme.php index 6315cc2ab5ce2..a943061f52fc6 100644 --- a/src/wp-includes/theme.php +++ b/src/wp-includes/theme.php @@ -188,39 +188,27 @@ function get_stylesheet() { * * @since 1.5.0 * @since 6.4.0 Memoizes filter execution so that it only runs once for the current theme. + * @since 6.4.1 Memoization removed. * * @global string $wp_stylesheet_path Current theme stylesheet directory path. * * @return string Path to active theme's stylesheet directory. */ function get_stylesheet_directory() { - global $wp_stylesheet_path; + $stylesheet = get_stylesheet(); + $theme_root = get_theme_root( $stylesheet ); + $stylesheet_dir = "$theme_root/$stylesheet"; - if ( null === $wp_stylesheet_path ) { - $stylesheet = get_stylesheet(); - $theme_root = get_theme_root( $stylesheet ); - $stylesheet_dir = "$theme_root/$stylesheet"; - - /** - * Filters the stylesheet directory path for the active theme. - * - * @since 1.5.0 - * - * @param string $stylesheet_dir Absolute path to the active theme. - * @param string $stylesheet Directory name of the active theme. - * @param string $theme_root Absolute path to themes directory. - */ - $stylesheet_dir = apply_filters( 'stylesheet_directory', $stylesheet_dir, $stylesheet, $theme_root ); - - // If there are filter callbacks, force the logic to execute on every call. - if ( has_filter( 'stylesheet' ) || has_filter( 'theme_root' ) || has_filter( 'stylesheet_directory' ) ) { - return $stylesheet_dir; - } - - $wp_stylesheet_path = $stylesheet_dir; - } - - return $wp_stylesheet_path; + /** + * Filters the stylesheet directory path for the active theme. + * + * @since 1.5.0 + * + * @param string $stylesheet_dir Absolute path to the active theme. + * @param string $stylesheet Directory name of the active theme. + * @param string $theme_root Absolute path to themes directory. + */ + return apply_filters( 'stylesheet_directory', $stylesheet_dir, $stylesheet, $theme_root ); } /** @@ -338,39 +326,27 @@ function get_template() { * * @since 1.5.0 * @since 6.4.0 Memoizes filter execution so that it only runs once for the current theme. + * @since 6.4.1 Memoization removed. * * @global string $wp_template_path Current theme template directory path. * * @return string Path to active theme's template directory. */ function get_template_directory() { - global $wp_template_path; + $template = get_template(); + $theme_root = get_theme_root( $template ); + $template_dir = "$theme_root/$template"; - if ( null === $wp_template_path ) { - $template = get_template(); - $theme_root = get_theme_root( $template ); - $template_dir = "$theme_root/$template"; - - /** - * Filters the active theme directory path. - * - * @since 1.5.0 - * - * @param string $template_dir The path of the active theme directory. - * @param string $template Directory name of the active theme. - * @param string $theme_root Absolute path to the themes directory. - */ - $template_dir = apply_filters( 'template_directory', $template_dir, $template, $theme_root ); - - // If there are filter callbacks, force the logic to execute on every call. - if ( has_filter( 'template' ) || has_filter( 'theme_root' ) || has_filter( 'template_directory' ) ) { - return $template_dir; - } - - $wp_template_path = $template_dir; - } - - return $wp_template_path; + /** + * Filters the active theme directory path. + * + * @since 1.5.0 + * + * @param string $template_dir The path of the active theme directory. + * @param string $template Directory name of the active theme. + * @param string $theme_root Absolute path to the themes directory. + */ + return apply_filters( 'template_directory', $template_dir, $template, $theme_root ); } /**