Skip to content

Block Hooks: Allow traversal callbacks to modify parent block #5598

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
ockham wants to merge 7 commits into from
Closed

Block Hooks: Allow traversal callbacks to modify parent block #5598

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
6da854f
Block Hooks: Pass parent block by reference
ockham Oct 31, 2023
6f4d043
Pass reference to callback
ockham Oct 31, 2023
340c72b
Plural
ockham Oct 31, 2023
dd0f252
Pass as reference at callsite
ockham Oct 31, 2023
50a340a
Document that block and parent_block are passed by reference
ockham Oct 31, 2023
39c5993
Document that parent_block, prev, and next default to null
ockham Oct 31, 2023
7aa6c9c
Rename var from parent to parent_block
ockham Oct 31, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Pass reference to callback
  • Loading branch information
@ockham
ockham committed Oct 31, 2023
commit 6f4d0439a2a4da0a784ee34df93dad9c811ff8c6
5 changes: 3 additions & 2 deletions src/wp-includes/blocks.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1132,6 +1132,7 @@ function traverse_and_serialize_block( $block, $pre_callback = null, $post_callb
*/
function traverse_and_serialize_blocks( $blocks, $pre_callback = null, $post_callback = null ) {
$result = '';
$parent = null; // At the top level, there is no parent block to pass to the callback; yet the callback expects a reference.
foreach ( $blocks as $index => $block ) {
if ( is_callable( $pre_callback ) ) {
$prev = 0 === $index
Expand All @@ -1140,7 +1141,7 @@ function traverse_and_serialize_blocks( $blocks, $pre_callback = null, $post_cal

$result .= call_user_func_array(
$pre_callback,
array( &$block, null, $prev ) // At the top level, there is no parent block to pass to the callback.
array( &$block, $parent, $prev )
);
}

Expand All @@ -1151,7 +1152,7 @@ function traverse_and_serialize_blocks( $blocks, $pre_callback = null, $post_cal

$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.
array( &$block, $parent, $next )
);
}

Expand Down