<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head><meta http-equiv="content-type" content="text/html; charset=utf-8" />
<title>[57790] trunk: Block Hooks: Use new Templates Controller filter instead of action.</title>
</head>
<body>
<style type="text/css"><!--
#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }
#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }
#msg dt:after { content:':';}
#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; }
#msg dl a { font-weight: bold}
#msg dl a:link { color:#fc3; }
#msg dl a:active { color:#ff0; }
#msg dl a:visited { color:#cc6; }
h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }
#msg pre { white-space: pre-line; overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }
#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }
#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }
#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }
#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }
#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }
#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }
#logmsg ul { text-indent: -1em; padding-left: 1em; }#logmsg ol { text-indent: -1.5em; padding-left: 1.5em; }
#logmsg > ul, #logmsg > ol { margin: 0 0 1em 0; }
#logmsg pre { background: #eee; padding: 1em; }
#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}
#logmsg dl { margin: 0; }
#logmsg dt { font-weight: bold; }
#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }
#logmsg dd:before { content:'\00bb';}
#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }
#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }
#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }
#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }
#logmsg table th.Corner { text-align: left; }
#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }
#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }
#patch { width: 100%; }
#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}
#patch .propset h4, #patch .binary h4 {margin:0;}
#patch pre {padding:0;line-height:1.2em;margin:0;}
#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}
#patch .propset .diff, #patch .binary .diff {padding:10px 0;}
#patch span {display:block;padding:0 10px;}
#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}
#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}
#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}
#patch .lines, .info {color:#888;background:#fff;}
--></style>
<div id="msg">
<dl class="meta" style="font-size: 105%">
<dt style="float: left; width: 6em; font-weight: bold">Revision</dt> <dd><a style="font-weight: bold" href="https://core.trac.wordpress.org/changeset/57790">57790</a><script type="application/ld+json">{"@context":"http://schema.org","@type":"EmailMessage","description":"Review this Commit","action":{"@type":"ViewAction","url":"https://core.trac.wordpress.org/changeset/57790","name":"Review Commit"}}</script></dd>
<dt style="float: left; width: 6em; font-weight: bold">Author</dt> <dd>Bernhard Reiter</dd>
<dt style="float: left; width: 6em; font-weight: bold">Date</dt> <dd>2024-03-07 14:10:31 +0000 (Thu, 07 Mar 2024)</dd>
</dl>
<pre style='padding-left: 1em; margin: 2em 0; border-left: 2px solid #ccc; line-height: 1.25; font-size: 105%; font-family: sans-serif'>Block Hooks: Use new Templates Controller filter instead of action.
This changeset adds a new `rest_pre_insert_{$this->post_type}` filter in the `WP_REST_Templates_Controller`, where it is applied to the return value of the `prepare_item_for_database` method. (This is consistent with the `WP_REST_Post_Controller`, where that filter has existed before.)
The new filter is then used to inject hooked blocks into the template (or template part) content received via the endpoint, prior to persisting it to the database.
This supersedes the previous mechanism, which was using the `rest_after_insert_{$this->post_type}` ''action'', from which it performed an additional `wp_update_post` call to update the template (part) content with the hooked blocks injected. The new technique eschews that additional call and the resulting extra revision it created, as well as a problem with regard to duplicated escaping and sanitization, which had caused some special characters to be garbled.
Props tomjcafferkey, gziolo, swissspidy, karolmanijak.
Fixes <a href="https://core.trac.wordpress.org/ticket/60671">#60671</a>.</pre>
<h3>Modified Paths</h3>
<ul>
<li><a href="#trunksrcwpincludesblocktemplateutilsphp">trunk/src/wp-includes/block-template-utils.php</a></li>
<li><a href="#trunksrcwpincludesdefaultfiltersphp">trunk/src/wp-includes/default-filters.php</a></li>
<li><a href="#trunksrcwpincludesrestapiendpointsclasswpresttemplatescontrollerphp">trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-templates-controller.php</a></li>
<li><a href="#trunktestsphpunittestsblocktemplateutilsphp">trunk/tests/phpunit/tests/block-template-utils.php</a></li>
<li><a href="#trunktestsphpunittestsrestapiwpRestTemplatesControllerphp">trunk/tests/phpunit/tests/rest-api/wpRestTemplatesController.php</a></li>
</ul>
</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunksrcwpincludesblocktemplateutilsphp"></a>
<div class="modfile"><h4 style="background-color: #eee; color: inherit; margin: 1em 0; padding: 1.3em; font-size: 115%">Modified: trunk/src/wp-includes/block-template-utils.php</h4>
<pre class="diff"><span>
<span class="info" style="display: block; padding: 0 10px; color: #888">--- trunk/src/wp-includes/block-template-utils.php 2024-03-07 07:06:53 UTC (rev 57789)
+++ trunk/src/wp-includes/block-template-utils.php 2024-03-07 14:10:31 UTC (rev 57790)
</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -1432,20 +1432,33 @@
</span><span class="cx" style="display: block; padding: 0 10px"> $template_hierarchy[] = 'index';
</span><span class="cx" style="display: block; padding: 0 10px"> return $template_hierarchy;
</span><span class="cx" style="display: block; padding: 0 10px"> }
</span><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+
</ins><span class="cx" style="display: block; padding: 0 10px"> /**
</span><span class="cx" style="display: block; padding: 0 10px"> * Inject ignoredHookedBlocks metadata attributes into a template or template part.
</span><span class="cx" style="display: block; padding: 0 10px"> *
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">- * Given a `wp_template` or `wp_template_part` post object, locate all blocks that have
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+ * Given an object that represents a `wp_template` or `wp_template_part` post object
+ * prepared for inserting or updating the database, locate all blocks that have
</ins><span class="cx" style="display: block; padding: 0 10px"> * hooked blocks, and inject a `metadata.ignoredHookedBlocks` attribute into the anchor
</span><span class="cx" style="display: block; padding: 0 10px"> * blocks to reflect the latter.
</span><span class="cx" style="display: block; padding: 0 10px"> *
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">- * @param WP_Post $post A post object with post type set to `wp_template` or `wp_template_part`.
- * @return WP_Post The updated post object.
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+ * @since 6.5.0
+ * @access private
+ *
+ * @param stdClass $post An object representing a template or template part
+ * prepared for inserting or updating the database.
+ * @param WP_REST_Request $request Request object.
+ * @return stdClass The updated object representing a template or template part.
</ins><span class="cx" style="display: block; padding: 0 10px"> */
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-function inject_ignored_hooked_blocks_metadata_attributes( $post ) {
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+function inject_ignored_hooked_blocks_metadata_attributes( $post, $request ) {
+ $filter_name = current_filter();
+ if ( ! str_starts_with( $filter_name, 'rest_pre_insert_' ) ) {
+ return $post;
+ }
+ $post_type = str_replace( 'rest_pre_insert_', '', $filter_name );
+
</ins><span class="cx" style="display: block; padding: 0 10px"> $hooked_blocks = get_hooked_blocks();
</span><span class="cx" style="display: block; padding: 0 10px"> if ( empty( $hooked_blocks ) && ! has_filter( 'hooked_block_types' ) ) {
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">- return;
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+ return $post;
</ins><span class="cx" style="display: block; padding: 0 10px"> }
</span><span class="cx" style="display: block; padding: 0 10px">
</span><span class="cx" style="display: block; padding: 0 10px"> // At this point, the post has already been created.
</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -1452,19 +1465,15 @@
</span><span class="cx" style="display: block; padding: 0 10px"> // We need to build the corresponding `WP_Block_Template` object as context argument for the visitor.
</span><span class="cx" style="display: block; padding: 0 10px"> // To that end, we need to suppress hooked blocks from getting inserted into the template.
</span><span class="cx" style="display: block; padding: 0 10px"> add_filter( 'hooked_block_types', '__return_empty_array', 99999, 0 );
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">- $template = _build_block_template_result_from_post( $post );
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+ $template = $request['id'] ? get_block_template( $request['id'], $post_type ) : null;
</ins><span class="cx" style="display: block; padding: 0 10px"> remove_filter( 'hooked_block_types', '__return_empty_array', 99999 );
</span><span class="cx" style="display: block; padding: 0 10px">
</span><span class="cx" style="display: block; padding: 0 10px"> $before_block_visitor = make_before_block_visitor( $hooked_blocks, $template, 'set_ignored_hooked_blocks_metadata' );
</span><span class="cx" style="display: block; padding: 0 10px"> $after_block_visitor = make_after_block_visitor( $hooked_blocks, $template, 'set_ignored_hooked_blocks_metadata' );
</span><span class="cx" style="display: block; padding: 0 10px">
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">- $blocks = parse_blocks( $template->content );
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+ $blocks = parse_blocks( $post->post_content );
</ins><span class="cx" style="display: block; padding: 0 10px"> $content = traverse_and_serialize_blocks( $blocks, $before_block_visitor, $after_block_visitor );
</span><span class="cx" style="display: block; padding: 0 10px">
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">- wp_update_post(
- array(
- 'ID' => $post->ID,
- 'post_content' => $content,
- )
- );
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+ $post->post_content = $content;
+ return $post;
</ins><span class="cx" style="display: block; padding: 0 10px"> }
</span></span></pre></div>
<a id="trunksrcwpincludesdefaultfiltersphp"></a>
<div class="modfile"><h4 style="background-color: #eee; color: inherit; margin: 1em 0; padding: 1.3em; font-size: 115%">Modified: trunk/src/wp-includes/default-filters.php</h4>
<pre class="diff"><span>
<span class="info" style="display: block; padding: 0 10px; color: #888">--- trunk/src/wp-includes/default-filters.php 2024-03-07 07:06:53 UTC (rev 57789)
+++ trunk/src/wp-includes/default-filters.php 2024-03-07 14:10:31 UTC (rev 57790)
</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -752,9 +752,8 @@
</span><span class="cx" style="display: block; padding: 0 10px"> add_action( 'before_delete_post', '_wp_before_delete_font_face', 10, 2 );
</span><span class="cx" style="display: block; padding: 0 10px"> add_action( 'init', '_wp_register_default_font_collections' );
</span><span class="cx" style="display: block; padding: 0 10px">
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-// It might be nice to use a filter instead of an action, but the `WP_REST_Templates_Controller` doesn't
-// provide one (unlike e.g. `WP_REST_Posts_Controller`, which has `rest_pre_insert_{$this->post_type}`).
-add_action( 'rest_after_insert_wp_template', 'inject_ignored_hooked_blocks_metadata_attributes', 10, 3 );
-add_action( 'rest_after_insert_wp_template_part', 'inject_ignored_hooked_blocks_metadata_attributes', 10, 3 );
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+// Add ignoredHookedBlocks metadata attribute to the template and template part post types.
+add_filter( 'rest_pre_insert_wp_template', 'inject_ignored_hooked_blocks_metadata_attributes', 10, 2 );
+add_filter( 'rest_pre_insert_wp_template_part', 'inject_ignored_hooked_blocks_metadata_attributes', 10, 2 );
</ins><span class="cx" style="display: block; padding: 0 10px">
</span><span class="cx" style="display: block; padding: 0 10px"> unset( $filter, $action );
</span></span></pre></div>
<a id="trunksrcwpincludesrestapiendpointsclasswpresttemplatescontrollerphp"></a>
<div class="modfile"><h4 style="background-color: #eee; color: inherit; margin: 1em 0; padding: 1.3em; font-size: 115%">Modified: trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-templates-controller.php</h4>
<pre class="diff"><span>
<span class="info" style="display: block; padding: 0 10px; color: #888">--- trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-templates-controller.php 2024-03-07 07:06:53 UTC (rev 57789)
+++ trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-templates-controller.php 2024-03-07 14:10:31 UTC (rev 57790)
</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -619,7 +619,8 @@
</span><span class="cx" style="display: block; padding: 0 10px"> $changes->post_author = $post_author;
</span><span class="cx" style="display: block; padding: 0 10px"> }
</span><span class="cx" style="display: block; padding: 0 10px">
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">- return $changes;
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+ /** This filter is documented in wp-includes/rest-api/endpoints/class-wp-rest-posts-controller.php */
+ return apply_filters( "rest_pre_insert_{$this->post_type}", $changes, $request );
</ins><span class="cx" style="display: block; padding: 0 10px"> }
</span><span class="cx" style="display: block; padding: 0 10px">
</span><span class="cx" style="display: block; padding: 0 10px"> /**
</span></span></pre></div>
<a id="trunktestsphpunittestsblocktemplateutilsphp"></a>
<div class="modfile"><h4 style="background-color: #eee; color: inherit; margin: 1em 0; padding: 1.3em; font-size: 115%">Modified: trunk/tests/phpunit/tests/block-template-utils.php</h4>
<pre class="diff"><span>
<span class="info" style="display: block; padding: 0 10px; color: #888">--- trunk/tests/phpunit/tests/block-template-utils.php 2024-03-07 07:06:53 UTC (rev 57789)
+++ trunk/tests/phpunit/tests/block-template-utils.php 2024-03-07 14:10:31 UTC (rev 57790)
</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -87,6 +87,28 @@
</span><span class="cx" style="display: block; padding: 0 10px"> }
</span><span class="cx" style="display: block; padding: 0 10px">
</span><span class="cx" style="display: block; padding: 0 10px"> /**
</span><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+ * Tear down after each test.
+ *
+ * @since 6.5.0
+ */
+ public function tear_down() {
+ global $wp_current_filter;
+
+ if (
+ 'rest_pre_insert_wp_template' === current_filter() ||
+ 'rest_pre_insert_wp_template_part' === current_filter()
+ ) {
+ array_pop( $wp_current_filter );
+ }
+
+ if ( WP_Block_Type_Registry::get_instance()->is_registered( 'tests/hooked-block' ) ) {
+ unregister_block_type( 'tests/hooked-block' );
+ }
+
+ parent::tear_down();
+ }
+
+ /**
</ins><span class="cx" style="display: block; padding: 0 10px"> * @ticket 59338
</span><span class="cx" style="display: block; padding: 0 10px"> *
</span><span class="cx" style="display: block; padding: 0 10px"> * @covers ::_inject_theme_attribute_in_template_part_block
</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -390,4 +412,70 @@
</span><span class="cx" style="display: block; padding: 0 10px"> }
</span><span class="cx" style="display: block; padding: 0 10px"> $this->assertTrue( $has_html_files, 'contains at least one html file' );
</span><span class="cx" style="display: block; padding: 0 10px"> }
</span><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+
+ /**
+ * @ticket 60671
+ *
+ * @covers inject_ignored_hooked_blocks_metadata_attributes
+ */
+ public function test_inject_ignored_hooked_blocks_metadata_attributes_into_template() {
+ global $wp_current_filter;
+ // Mock currently set filter.
+ $wp_current_filter[] = 'rest_pre_insert_wp_template';
+
+ register_block_type(
+ 'tests/hooked-block',
+ array(
+ 'block_hooks' => array(
+ 'tests/anchor-block' => 'after',
+ ),
+ )
+ );
+
+ $id = self::TEST_THEME . '//' . 'my_template';
+ $request = new WP_REST_Request( 'POST', '/wp/v2/templates/' . $id );
+
+ $changes = new stdClass();
+ $changes->post_content = '<!-- wp:tests/anchor-block -->Hello<!-- /wp:tests/anchor-block -->';
+
+ $post = inject_ignored_hooked_blocks_metadata_attributes( $changes, $request );
+ $this->assertSame(
+ '<!-- wp:tests/anchor-block {"metadata":{"ignoredHookedBlocks":["tests/hooked-block"]}} -->Hello<!-- /wp:tests/anchor-block -->',
+ $post->post_content,
+ 'The hooked block was not injected into the anchor block\'s ignoredHookedBlocks metadata.'
+ );
+ }
+
+ /**
+ * @ticket 60671
+ *
+ * @covers inject_ignored_hooked_blocks_metadata_attributes
+ */
+ public function test_inject_ignored_hooked_blocks_metadata_attributes_into_template_part() {
+ global $wp_current_filter;
+ // Mock currently set filter.
+ $wp_current_filter[] = 'rest_pre_insert_wp_template_part';
+
+ register_block_type(
+ 'tests/hooked-block',
+ array(
+ 'block_hooks' => array(
+ 'tests/anchor-block' => 'after',
+ ),
+ )
+ );
+
+ $id = self::TEST_THEME . '//' . 'my_template_part';
+ $request = new WP_REST_Request( 'POST', '/wp/v2/template-parts/' . $id );
+
+ $changes = new stdClass();
+ $changes->post_content = '<!-- wp:tests/anchor-block -->Hello<!-- /wp:tests/anchor-block -->';
+
+ $post = inject_ignored_hooked_blocks_metadata_attributes( $changes, $request );
+ $this->assertSame(
+ '<!-- wp:tests/anchor-block {"metadata":{"ignoredHookedBlocks":["tests/hooked-block"]}} -->Hello<!-- /wp:tests/anchor-block -->',
+ $post->post_content,
+ 'The hooked block was not injected into the anchor block\'s ignoredHookedBlocks metadata.'
+ );
+ }
</ins><span class="cx" style="display: block; padding: 0 10px"> }
</span></span></pre></div>
<a id="trunktestsphpunittestsrestapiwpRestTemplatesControllerphp"></a>
<div class="modfile"><h4 style="background-color: #eee; color: inherit; margin: 1em 0; padding: 1.3em; font-size: 115%">Modified: trunk/tests/phpunit/tests/rest-api/wpRestTemplatesController.php</h4>
<pre class="diff"><span>
<span class="info" style="display: block; padding: 0 10px; color: #888">--- trunk/tests/phpunit/tests/rest-api/wpRestTemplatesController.php 2024-03-07 07:06:53 UTC (rev 57789)
+++ trunk/tests/phpunit/tests/rest-api/wpRestTemplatesController.php 2024-03-07 14:10:31 UTC (rev 57790)
</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -50,6 +50,22 @@
</span><span class="cx" style="display: block; padding: 0 10px"> }
</span><span class="cx" style="display: block; padding: 0 10px">
</span><span class="cx" style="display: block; padding: 0 10px"> /**
</span><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+ * Tear down after each test.
+ *
+ * @since 6.5.0
+ */
+ public function tear_down() {
+ if ( has_filter( 'rest_pre_insert_wp_template_part', 'inject_ignored_hooked_blocks_metadata_attributes' ) ) {
+ remove_filter( 'rest_pre_insert_wp_template_part', 'inject_ignored_hooked_blocks_metadata_attributes', 10 );
+ }
+ if ( WP_Block_Type_Registry::get_instance()->is_registered( 'tests/block' ) ) {
+ unregister_block_type( 'tests/hooked-block' );
+ }
+
+ parent::tear_down();
+ }
+
+ /**
</ins><span class="cx" style="display: block; padding: 0 10px"> * @covers WP_REST_Templates_Controller::register_routes
</span><span class="cx" style="display: block; padding: 0 10px"> * @ticket 54596
</span><span class="cx" style="display: block; padding: 0 10px"> * @ticket 56467
</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -911,4 +927,44 @@
</span><span class="cx" style="display: block; padding: 0 10px">
</span><span class="cx" style="display: block; padding: 0 10px"> $this->assertEmpty( $prepared->post_content, 'The content was not correct in the prepared template part.' );
</span><span class="cx" style="display: block; padding: 0 10px"> }
</span><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+
+ /**
+ * @ticket 60671
+ *
+ * @covers WP_REST_Templates_Controller::prepare_item_for_database
+ * @covers inject_ignored_hooked_blocks_metadata_attributes
+ */
+ public function test_prepare_item_for_database_injects_hooked_block() {
+ register_block_type(
+ 'tests/hooked-block',
+ array(
+ 'block_hooks' => array(
+ 'tests/anchor-block' => 'after',
+ ),
+ )
+ );
+
+ add_filter( 'rest_pre_insert_wp_template_part', 'inject_ignored_hooked_blocks_metadata_attributes', 10, 2 );
+
+ $endpoint = new WP_REST_Templates_Controller( 'wp_template_part' );
+
+ $prepare_item_for_database = new ReflectionMethod( $endpoint, 'prepare_item_for_database' );
+ $prepare_item_for_database->setAccessible( true );
+
+ $body_params = array(
+ 'title' => 'Untitled Template Part',
+ 'slug' => 'untitled-template-part',
+ 'content' => '<!-- wp:tests/anchor-block -->Hello<!-- /wp:tests/anchor-block -->',
+ );
+
+ $request = new WP_REST_Request( 'POST', '/wp/v2/template-parts' );
+ $request->set_body_params( $body_params );
+
+ $prepared = $prepare_item_for_database->invoke( $endpoint, $request );
+ $this->assertSame(
+ '<!-- wp:tests/anchor-block {"metadata":{"ignoredHookedBlocks":["tests/hooked-block"]}} -->Hello<!-- /wp:tests/anchor-block -->',
+ $prepared->post_content,
+ 'The hooked block was not injected into the anchor block\'s ignoredHookedBlocks metadata.'
+ );
+ }
</ins><span class="cx" style="display: block; padding: 0 10px"> }
</span></span></pre>
</div>
</div>
</body>
</html>