Changeset 56970 for trunk/src/wp-includes/blocks.php

Timestamp:

10/18/2023 07:30:40 PM (15 months ago)

Author:

Bernhard Reiter

Message:

Blocks: During traversal, allow post callback to modify block.

Both the $pre_callback and $post_callback functions that are given as arguments to traverse_and_serialize_block(s) receive a reference to the current block as their first argument. However, while any changes that the "pre" callback makes to the block are reflected by the serialized markup, the same wasn't true for the "post" callback: Any changes that it made were only applied after the block had already been serialized.

This commit changes the behavior such that $post_callback's changes to the current block are also reflected in the serialized markup.

See #59646.
Props gziolo.
Fixes #59669.

File:

• trunk/src/wp-includes/blocks.php (modified) (4 diffs)

trunk/src/wp-includes/blocks.php

@@ -1063 +1063 @@
             }
 
-            $block_content .= traverse_and_serialize_block( $inner_block, $pre_callback, $post_callback );
-
             if ( is_callable( $post_callback ) ) {
                 $next = count( $block['innerBlocks'] ) - 1 === $block_index
@@ -1070 +1068 @@
                     : $block['innerBlocks'][ $block_index + 1 ];
 
-                $block_content .= call_user_func_array(
+                $post_markup = call_user_func_array(
                     $post_callback,
                     array( &$inner_block, $block, $next )
                 );
             }
+
+            $block_content .= traverse_and_serialize_block( $inner_block, $pre_callback, $post_callback );
+            $block_content .= isset( $post_markup ) ? $post_markup : '';
+
             ++$block_index;
         }
@@ -1136 +1138 @@
         }
 
-        $result .= traverse_and_serialize_block( $block, $pre_callback, $post_callback );
-
         if ( is_callable( $post_callback ) ) {
             $next = count( $blocks ) - 1 === $index
@@ -1143 +1143 @@
                 : $blocks[ $index + 1 ];
 
-            $result .= call_user_func_array(
+            $post_markup = call_user_func_array(
                 $post_callback,
                 array( &$block, null, $next ) // At the top level, there is no parent block to pass to the callback.
             );
         }
+
+        $result .= traverse_and_serialize_block( $block, $pre_callback, $post_callback );
+        $result .= isset( $post_markup ) ? $post_markup : '';
     }