<!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>[49694] branches/5.6: Editor: Remove render_block hooks from WP_Block</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/49694">49694</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/49694","name":"Review Commit"}}</script></dd>
<dt style="float: left; width: 6em; font-weight: bold">Author</dt> <dd>noisysocks</dd>
<dt style="float: left; width: 6em; font-weight: bold">Date</dt> <dd>2020-11-25 01:13:09 +0000 (Wed, 25 Nov 2020)</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'>Editor: Remove render_block hooks from WP_Block

Reverts the move of pre_render_block, render_block_data, and
render_block_context to WP_Block.

This change has more implications than first thought so will be revisted later
in 5.7.

Reverts [49609,49608].
See <a href="https://core.trac.wordpress.org/ticket/51612">#51612</a>.</pre>

<h3>Modified Paths</h3>
<ul>
<li><a href="#branches56srcwpincludesblocksphp">branches/5.6/src/wp-includes/blocks.php</a></li>
<li><a href="#branches56srcwpincludesclasswpblockphp">branches/5.6/src/wp-includes/class-wp-block.php</a></li>
<li><a href="#branches56testsphpunittestsblocksblockphp">branches/5.6/tests/phpunit/tests/blocks/block.php</a></li>
</ul>

</div>
<div id="patch">
<h3>Diff</h3>
<a id="branches56srcwpincludesblocksphp"></a>
<div class="modfile"><h4 style="background-color: #eee; color: inherit; margin: 1em 0; padding: 1.3em; font-size: 115%">Modified: branches/5.6/src/wp-includes/blocks.php</h4>
<pre class="diff"><span>
<span class="info" style="display: block; padding: 0 10px; color: #888">--- branches/5.6/src/wp-includes/blocks.php   2020-11-24 21:25:46 UTC (rev 49693)
+++ branches/5.6/src/wp-includes/blocks.php     2020-11-25 01:13:09 UTC (rev 49694)
</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -662,8 +662,7 @@
</span><span class="cx" style="display: block; padding: 0 10px">        global $post, $wp_query;
</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">-         * Allows render_block() or WP_Block::render() to be short-circuited, by
-        * returning a non-null value.
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * Allows render_block() to be short-circuited, by returning a non-null value.
</ins><span class="cx" style="display: block; padding: 0 10px">          *
</span><span class="cx" style="display: block; padding: 0 10px">         * @since 5.1.0
</span><span class="cx" style="display: block; padding: 0 10px">         *
</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -675,6 +674,18 @@
</span><span class="cx" style="display: block; padding: 0 10px">                return $pre_render;
</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">+        $source_block = $parsed_block;
+
+       /**
+        * Filters the block being rendered in render_block(), before it's processed.
+        *
+        * @since 5.1.0
+        *
+        * @param array $parsed_block The block being rendered.
+        * @param array $source_block An un-modified copy of $parsed_block, as it appeared in the source content.
+        */
+       $parsed_block = apply_filters( 'render_block_data', $parsed_block, $source_block );
+
</ins><span class="cx" style="display: block; padding: 0 10px">         $context = array();
</span><span class="cx" style="display: block; padding: 0 10px"> 
</span><span class="cx" style="display: block; padding: 0 10px">        if ( $post instanceof WP_Post ) {
</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -696,6 +707,16 @@
</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">+        /**
+        * Filters the default context provided to a rendered block.
+        *
+        * @since 5.5.0
+        *
+        * @param array $context      Default context.
+        * @param array $parsed_block Block being rendered, filtered by `render_block_data`.
+        */
+       $context = apply_filters( 'render_block_context', $context, $parsed_block );
+
</ins><span class="cx" style="display: block; padding: 0 10px">         $block = new WP_Block( $parsed_block, $context );
</span><span class="cx" style="display: block; padding: 0 10px"> 
</span><span class="cx" style="display: block; padding: 0 10px">        return $block->render();
</span></span></pre></div>
<a id="branches56srcwpincludesclasswpblockphp"></a>
<div class="modfile"><h4 style="background-color: #eee; color: inherit; margin: 1em 0; padding: 1.3em; font-size: 115%">Modified: branches/5.6/src/wp-includes/class-wp-block.php</h4>
<pre class="diff"><span>
<span class="info" style="display: block; padding: 0 10px; color: #888">--- branches/5.6/src/wp-includes/class-wp-block.php   2020-11-24 21:25:46 UTC (rev 49693)
+++ branches/5.6/src/wp-includes/class-wp-block.php     2020-11-25 01:13:09 UTC (rev 49694)
</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -11,10 +11,6 @@
</span><span class="cx" style="display: block; padding: 0 10px">  *
</span><span class="cx" style="display: block; padding: 0 10px">  * @since 5.5.0
</span><span class="cx" style="display: block; padding: 0 10px">  * @property array $attributes
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">- * @property array $context
- * @property WP_Block[] $inner_blocks;
- * @property string $inner_html;
- * @property array $inner_content;
</del><span class="cx" style="display: block; padding: 0 10px">  */
</span><span class="cx" style="display: block; padding: 0 10px"> class WP_Block {
</span><span class="cx" style="display: block; padding: 0 10px"> 
</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -27,15 +23,6 @@
</span><span class="cx" style="display: block; padding: 0 10px">        public $parsed_block;
</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">-         * All available context of the current hierarchy.
-        *
-        * @since 5.5.0
-        * @var array
-        * @access protected
-        */
-       protected $available_context;
-
-       /**
</del><span class="cx" style="display: block; padding: 0 10px">          * Name of block.
</span><span class="cx" style="display: block; padding: 0 10px">         *
</span><span class="cx" style="display: block; padding: 0 10px">         * @example "core/paragraph"
</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -54,147 +41,93 @@
</span><span class="cx" style="display: block; padding: 0 10px">        public $block_type;
</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">-         * Map of block property names and their cached value.
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * Block context values.
</ins><span class="cx" style="display: block; padding: 0 10px">          *
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         * Some block properties are computed lazily using a getter function. The
-        * result is then cached here for subsequent use.
-        *
-        * @since 5.6.0
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * @since 5.5.0
</ins><span class="cx" style="display: block; padding: 0 10px">          * @var array
</span><span class="cx" style="display: block; padding: 0 10px">         */
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-        protected $cached_properties = array();
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+ public $context = array();
</ins><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">-         * Creates a block instance from a backing `$parsed_block` array and list of
-        * `$available_context`. From these, the block's dynamic properties can be
-        * derived.
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * All available context of the current hierarchy.
</ins><span class="cx" style="display: block; padding: 0 10px">          *
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         * The given array of context values will not necessarily be available on
-        * the instance itself, but is treated as the full set of values provided by
-        * the block's ancestry. This is assigned to the private `available_context`
-        * property. Only values which are configured to consumed by the block via
-        * its registered type will be assigned to the block's `context` property.
-        *
</del><span class="cx" style="display: block; padding: 0 10px">          * @since 5.5.0
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         *
-        * @param array                  $parsed_block      Array of parsed block properties.
-        * @param array                  $available_context Optional array of ancestry context values.
-        * @param WP_Block_Type_Registry $registry          Optional block type registry.
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * @var array
+        * @access protected
</ins><span class="cx" style="display: block; padding: 0 10px">          */
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-        public function __construct( $parsed_block, $available_context = array(), $registry = null ) {
-               if ( is_null( $registry ) ) {
-                       $this->registry = WP_Block_Type_Registry::get_instance();
-               } else {
-                       $this->registry = $registry;
-               }
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+ protected $available_context;
</ins><span class="cx" style="display: block; padding: 0 10px"> 
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-                $this->reset( $parsed_block, $available_context );
-       }
-
</del><span class="cx" style="display: block; padding: 0 10px">         /**
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         * Changes the backing `$parsed_block` and `$available_context` used to
-        * derive the block's dynamic properties.
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * List of inner blocks (of this same class)
</ins><span class="cx" style="display: block; padding: 0 10px">          *
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         * @since 5.6.0
-
-        * @param array $parsed_block      Array of parsed block properties.
-        * @param array $available_context Optional array of ancestry context values.
-        * @param array $cached_properties Optional cache of dynamic properties to use.
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * @since 5.5.0
+        * @var WP_Block[]
</ins><span class="cx" style="display: block; padding: 0 10px">          */
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-        protected function reset(
-               $parsed_block,
-               $available_context = array(),
-               $cached_properties = array()
-       ) {
-               $this->parsed_block      = $parsed_block;
-               $this->available_context = $available_context;
-               $this->name              = $parsed_block['blockName'];
-               $this->block_type        = $this->registry->get_registered( $this->name );
-               $this->cached_properties = $cached_properties;
-       }
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+ public $inner_blocks = array();
</ins><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">-         * Getter used for the block's dynamic properties:
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * Resultant HTML from inside block comment delimiters after removing inner
+        * blocks.
</ins><span class="cx" style="display: block; padding: 0 10px">          *
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         * - `$block->attributes`
-        * - `$block->context`
-        * - `$block->inner_blocks`
-        * - `$block->inner_html`
-        * - `$block->inner_content`
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * @example "...Just <!-- wp:test /--> testing..." -> "Just testing..."
</ins><span class="cx" style="display: block; padding: 0 10px">          *
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         * Each dynamic property is obtained by calling the associated getter
-        * function (e.g. `this->get_attributes()`). The result is then cached in
-        * `$this->cached_attributes` for subsequent calls.
-        *
</del><span class="cx" style="display: block; padding: 0 10px">          * @since 5.5.0
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         *
-        * @param string $name Property name.
-        * @return array|null Prepared attributes, or null.
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * @var string
</ins><span class="cx" style="display: block; padding: 0 10px">          */
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-        public function __get( $name ) {
-               if ( method_exists( $this, "get_$name" ) ) {
-                       if ( ! isset( $this->cached_properties[ $name ] ) ) {
-                               $this->cached_properties[ $name ] = $this->{"get_$name"}();
-                       }
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+ public $inner_html = '';
</ins><span class="cx" style="display: block; padding: 0 10px"> 
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-                        return $this->cached_properties[ $name ];
-               }
-
-               return null;
-       }
-
</del><span class="cx" style="display: block; padding: 0 10px">         /**
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         * Block attributes.
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * List of string fragments and null markers where inner blocks were found
</ins><span class="cx" style="display: block; padding: 0 10px">          *
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         * Use `$block->attributes` to access this.
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * @example array(
+        *   'inner_html'    => 'BeforeInnerAfter',
+        *   'inner_blocks'  => array( block, block ),
+        *   'inner_content' => array( 'Before', null, 'Inner', null, 'After' ),
+        * )
</ins><span class="cx" style="display: block; padding: 0 10px">          *
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         * @since 5.6.0
-        * @return array
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * @since 5.5.0
+        * @var array
</ins><span class="cx" style="display: block; padding: 0 10px">          */
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-        protected function get_attributes() {
-               $attributes = isset( $this->parsed_block['attrs'] ) ?
-                       $this->parsed_block['attrs'] :
-                       array();
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+ public $inner_content = array();
</ins><span class="cx" style="display: block; padding: 0 10px"> 
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-                if ( ! is_null( $this->block_type ) ) {
-                       return $this->block_type->prepare_attributes_for_render( $attributes );
-               }
-
-               return $attributes;
-       }
-
</del><span class="cx" style="display: block; padding: 0 10px">         /**
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         * Block context values.
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * Constructor.
</ins><span class="cx" style="display: block; padding: 0 10px">          *
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         * Use `$block->context` to access this.
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * Populates object properties from the provided block instance argument.
</ins><span class="cx" style="display: block; padding: 0 10px">          *
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         * @since 5.6.0
-        * @return array
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * The given array of context values will not necessarily be available on
+        * the instance itself, but is treated as the full set of values provided by
+        * the block's ancestry. This is assigned to the private `available_context`
+        * property. Only values which are configured to consumed by the block via
+        * its registered type will be assigned to the block's `context` property.
+        *
+        * @since 5.5.0
+        *
+        * @param array                  $block             Array of parsed block properties.
+        * @param array                  $available_context Optional array of ancestry context values.
+        * @param WP_Block_Type_Registry $registry          Optional block type registry.
</ins><span class="cx" style="display: block; padding: 0 10px">          */
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-        protected function get_context() {
-               $context = array();
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+ public function __construct( $block, $available_context = array(), $registry = null ) {
+               $this->parsed_block = $block;
+               $this->name         = $block['blockName'];
</ins><span class="cx" style="display: block; padding: 0 10px"> 
</span><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+                if ( is_null( $registry ) ) {
+                       $registry = WP_Block_Type_Registry::get_instance();
+               }
+
+               $this->block_type = $registry->get_registered( $this->name );
+
+               $this->available_context = $available_context;
+
</ins><span class="cx" style="display: block; padding: 0 10px">                 if ( ! empty( $this->block_type->uses_context ) ) {
</span><span class="cx" style="display: block; padding: 0 10px">                        foreach ( $this->block_type->uses_context as $context_name ) {
</span><span class="cx" style="display: block; padding: 0 10px">                                if ( array_key_exists( $context_name, $this->available_context ) ) {
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-                                        $context[ $context_name ] = $this->available_context[ $context_name ];
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+                                 $this->context[ $context_name ] = $this->available_context[ $context_name ];
</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 class="cx" style="display: block; padding: 0 10px"> 
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-                return $context;
-       }
-
-       /**
-        * List of inner blocks (of this same class).
-        *
-        * Use `$block->inner_blocks` to access this.
-        *
-        * @since 5.6.0
-        * @return WP_Block[]
-        */
-       protected function get_inner_blocks() {
-               if ( ! empty( $this->parsed_block['innerBlocks'] ) ) {
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+         if ( ! empty( $block['innerBlocks'] ) ) {
</ins><span class="cx" style="display: block; padding: 0 10px">                         $child_context = $this->available_context;
</span><span class="cx" style="display: block; padding: 0 10px"> 
</span><span class="cx" style="display: block; padding: 0 10px">                        if ( ! empty( $this->block_type->provides_context ) ) {
</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -205,55 +138,45 @@
</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><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-                        return new WP_Block_List(
-                               $this->parsed_block['innerBlocks'],
-                               $child_context,
-                               $this->registry
-                       );
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+                 $this->inner_blocks = new WP_Block_List( $block['innerBlocks'], $child_context, $registry );
</ins><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 array();
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+         if ( ! empty( $block['innerHTML'] ) ) {
+                       $this->inner_html = $block['innerHTML'];
+               }
+
+               if ( ! empty( $block['innerContent'] ) ) {
+                       $this->inner_content = $block['innerContent'];
+               }
</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><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         * Resultant HTML from inside block comment delimiters after removing inner
-        * blocks.
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * Returns a value from an inaccessible property.
</ins><span class="cx" style="display: block; padding: 0 10px">          *
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         * Use `$block->inner_html` to access this.
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * This is used to lazily initialize the `attributes` property of a block,
+        * such that it is only prepared with default attributes at the time that
+        * the property is accessed. For all other inaccessible properties, a `null`
+        * value is returned.
</ins><span class="cx" style="display: block; padding: 0 10px">          *
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         * @example "...Just <!-- wp:test /--> testing..." -> "Just testing..."
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * @since 5.5.0
</ins><span class="cx" style="display: block; padding: 0 10px">          *
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-         * @since 5.6.0
-        * @return string
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+  * @param string $name Property name.
+        * @return array|null Prepared attributes, or null.
</ins><span class="cx" style="display: block; padding: 0 10px">          */
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-        protected function get_inner_html() {
-               if ( ! empty( $this->parsed_block['innerHTML'] ) ) {
-                       return $this->parsed_block['innerHTML'];
-               }
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+ public function __get( $name ) {
+               if ( 'attributes' === $name ) {
+                       $this->attributes = isset( $this->parsed_block['attrs'] ) ?
+                               $this->parsed_block['attrs'] :
+                               array();
</ins><span class="cx" style="display: block; padding: 0 10px"> 
</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">+                 if ( ! is_null( $this->block_type ) ) {
+                               $this->attributes = $this->block_type->prepare_attributes_for_render( $this->attributes );
+                       }
</ins><span class="cx" style="display: block; padding: 0 10px"> 
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-        /**
-        * List of string fragments and null markers where inner blocks were found
-        *
-        * Use `$block->inner_content` to access this.
-        *
-        * @example array(
-        *   'inner_html'    => 'BeforeInnerAfter',
-        *   'inner_blocks'  => array( block, block ),
-        *   'inner_content' => array( 'Before', null, 'Inner', null, 'After' ),
-        * )
-        *
-        * @since 5.6.0
-        * @return array
-        */
-       protected function get_inner_content() {
-               if ( ! empty( $this->parsed_block['innerContent'] ) ) {
-                       return $this->parsed_block['innerContent'];
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+                 return $this->attributes;
</ins><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 array();
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+         return null;
</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 class="lines" style="display: block; padding: 0 10px; color: #888">@@ -270,13 +193,6 @@
</span><span class="cx" style="display: block; padding: 0 10px">         */
</span><span class="cx" style="display: block; padding: 0 10px">        public function render( $options = array() ) {
</span><span class="cx" style="display: block; padding: 0 10px">                global $post;
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-
-               /** This filter is documented in wp-includes/blocks.php */
-               $pre_render = apply_filters( 'pre_render_block', null, $this->parsed_block );
-               if ( ! is_null( $pre_render ) ) {
-                       return $pre_render;
-               }
-
</del><span class="cx" style="display: block; padding: 0 10px">                 $options = wp_parse_args(
</span><span class="cx" style="display: block; padding: 0 10px">                        $options,
</span><span class="cx" style="display: block; padding: 0 10px">                        array(
</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -284,47 +200,7 @@
</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><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-                $initial_parsed_block      = $this->parsed_block;
-               $initial_available_context = $this->available_context;
-               $initial_cached_properties = $this->cached_properties;
-
-               /**
-                * Filters a block which is to be rendered by render_block() or
-                * WP_Block::render().
-                *
-                * @since 5.1.0
-                *
-                * @param array $parsed_block The block being rendered.
-                * @param array $source_block An un-modified copy of $parsed_block, as it appeared in the source content.
-                */
-               $parsed_block = apply_filters(
-                       'render_block_data',
-                       $this->parsed_block,
-                       $initial_parsed_block
-               );
-
-               /**
-                * Filters the default context of a block which is to be rendered by
-                * render_block() or WP_Block::render().
-                *
-                * @since 5.5.0
-                *
-                * @param array $available_context Default context.
-                * @param array $parsed_block      Block being rendered, filtered by `render_block_data`.
-                */
-               $available_context = apply_filters(
-                       'render_block_context',
-                       $this->available_context,
-                       $this->parsed_block
-               );
-
-               $this->reset( $parsed_block, $available_context );
-
-               $is_dynamic = $options['dynamic']
-                       && $this->name
-                       && null !== $this->block_type
-                       && $this->block_type->is_dynamic();
-
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+         $is_dynamic    = $options['dynamic'] && $this->name && null !== $this->block_type && $this->block_type->is_dynamic();
</ins><span class="cx" style="display: block; padding: 0 10px">                 $block_content = '';
</span><span class="cx" style="display: block; padding: 0 10px"> 
</span><span class="cx" style="display: block; padding: 0 10px">                if ( ! $options['dynamic'] || empty( $this->block_type->skip_inner_blocks ) ) {
</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -342,12 +218,7 @@
</span><span class="cx" style="display: block; padding: 0 10px"> 
</span><span class="cx" style="display: block; padding: 0 10px">                        WP_Block_Supports::$block_to_render = $this->parsed_block;
</span><span class="cx" style="display: block; padding: 0 10px"> 
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-                        $block_content = (string) call_user_func(
-                               $this->block_type->render_callback,
-                               $this->attributes,
-                               $block_content,
-                               $this
-                       );
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+                 $block_content = (string) call_user_func( $this->block_type->render_callback, $this->attributes, $block_content, $this );
</ins><span class="cx" style="display: block; padding: 0 10px"> 
</span><span class="cx" style="display: block; padding: 0 10px">                        WP_Block_Supports::$block_to_render = $parent;
</span><span class="cx" style="display: block; padding: 0 10px"> 
</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -370,15 +241,7 @@
</span><span class="cx" style="display: block; padding: 0 10px">                 * @param string $block_content The block content about to be appended.
</span><span class="cx" style="display: block; padding: 0 10px">                 * @param array  $block         The full block, including name and attributes.
</span><span class="cx" style="display: block; padding: 0 10px">                 */
</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-                $block_content = apply_filters( 'render_block', $block_content, $this->parsed_block );
-
-               $this->reset(
-                       $initial_parsed_block,
-                       $initial_available_context,
-                       $initial_cached_properties
-               );
-
-               return $block_content;
</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+         return apply_filters( 'render_block', $block_content, $this->parsed_block );
</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="branches56testsphpunittestsblocksblockphp"></a>
<div class="modfile"><h4 style="background-color: #eee; color: inherit; margin: 1em 0; padding: 1.3em; font-size: 115%">Modified: branches/5.6/tests/phpunit/tests/blocks/block.php</h4>
<pre class="diff"><span>
<span class="info" style="display: block; padding: 0 10px; color: #888">--- branches/5.6/tests/phpunit/tests/blocks/block.php 2020-11-24 21:25:46 UTC (rev 49693)
+++ branches/5.6/tests/phpunit/tests/blocks/block.php   2020-11-25 01:13:09 UTC (rev 49694)
</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -45,23 +45,6 @@
</span><span class="cx" style="display: block; padding: 0 10px">                return 'Original: "' . $content . '", from block "' . $parsed_block['blockName'] . '"';
</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">-        function filter_pre_render_block( $pre_render, $parsed_block ) {
-               if ( 'core/skip' === $parsed_block['blockName'] ) {
-                       return 'Hello world!';
-               }
-               return null;
-       }
-
-       function filter_render_block_data( $parsed_block, $source_block ) {
-               $parsed_block['attrs']['tag'] = $parsed_block['attrs']['tag'] . '-filtered';
-               return $parsed_block;
-       }
-
-       function filter_render_block_context( $available_context, $parsed_block ) {
-               $available_context['core/recordId'] += 1;
-               return $available_context;
-       }
-
</del><span class="cx" style="display: block; padding: 0 10px">         /**
</span><span class="cx" style="display: block; padding: 0 10px">         * @ticket 49927
</span><span class="cx" style="display: block; padding: 0 10px">         */
</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -411,127 +394,4 @@
</span><span class="cx" style="display: block; padding: 0 10px">                $this->assertSame( 'abc', $block->render() );
</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">-        /**
-        * @ticket 51612
-        */
-       function test_applies_pre_render_block_filter() {
-               $this->registry->register( 'core/skip', array() );
-
-               add_filter( 'pre_render_block', array( $this, 'filter_pre_render_block' ), 10, 2 );
-
-               $parsed_blocks = parse_blocks( '<!-- wp:skip /-->' );
-               $parsed_block  = $parsed_blocks[0];
-               $context       = array();
-               $block         = new WP_Block( $parsed_block, $context, $this->registry );
-
-               $rendered_content = $block->render();
-
-               remove_filter( 'pre_render_block', array( $this, 'filter_pre_render_block' ) );
-
-               $this->assertSame( 'Hello world!', $rendered_content );
-       }
-
-       /**
-        * @ticket 51612
-        */
-       function test_applies_pre_render_block_filter_to_inner_blocks() {
-               $this->registry->register( 'core/outer', array() );
-               $this->registry->register( 'core/skip', array() );
-
-               add_filter( 'pre_render_block', array( $this, 'filter_pre_render_block' ), 10, 2 );
-
-               $parsed_blocks = parse_blocks( '<!-- wp:outer --><!-- wp:skip /--> How are you?<!-- /wp:outer -->' );
-               $parsed_block  = $parsed_blocks[0];
-               $context       = array();
-               $block         = new WP_Block( $parsed_block, $context, $this->registry );
-
-               $rendered_content = $block->render();
-
-               remove_filter( 'pre_render_block', array( $this, 'filter_pre_render_block' ) );
-
-               $this->assertSame( 'Hello world! How are you?', $rendered_content );
-       }
-
-       /**
-        * @ticket 51612
-        */
-       function test_applies_render_block_data_filter() {
-               $this->registry->register(
-                       'core/wrapper',
-                       array(
-                               'attributes'      => array(
-                                       'tag' => array(
-                                               'type' => 'string',
-                                       ),
-                               ),
-                               'render_callback' => function( $block_attributes, $content ) {
-                                       return sprintf(
-                                               '<%1$s>%2$s</%1$s>',
-                                               $block_attributes['tag'],
-                                               $content
-                                       );
-                               },
-                       )
-               );
-
-               add_filter( 'render_block_data', array( $this, 'filter_render_block_data' ), 10, 2 );
-
-               $parsed_blocks = parse_blocks( '<!-- wp:wrapper {"tag":"outer"} --><!-- wp:wrapper {"tag":"inner"} -->Hello!<!-- /wp:wrapper --><!-- /wp:wrapper -->' );
-               $parsed_block  = $parsed_blocks[0];
-               $context       = array();
-               $block         = new WP_Block( $parsed_block, $context, $this->registry );
-
-               $rendered_content = $block->render();
-
-               remove_filter( 'render_block_data', array( $this, 'filter_render_block_data' ) );
-
-               $this->assertSame( '<outer-filtered><inner-filtered>Hello!</inner-filtered></outer-filtered>', $rendered_content );
-       }
-
-       /**
-        * @ticket 51612
-        */
-       function test_applies_render_block_context_filter() {
-               $this->registry->register(
-                       'core/provider',
-                       array(
-                               'attributes'       => array(
-                                       'recordId' => array(
-                                               'type' => 'number',
-                                       ),
-                               ),
-                               'uses_context'     => array( 'core/recordId' ),
-                               'provides_context' => array(
-                                       'core/recordId' => 'recordId',
-                               ),
-                       )
-               );
-               $this->registry->register(
-                       'core/consumer',
-                       array(
-                               'uses_context'    => array( 'core/recordId' ),
-                               'render_callback' => function( $block_attributes, $content, $block ) {
-                                       return sprintf( 'Record ID: %d ', $block->context['core/recordId'] );
-                               },
-                       )
-               );
-
-               add_filter( 'render_block_context', array( $this, 'filter_render_block_context' ), 10, 2 );
-
-               $parsed_blocks = parse_blocks( '<!-- wp:consumer /--><!-- wp:provider {"recordId":20} --><!-- wp:consumer /--><!-- /wp:provider -->' );
-               $context       = array( 'core/recordId' => 10 );
-
-               $rendered_content = '';
-
-               foreach ( $parsed_blocks as $parsed_block ) {
-                       $block = new WP_Block( $parsed_block, $context, $this->registry );
-
-                       $rendered_content .= $block->render();
-               }
-
-               remove_filter( 'render_block_context', array( $this, 'filter_render_block_context' ) );
-
-               $this->assertSame( 'Record ID: 11 Record ID: 21 ', $rendered_content );
-       }
-
</del><span class="cx" style="display: block; padding: 0 10px"> }
</span></span></pre>
</div>
</div>

</body>
</html>