Coding Standards: Fix a few newly introduced WPCS issues.

Follow-up to [56570], [56573], [56589], [56604], [56612], [56620], [56629], [56631], [56638], [56642], [56644], [56649].

Props jrf.
See #59161, #58831.
Built from https://develop.svn.wordpress.org/trunk@56680


git-svn-id: http://core.svn.wordpress.org/trunk@56192 1a063a9b-81f0-0310-95a4-ce76da25c4cd
This commit is contained in:
Sergey Biryukov 2023-09-25 15:36:18 +00:00
parent d9f4318a56
commit 28af9eafb0
8 changed files with 28 additions and 21 deletions

View File

@ -140,7 +140,7 @@ if ( ! is_multisite() ) {
<?php <?php
$new_admin_email = get_option( 'new_admin_email' ); $new_admin_email = get_option( 'new_admin_email' );
if ( $new_admin_email && get_option( 'admin_email' ) !== $new_admin_email ) { if ( $new_admin_email && get_option( 'admin_email' ) !== $new_admin_email ) {
$pending_admin_email_message = sprintf( $pending_admin_email_message = sprintf(
/* translators: %s: New admin email. */ /* translators: %s: New admin email. */
__( 'There is a pending change of the admin email to %s.' ), __( 'There is a pending change of the admin email to %s.' ),
'<code>' . esc_html( $new_admin_email ) . '</code>' '<code>' . esc_html( $new_admin_email ) . '</code>'

View File

@ -690,7 +690,7 @@ if ( isset( $_GET['error'] ) ) {
); );
} elseif ( isset( $_GET['deleted'] ) ) { } elseif ( isset( $_GET['deleted'] ) ) {
$delete_result = get_transient( 'plugins_delete_result_' . $user_ID ); $delete_result = get_transient( 'plugins_delete_result_' . $user_ID );
// Delete it once we're done. // Delete it once we're done.
delete_transient( 'plugins_delete_result_' . $user_ID ); delete_transient( 'plugins_delete_result_' . $user_ID );

View File

@ -550,7 +550,7 @@ switch ( $action ) {
$new_email = get_user_meta( $current_user->ID, '_new_email', true ); $new_email = get_user_meta( $current_user->ID, '_new_email', true );
if ( $new_email && $new_email['newemail'] !== $current_user->user_email && $profile_user->ID === $current_user->ID ) : if ( $new_email && $new_email['newemail'] !== $current_user->user_email && $profile_user->ID === $current_user->ID ) :
$pending_change_message = sprintf( $pending_change_message = sprintf(
/* translators: %s: New email. */ /* translators: %s: New email. */
__( 'There is a pending change of your email to %s.' ), __( 'There is a pending change of your email to %s.' ),
'<code>' . esc_html( $new_email['newemail'] ) . '</code>' '<code>' . esc_html( $new_email['newemail'] ) . '</code>'

View File

@ -98,7 +98,7 @@ function wp_render_elements_support( $block_content, $block ) {
foreach ( $element_config['paths'] as $path ) { foreach ( $element_config['paths'] as $path ) {
if ( null !== _wp_array_get( $block['attrs'], explode( '.', $path ), null ) ) { if ( null !== _wp_array_get( $block['attrs'], explode( '.', $path ), null ) ) {
$element_colors_set++; ++$element_colors_set;
} }
} }
} }

View File

@ -790,20 +790,20 @@ function make_before_block_visitor( $context ) {
* Furthermore, prepend the markup for any blocks hooked `before` the given block and as its parent's * Furthermore, prepend the markup for any blocks hooked `before` the given block and as its parent's
* `first_child`, respectively, to the serialized markup for the given block. * `first_child`, respectively, to the serialized markup for the given block.
* *
* @param array $block The block to inject the theme attribute into, and hooked blocks before. * @param array $block The block to inject the theme attribute into, and hooked blocks before.
* @param array $parent The parent block of the given block. * @param array $parent_block The parent block of the given block.
* @param array $prev The previous sibling block of the given block. * @param array $prev The previous sibling block of the given block.
* @return string The serialized markup for the given block, with the markup for any hooked blocks prepended to it. * @return string The serialized markup for the given block, with the markup for any hooked blocks prepended to it.
*/ */
return function( &$block, $parent = null, $prev = null ) use ( $context ) { return function ( &$block, $parent_block = null, $prev = null ) use ( $context ) {
_inject_theme_attribute_in_template_part_block( $block ); _inject_theme_attribute_in_template_part_block( $block );
$markup = ''; $markup = '';
if ( $parent && ! $prev ) { if ( $parent_block && ! $prev ) {
// Candidate for first-child insertion. // Candidate for first-child insertion.
$relative_position = 'first_child'; $relative_position = 'first_child';
$anchor_block_type = $parent['blockName']; $anchor_block_type = $parent_block['blockName'];
$hooked_block_types = array_keys( get_hooked_blocks( $anchor_block_type, $relative_position ) ); $hooked_block_types = array_keys( get_hooked_blocks( $anchor_block_type, $relative_position ) );
/** /**
* Filters the list of hooked block types for a given anchor block type and relative position. * Filters the list of hooked block types for a given anchor block type and relative position.
@ -856,12 +856,12 @@ function make_after_block_visitor( $context ) {
* Append the markup for any blocks hooked `after` the given block and as its parent's * Append the markup for any blocks hooked `after` the given block and as its parent's
* `last_child`, respectively, to the serialized markup for the given block. * `last_child`, respectively, to the serialized markup for the given block.
* *
* @param array $block The block to inject the hooked blocks after. * @param array $block The block to inject the hooked blocks after.
* @param array $parent The parent block of the given block. * @param array $parent_block The parent block of the given block.
* @param array $next The next sibling block of the given block. * @param array $next The next sibling block of the given block.
* @return string The serialized markup for the given block, with the markup for any hooked blocks appended to it. * @return string The serialized markup for the given block, with the markup for any hooked blocks appended to it.
*/ */
return function( &$block, $parent = null, $next = null ) use ( $context ) { return function ( &$block, $parent_block = null, $next = null ) use ( $context ) {
$markup = ''; $markup = '';
$relative_position = 'after'; $relative_position = 'after';
@ -873,10 +873,10 @@ function make_after_block_visitor( $context ) {
$markup .= get_comment_delimited_block_content( $hooked_block_type, array(), '' ); $markup .= get_comment_delimited_block_content( $hooked_block_type, array(), '' );
} }
if ( $parent && ! $next ) { if ( $parent_block && ! $next ) {
// Candidate for last-child insertion. // Candidate for last-child insertion.
$relative_position = 'last_child'; $relative_position = 'last_child';
$anchor_block_type = $parent['blockName']; $anchor_block_type = $parent_block['blockName'];
$hooked_block_types = array_keys( get_hooked_blocks( $anchor_block_type, $relative_position ) ); $hooked_block_types = array_keys( get_hooked_blocks( $anchor_block_type, $relative_position ) );
/** This filter is documented in wp-includes/blocks.php */ /** This filter is documented in wp-includes/blocks.php */
$hooked_block_types = apply_filters( 'hooked_block_types', $hooked_block_types, $relative_position, $anchor_block_type, $context ); $hooked_block_types = apply_filters( 'hooked_block_types', $hooked_block_types, $relative_position, $anchor_block_type, $context );
@ -1059,6 +1059,7 @@ function traverse_and_serialize_block( $block, $pre_callback = null, $post_callb
$prev = 0 === $block_index $prev = 0 === $block_index
? null ? null
: $block['innerBlocks'][ $block_index - 1 ]; : $block['innerBlocks'][ $block_index - 1 ];
$block_content .= call_user_func_array( $block_content .= call_user_func_array(
$pre_callback, $pre_callback,
array( &$inner_block, $block, $prev ) array( &$inner_block, $block, $prev )
@ -1071,12 +1072,13 @@ function traverse_and_serialize_block( $block, $pre_callback = null, $post_callb
$next = count( $block['innerBlocks'] ) - 1 === $block_index $next = count( $block['innerBlocks'] ) - 1 === $block_index
? null ? null
: $block['innerBlocks'][ $block_index + 1 ]; : $block['innerBlocks'][ $block_index + 1 ];
$block_content .= call_user_func_array( $block_content .= call_user_func_array(
$post_callback, $post_callback,
array( &$inner_block, $block, $next ) array( &$inner_block, $block, $next )
); );
} }
$block_index++; ++$block_index;
} }
} }
@ -1130,22 +1132,27 @@ function traverse_and_serialize_blocks( $blocks, $pre_callback = null, $post_cal
$prev = 0 === $index $prev = 0 === $index
? null ? null
: $blocks[ $index - 1 ]; : $blocks[ $index - 1 ];
$result .= call_user_func_array( $result .= call_user_func_array(
$pre_callback, $pre_callback,
array( &$block, null, $prev ) // At the top level, there is no parent block to pass to the callback. array( &$block, null, $prev ) // 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 .= traverse_and_serialize_block( $block, $pre_callback, $post_callback );
if ( is_callable( $post_callback ) ) { if ( is_callable( $post_callback ) ) {
$next = count( $blocks ) - 1 === $index $next = count( $blocks ) - 1 === $index
? null ? null
: $blocks[ $index + 1 ]; : $blocks[ $index + 1 ];
$result .= call_user_func_array( $result .= call_user_func_array(
$post_callback, $post_callback,
array( &$block, null, $next ) // At the top level, there is no parent block to pass to the callback. array( &$block, null, $next ) // At the top level, there is no parent block to pass to the callback.
); );
} }
} }
return $result; return $result;
} }

View File

@ -5730,7 +5730,7 @@ function wp_get_loading_optimization_attributes( $tag_name, $attr, $context ) {
if ( null === $maybe_in_viewport ) { if ( null === $maybe_in_viewport ) {
$header_enforced_contexts = array( $header_enforced_contexts = array(
'template_part_' . WP_TEMPLATE_PART_AREA_HEADER => true, 'template_part_' . WP_TEMPLATE_PART_AREA_HEADER => true,
'get_header_image_tag' => true, 'get_header_image_tag' => true,
); );
/** /**

View File

@ -231,8 +231,8 @@ function create_initial_taxonomies() {
'publicly_queryable' => false, 'publicly_queryable' => false,
'hierarchical' => false, 'hierarchical' => false,
'labels' => array( 'labels' => array(
'name' => _x( 'Pattern Categories', 'taxonomy general name' ), 'name' => _x( 'Pattern Categories', 'taxonomy general name' ),
'singular_name' => _x( 'Pattern Category', 'taxonomy singular name' ), 'singular_name' => _x( 'Pattern Category', 'taxonomy singular name' ),
), ),
'query_var' => false, 'query_var' => false,
'rewrite' => false, 'rewrite' => false,

View File

@ -16,7 +16,7 @@
* *
* @global string $wp_version * @global string $wp_version
*/ */
$wp_version = '6.4-alpha-56679'; $wp_version = '6.4-alpha-56680';
/** /**
* Holds the WordPress DB revision, increments when changes are made to the WordPress DB schema. * Holds the WordPress DB revision, increments when changes are made to the WordPress DB schema.