Changeset 53160

Timestamp:

04/12/2022 03:59:47 PM (3 years ago)

Author:

gziolo

Message:

Editor: Use block settings to pass assets for the iframed editor

Backports changes applied in Gutenberg in ​https://github.com/WordPress/gutenberg/pull/37193.

Props zieladam.
See #55505.
Follow-up for [53157].

Location:

trunk/src/wp-includes

Files:

• block-editor.php (modified) (2 diffs)
• default-filters.php (modified) (1 diff)
• deprecated.php (modified) (1 diff)
• script-loader.php (modified) (1 diff)

trunk/src/wp-includes/block-editor.php

@@ -289 +289 @@
 
 /**
+ * Collect the block editor assets that need to be loaded into the editor's iframe.
+ *
+ * @since 6.0.0
+ * @access private
+ *
+ * @global string $pagenow The filename of the current screen.
+ *
+ * @return array The block editor assets: styles and scripts.
+ */
+function _wp_get_iframed_editor_assets() {
+    global $pagenow;
+
+    $script_handles = array();
+    $style_handles  = array(
+        'wp-block-editor',
+        'wp-block-library',
+        'wp-block-library-theme',
+        'wp-edit-blocks',
+    );
+
+    if ( 'widgets.php' === $pagenow || 'customize.php' === $pagenow ) {
+        $style_handles[] = 'wp-widgets';
+        $style_handles[] = 'wp-edit-widgets';
+    }
+
+    $block_registry = WP_Block_Type_Registry::get_instance();
+
+    foreach ( $block_registry->get_all_registered() as $block_type ) {
+        if ( ! empty( $block_type->style ) ) {
+            $style_handles[] = $block_type->style;
+        }
+
+        if ( ! empty( $block_type->editor_style ) ) {
+            $style_handles[] = $block_type->editor_style;
+        }
+
+        if ( ! empty( $block_type->script ) ) {
+            $script_handles[] = $block_type->script;
+        }
+    }
+
+    $style_handles = array_unique( $style_handles );
+    $done          = wp_styles()->done;
+
+    ob_start();
+
+    // We do not need reset styles for the iframed editor.
+    wp_styles()->done = array( 'wp-reset-editor-styles' );
+    wp_styles()->do_items( $style_handles );
+    wp_styles()->done = $done;
+
+    $styles = ob_get_clean();
+
+    $script_handles = array_unique( $script_handles );
+    $done           = wp_scripts()->done;
+
+    ob_start();
+
+    wp_scripts()->done = array();
+    wp_scripts()->do_items( $script_handles );
+    wp_scripts()->done = $done;
+
+    $scripts = ob_get_clean();
+
+    return array(
+        'styles'  => $styles,
+        'scripts' => $scripts,
+    );
+}
+
+/**
  * Returns the contextualized block editor settings for a selected editor context.
  *
@@ -395 +466 @@
     }
 
-    $editor_settings['localAutosaveInterval'] = 15;
-
+    $editor_settings['__unstableResolvedAssets']         = _wp_get_iframed_editor_assets();
+    $editor_settings['localAutosaveInterval']            = 15;
     $editor_settings['__experimentalDiscussionSettings'] = array(
         'commentOrder'        => get_option( 'comment_order' ),

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

@@ -592 +592 @@
 add_action( 'wp_footer', 'wp_maybe_inline_styles', 1 ); // Run for late-loaded styles in the footer.
 
-add_action( 'admin_footer-post.php', 'wp_add_iframed_editor_assets_html' );
-add_action( 'admin_footer-post-new.php', 'wp_add_iframed_editor_assets_html' );
-add_action( 'admin_footer-widgets.php', 'wp_add_iframed_editor_assets_html' );
-add_action( 'admin_footer-site-editor.php', 'wp_add_iframed_editor_assets_html' );
-
 add_action( 'use_block_editor_for_post_type', '_disable_block_editor_for_navigation_post_type', 10, 2 );
 add_action( 'edit_form_after_title', '_disable_content_editor_for_navigation_post_type' );

trunk/src/wp-includes/deprecated.php

@@ -4302 +4302 @@
         $spacing_support['__experimentalSkipSerialization'];
 }
+
+/**
+ * Inject the block editor assets that need to be loaded into the editor's iframe as an inline script.
+ *
+ * @since 5.8.0
+ * @deprecated 6.0.0
+ */
+function wp_add_iframed_editor_assets_html() {
+    _deprecated_function( __FUNCTION__, '6.0.0' );
+}

trunk/src/wp-includes/script-loader.php

@@ -2885 +2885 @@
 
 /**
- * Inject the block editor assets that need to be loaded into the editor's iframe as an inline script.
- *
- * @since 5.8.0
- *
- * @global string $pagenow The filename of the current screen.
- */
-function wp_add_iframed_editor_assets_html() {
-    global $pagenow;
-
-    if ( ! wp_should_load_block_editor_scripts_and_styles() ) {
-        return;
-    }
-
-    $script_handles = array();
-    $style_handles  = array(
-        'wp-block-editor',
-        'wp-block-library',
-        'wp-block-library-theme',
-        'wp-edit-blocks',
-    );
-
-    if ( 'widgets.php' === $pagenow || 'customize.php' === $pagenow ) {
-        $style_handles[] = 'wp-widgets';
-        $style_handles[] = 'wp-edit-widgets';
-    }
-
-    $block_registry = WP_Block_Type_Registry::get_instance();
-
-    foreach ( $block_registry->get_all_registered() as $block_type ) {
-        if ( ! empty( $block_type->style ) ) {
-            $style_handles[] = $block_type->style;
-        }
-
-        if ( ! empty( $block_type->editor_style ) ) {
-            $style_handles[] = $block_type->editor_style;
-        }
-
-        if ( ! empty( $block_type->script ) ) {
-            $script_handles[] = $block_type->script;
-        }
-    }
-
-    $style_handles = array_unique( $style_handles );
-    $done          = wp_styles()->done;
-
-    ob_start();
-
-    // We do not need reset styles for the iframed editor.
-    wp_styles()->done = array( 'wp-reset-editor-styles' );
-    wp_styles()->do_items( $style_handles );
-    wp_styles()->done = $done;
-
-    $styles = ob_get_clean();
-
-    $script_handles = array_unique( $script_handles );
-    $done           = wp_scripts()->done;
-
-    ob_start();
-
-    wp_scripts()->done = array();
-    wp_scripts()->do_items( $script_handles );
-    wp_scripts()->done = $done;
-
-    $scripts = ob_get_clean();
-
-    $editor_assets = wp_json_encode(
-        array(
-            'styles'  => $styles,
-            'scripts' => $scripts,
-        )
-    );
-
-    echo "<script>window.__editorAssets = $editor_assets</script>";
-}
-
-/**
  * Function that enqueues the CSS Custom Properties coming from theme.json.
  *