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

Timestamp:

09/12/2023 07:18:34 PM (3 years 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' ) );
+        }
 }