mirror of
https://github.com/WordPress/WordPress.git
synced 2024-09-28 15:17:42 +02:00
Posts: Prevent an empty excerpt when groups and nested column blocks are present.
This improves the logic within excerpt_remove_blocks() to better handle innerBlocks. This prevents an empty excerpt from being returned when core/columns, core/column, and core/group blocks are present. This issue has been surfaced in the Query Loop block, where excerpts can be set to display. This introduces the excerpt_allowed_wrapper_blocks filter for controlling which blocks should be considered wrapper blocks. Wrapper blocks and their nested contents are not stripped by excerpt_remove_blocks(), allowing their contents to appear in generated excerpts. Backports [51348], [51375], [51378], and [51379] to the 5.8 branch. Fixes #53604. Props aristath, jorbin, SergeyBiryukov, desrosj. Unprops jorbin. Built from https://develop.svn.wordpress.org/branches/5.8@51382 git-svn-id: http://core.svn.wordpress.org/branches/5.8@50993 1a063a9b-81f0-0310-95a4-ce76da25c4cd
This commit is contained in:
parent
0ed55411af
commit
2360563683
@ -710,7 +710,23 @@ function excerpt_remove_blocks( $content ) {
|
||||
'core/verse',
|
||||
);
|
||||
|
||||
$allowed_blocks = array_merge( $allowed_inner_blocks, array( 'core/columns' ) );
|
||||
$allowed_wrapper_blocks = array(
|
||||
'core/columns',
|
||||
'core/column',
|
||||
'core/group',
|
||||
);
|
||||
|
||||
/**
|
||||
* Filters the list of blocks that can be used as wrapper blocks, allowing
|
||||
* excerpts to be generated from the `innerBlocks` of these wrappers.
|
||||
*
|
||||
* @since 5.8.0
|
||||
*
|
||||
* @param array $allowed_wrapper_blocks The list of allowed wrapper blocks.
|
||||
*/
|
||||
$allowed_wrapper_blocks = apply_filters( 'excerpt_allowed_wrapper_blocks', $allowed_wrapper_blocks );
|
||||
|
||||
$allowed_blocks = array_merge( $allowed_inner_blocks, $allowed_wrapper_blocks );
|
||||
|
||||
/**
|
||||
* Filters the list of blocks that can contribute to the excerpt.
|
||||
@ -729,8 +745,8 @@ function excerpt_remove_blocks( $content ) {
|
||||
foreach ( $blocks as $block ) {
|
||||
if ( in_array( $block['blockName'], $allowed_blocks, true ) ) {
|
||||
if ( ! empty( $block['innerBlocks'] ) ) {
|
||||
if ( 'core/columns' === $block['blockName'] ) {
|
||||
$output .= _excerpt_render_inner_columns_blocks( $block, $allowed_inner_blocks );
|
||||
if ( in_array( $block['blockName'], $allowed_wrapper_blocks, true ) ) {
|
||||
$output .= _excerpt_render_inner_blocks( $block, $allowed_blocks );
|
||||
continue;
|
||||
}
|
||||
|
||||
@ -753,23 +769,28 @@ function excerpt_remove_blocks( $content ) {
|
||||
}
|
||||
|
||||
/**
|
||||
* Render inner blocks from the `core/columns` block for generating an excerpt.
|
||||
* Render inner blocks from the allowed wrapper blocks
|
||||
* for generating an excerpt.
|
||||
*
|
||||
* @since 5.2.0
|
||||
* @since 5.8
|
||||
* @access private
|
||||
*
|
||||
* @param array $columns The parsed columns block.
|
||||
* @param array $parsed_block The parsed block.
|
||||
* @param array $allowed_blocks The list of allowed inner blocks.
|
||||
* @return string The rendered inner blocks.
|
||||
*/
|
||||
function _excerpt_render_inner_columns_blocks( $columns, $allowed_blocks ) {
|
||||
function _excerpt_render_inner_blocks( $parsed_block, $allowed_blocks ) {
|
||||
$output = '';
|
||||
|
||||
foreach ( $columns['innerBlocks'] as $column ) {
|
||||
foreach ( $column['innerBlocks'] as $inner_block ) {
|
||||
if ( in_array( $inner_block['blockName'], $allowed_blocks, true ) && empty( $inner_block['innerBlocks'] ) ) {
|
||||
$output .= render_block( $inner_block );
|
||||
}
|
||||
foreach ( $parsed_block['innerBlocks'] as $inner_block ) {
|
||||
if ( ! in_array( $inner_block['blockName'], $allowed_blocks, true ) ) {
|
||||
continue;
|
||||
}
|
||||
|
||||
if ( empty( $inner_block['innerBlocks'] ) ) {
|
||||
$output .= render_block( $inner_block );
|
||||
} else {
|
||||
$output .= _excerpt_render_inner_blocks( $inner_block, $allowed_blocks );
|
||||
}
|
||||
}
|
||||
|
||||
|
@ -4207,3 +4207,20 @@ function wp_sensitive_page_meta() {
|
||||
<?php
|
||||
wp_strict_cross_origin_referrer();
|
||||
}
|
||||
|
||||
/**
|
||||
* Render inner blocks from the `core/columns` block for generating an excerpt.
|
||||
*
|
||||
* @since 5.2.0
|
||||
* @deprecated 5.8.0
|
||||
*
|
||||
* @access private
|
||||
*
|
||||
* @param array $columns The parsed columns block.
|
||||
* @param array $allowed_blocks The list of allowed inner blocks.
|
||||
* @return string The rendered inner blocks.
|
||||
*/
|
||||
function _excerpt_render_inner_columns_blocks( $columns, $allowed_blocks ) {
|
||||
_deprecated_function( __FUNCTION__, '5.8.0', '_excerpt_render_inner_blocks()' );
|
||||
return _excerpt_render_inner_blocks( $columns, $allowed_blocks );
|
||||
}
|
||||
|
@ -13,7 +13,7 @@
|
||||
*
|
||||
* @global string $wp_version
|
||||
*/
|
||||
$wp_version = '5.8-RC2-51376';
|
||||
$wp_version = '5.8-RC2-51382';
|
||||
|
||||
/**
|
||||
* Holds the WordPress DB revision, increments when changes are made to the WordPress DB schema.
|
||||
|
Loading…
Reference in New Issue
Block a user