Changeset 56560 for trunk/tests/phpunit/tests/formatting/wpTrimExcerpt.php

Timestamp:

09/12/2023 07:18:34 PM (12 months ago)

Author:

flixos90

Message:

Posts, Post Types: Avoid unnecessarily parsing blocks twice in wp_trim_excerpt().

All blocks relevant for the excerpt are already being parsed in excerpt_remove_blocks(). Therefore running do_blocks() on the post content only to create the excerpt is unnecessary and wasteful from a performance perspective.

Props thekt12, spacedmonkey, mukesh27, joemcgill.
Fixes #58682.

File:

• trunk/tests/phpunit/tests/formatting/wpTrimExcerpt.php (modified) (1 diff)

trunk/tests/phpunit/tests/formatting/wpTrimExcerpt.php

@@ -149 +149 @@
         $this->assertFalse( has_filter( 'the_content', 'wp_filter_content_tags' ) );
     }
+
+    /**
+     * Tests that `wp_trim_excerpt()` does process valid blocks.
+     *
+     * @ticket 58682
+     *
+     * @covers ::wp_trim_excerpt
+     */
+    public function test_wp_trim_excerpt_check_if_block_renders() {
+        $post = self::factory()->post->create(
+            array(
+                'post_content' => '<!-- wp:paragraph --> <p>A test paragraph</p> <!-- /wp:paragraph -->',
+            )
+        );
+
+        $output_text = wp_trim_excerpt( '', $post );
+
+        $this->assertSame( 'A test paragraph', $output_text, 'wp_trim_excerpt() did not process paragraph block.' );
+    }
+
+    /**
+     * Tests that `wp_trim_excerpt()` unhooks `do_blocks()` from 'the_content' filter.
+     *
+     * @ticket 58682
+     *
+     * @covers ::wp_trim_excerpt
+     */
+    public function test_wp_trim_excerpt_unhooks_do_blocks() {
+        $post = self::factory()->post->create();
+
+        /*
+         * Record that during 'the_content' filter run by wp_trim_excerpt() the
+         * do_blocks() callback is not used.
+         */
+        $has_filter = true;
+        add_filter(
+            'the_content',
+            static function( $content ) use ( &$has_filter ) {
+                $has_filter = has_filter( 'the_content', 'do_blocks' );
+                return $content;
+            }
+        );
+
+        wp_trim_excerpt( '', $post );
+
+        $this->assertFalse( $has_filter, 'do_blocks() was not unhooked in wp_trim_excerpt()' );
+    }
+
+    /**
+     * Tests that `wp_trim_excerpt()` doesn't permanently unhook `do_blocks()` from 'the_content' filter.
+     *
+     * @ticket 58682
+     *
+     * @covers ::wp_trim_excerpt
+     */
+    public function test_wp_trim_excerpt_should_not_permanently_unhook_do_blocks() {
+        $post = self::factory()->post->create();
+
+        wp_trim_excerpt( '', $post );
+
+        $this->assertSame( 9, has_filter( 'the_content', 'do_blocks' ), 'do_blocks() was not restored in wp_trim_excerpt()' );
+    }
+
+    /**
+     * Tests that `wp_trim_excerpt()` doesn't restore `do_blocks()` if it was previously unhooked.
+     *
+     * @ticket 58682
+     *
+     * @covers ::wp_trim_excerpt
+     */
+    public function test_wp_trim_excerpt_does_not_restore_do_blocks_if_previously_unhooked() {
+        $post = self::factory()->post->create();
+
+        // Remove do_blocks() from 'the_content' filter generally.
+        remove_filter( 'the_content', 'do_blocks', 9 );
+
+        wp_trim_excerpt( '', $post );
+
+        // Assert that the filter callback was not restored after running 'the_content'.
+        $this->assertFalse( has_filter( 'the_content', 'do_blocks' ) );
+    }
 }