Changeset 57685 for trunk/tests/phpunit/tests/theme.php

Timestamp:

02/21/2024 07:24:12 PM (10 months ago)

Author:

joemcgill

Message:

Themes: Use original template paths when switching blogs.

This fixes a bug introduced by [57129] and [56635] in which deprecating the previous TEMPLATEPATH and STYLESHEETPATH constants in favor of get_template_directory() and get_stylesheet_directory() functions caused the active theme template path to change when using switch_to_blog().

This introduces a new function, wp_set_template_globals(), which is called during the bootstrap process to store the template paths to new globals values $wp_template_path and $wp_stylesheet_path. This restores behavior to how things worked prior to [56635] but retains the ability for template values to be reset for better testability.

Related #18298, #60025.

Props joemcgill, flixos90, mukesh27, swissspidy, manfcarlo, metropolis_john, jeremyfelt.
Fixes #60290.

File:

• trunk/tests/phpunit/tests/theme.php (modified) (1 diff)

trunk/tests/phpunit/tests/theme.php

@@ -1306 +1306 @@
         $this->assertSame( $new_root . '/test-parent', $path2, 'The new template path is not correct' );
     }
+
+    /**
+     * Tests that switch_to_blog() uses the original template path.
+     *
+     * @ticket 60290
+     *
+     * @group ms-required
+     *
+     * @covers ::locate_template
+     */
+    public function test_switch_to_blog_uses_original_template_path() {
+        $old_theme     = wp_get_theme();
+        $template_path = locate_template( 'index.php' );
+
+        $blog_id = self::factory()->blog->create();
+        switch_to_blog( $blog_id );
+
+        switch_theme( 'block-theme' );
+        $new_template_path = locate_template( 'index.php' );
+
+        // Cleanup.
+        restore_current_blog();
+        switch_theme( $old_theme->get_stylesheet() );
+
+        $this->assertSame( $template_path, $new_template_path, 'Switching blogs switches the template path' );
+    }
 }