Changeset 56991

Timestamp:

10/23/2023 11:37:55 PM (15 months ago)

Author:

hellofromTonya

Message:

Editor: Fix render_duotone_support() to be compatible with enhanced pagination.

Some blocks do not have content. For duotone support, blocks without content still need to run through the render_duotone_support() to render their duotone CSS.

This fix makes the duotone compatible with the enhanced pagination (introduced in 6.4.0) by making sure that the CSS is always on the page, even when the posts have no featured image. It also prevents the duotone from interfering with other blocks using wp_unique_id().

References:

• ​Gutenberg PR 55415

Follow-up to [56226].

Props cbravobernal, luisherranz, hellofromTonya, isabel_brison, jorbin.
Fixes #59694.

Location:

trunk

Files:

• src/wp-includes/class-wp-duotone.php (modified) (1 diff)
• tests/phpunit/tests/block-supports/duotone.php (modified) (1 diff)

trunk/src/wp-includes/class-wp-duotone.php

@@ -1075 +1075 @@
      */
     public static function render_duotone_support( $block_content, $block, $wp_block ) {
-        if ( empty( $block_content ) || ! $block['blockName'] ) {
+        if ( ! $block['blockName'] ) {
             return $block_content;
         }

trunk/tests/phpunit/tests/block-supports/duotone.php

@@ -104 +104 @@
 
     /**
+     * Tests whether the CSS declarations are generated even if the block content is
+     * empty. This is needed to make the CSS output stable across paginations for
+     * features like the enhanced pagination of the Query block.
+     *
+     * @ticket 59694
+     *
+     * @covers ::render_duotone_support
+     */
+    public function test_css_declarations_are_generated_even_with_empty_block_content() {
+        $block                           = array(
+            'blockName' => 'core/image',
+            'attrs'     => array( 'style' => array( 'color' => array( 'duotone' => 'var:preset|duotone|blue-orange' ) ) ),
+        );
+        $wp_block                        = new WP_Block( $block );
+        $block_css_declarations_property = new ReflectionProperty( 'WP_Duotone', 'block_css_declarations' );
+        $block_css_declarations_property->setAccessible( true );
+        $block_css_declarations_property->setValue( $wp_block, array() );
+
+        WP_Duotone::render_duotone_support( '', $block, $wp_block );
+        $actual = $block_css_declarations_property->getValue();
+        // Reset the property's visibility.
+        $block_css_declarations_property->setAccessible( false );
+
+        $this->assertNotEmpty( $actual );
+    }
+
+    /**
      * @dataProvider data_is_preset
      */