Changeset 56529

Timestamp:

09/06/2023 09:46:09 PM (11 months ago)

Author:

azaozz

Message:

Editor: Prevent possibility of a fatal error when previewing block themes.

Ensures that preview callbacks attached to the stylesheet and template filters do not run before pluggable.php has been included. These callbacks need functionality from pluggable.php.

Props: scruffian, johnbillion, SergeyBiryukov, okat, okat.
Fixes: #59000.

Location:

trunk

Files:

• src/wp-includes/default-filters.php (modified) (1 diff)
• src/wp-includes/theme-previews.php (modified) (1 diff)
• tests/phpunit/tests/theme-previews.php (added)

trunk/src/wp-includes/default-filters.php

@@ -533 +533 @@
 add_action( 'transition_post_status', '_wp_keep_alive_customize_changeset_dependent_auto_drafts', 20, 3 );
 
+
+
+
 // Calendar widget cache.
 add_action( 'save_post', 'delete_get_calendar_cache' );

trunk/src/wp-includes/theme-previews.php

@@ -76 +76 @@
 }
 
-// Attaches filters to enable theme previews in the Site Editor.
-if ( ! empty( $_GET['wp_theme_preview'] ) ) {
-    add_filter( 'stylesheet', 'wp_get_theme_preview_path' );
-    add_filter( 'template', 'wp_get_theme_preview_path' );
-    add_action( 'init', 'wp_attach_theme_preview_middleware' );
-    add_action( 'admin_head', 'wp_block_theme_activate_nonce' );
+/**
+ * Add filters and actions to enable Block Theme Previews in the Site Editor.
+ *
+ * The filters and actions should be added after `pluggable.php` is included as they may
+ * trigger code that uses `current_user_can()` which requires functionality from `pluggable.php`.
+ *
+ * @since 6.3.2
+ */
+function initialize_theme_preview_hooks() {
+    if ( ! empty( $_GET['wp_theme_preview'] ) ) {
+        add_filter( 'stylesheet', 'wp_get_theme_preview_path' );
+        add_filter( 'template', 'wp_get_theme_preview_path' );
+        add_action( 'init', 'wp_attach_theme_preview_middleware' );
+        add_action( 'admin_head', 'wp_block_theme_activate_nonce' );
+    }
 }