Changeset 44226 for trunk/tests/phpunit/tests/blocks/render.php

Timestamp:

12/16/2018 02:22:43 AM (8 years ago)

Author:

jeremyfelt

Message:

Formatting: Ensure wpautop() isn't run on content generated from blocks.

As do_blocks() is run before wpautop() in the_content filter, we can remove in a Just In Time fashion, before that filter is run.

After wpautop()s original priority has passed, we can re-add it in a Just Too Late fashion, to ensure it's available if the_content filter is run multiple times on a page load.

Merges [43879] and [43881] from the 5.0 branch to trunk.

Props pento, nerrad.
Fixes #45290.

Location:

trunk

Files:

• . (modified) (1 prop)
• tests/phpunit/tests/blocks/render.php (modified) (1 diff)

trunk/tests/phpunit/tests/blocks/render.php

@@ -76 +76 @@
                 $block_filtered_content = preg_replace( "/\n{2,}/", "\n", $block_filtered_content );
 
-                $this->assertEquals( $classic_filtered_content, $block_filtered_content );
+                $this->assertEquals( trim( $classic_filtered_content ), trim( $block_filtered_content ) );
         }
 
         function handle_shortcode( $atts, $content ) {
                 return $content;
+        }
+
+        /**
+         * @ticket 45290
+         */
+        public function test_blocks_arent_autopeed() {
+                $expected_content = 'test';
+                $test_content     = "<!-- wp:fake/block -->\n$expected_content\n<!-- /wp:fake/block -->";
+
+                $current_priority = has_action( 'the_content', 'wpautop' );
+
+                $filtered_content = trim( apply_filters( 'the_content', $test_content ) );
+
+                $this->assertEquals( $expected_content, $filtered_content );
+
+                // Check that wpautop() is still defined in the same place.
+                $this->assertSame( $current_priority, has_action( 'the_content', 'wpautop' ) );
+                // ... and that the restore function has removed itself.
+                $this->assertFalse( has_action( 'the_content', '_restore_wpautop_hook' ) );
+
+                $test_content     = 'test';
+                $expected_content = "<p>$test_content</p>";
+
+                $current_priority = has_action( 'the_content', 'wpautop' );
+
+                $filtered_content = trim( apply_filters( 'the_content', $test_content ) );
+
+                $this->assertEquals( $expected_content, $filtered_content );
+
+                $this->assertSame( $current_priority, has_action( 'the_content', 'wpautop' ) );
+                $this->assertFalse( has_action( 'the_content', '_restore_wpautop_hook' ) );
         }