Changeset 41361 for trunk/tests/phpunit/tests/widgets/text-widget.php

Timestamp:

09/10/2017 06:32:34 AM (8 years ago)

Author:

westonruter

Message:

Widgets: Add shortcode support inside Text widgets.

• Used now in core to facilitate displaying inserted media. See #40854.
• The [embed] shortcode is not supported because there is no post context for caching oEmbed responses. This depends on #34115.
• Add do_shortcode() to the widget_text_content filter in the same way it is added for the_content at priority 11, with shortcode_unautop() called at priority 10 after wpautop().
• For Text widget in legacy mode, manually apply do_shortcode() (and shortcode_unautop() if auto-paragraph checked) if the core-added widget_text_content filter remains, unless a plugin added do_shortcode() to widget_text to prevent applying shortcodes twice.
• Ensure that global $post is null while filters apply in the Text widget so shortcode handlers won't run with unexpected contexts.

Props westonruter, nacin, aaroncampbell.
See #40854, #34115.
Fixes #10457.

File:

• trunk/tests/phpunit/tests/widgets/text-widget.php (modified) (3 diffs)

trunk/tests/phpunit/tests/widgets/text-widget.php

@@ -223 +223 @@
      * @var string
      */
-    protected $example_shortcode_content = "<p>One\nTwo\n\nThree</p>\n<script>\ndocument.write('Test1');\n\ndocument.write('Test2');\n</script>";
+    protected $example_shortcode_content = "<p class='sortcodep'>One\nTwo\n\nThree\n\nThis is testing the <code>[example note='This will not get processed since it is part of shortcode output itself.']</code> shortcode.</p>\n<script>\ndocument.write('Test1');\n\ndocument.write('Test2');\n</script>";
+
+    /**
+     * The captured global post during shortcode rendering.
+     *
+     * @var WP_Post|null
+     */
+    protected $post_during_shortcode = null;
+
+    /**
+     * Number of times the shortcode was rendered.
+     *
+     * @var int
+     */
+    protected $shortcode_render_count = 0;
 
     /**
@@ -231 +245 @@
      */
     function do_example_shortcode() {
+        $this->post_during_shortcode = get_post();
+        $this->shortcode_render_count++;
         return $this->example_shortcode_content;
     }
 
     /**
-     * Test widget method when a plugin has added shortcode support.
+     * Test widget method with shortcodes.
      *
      * @covers WP_Widget_Text::widget
      */
     function test_widget_shortcodes() {
+        global $post;
+        $post_id = $this->factory()->post->create();
+        $post = get_post( $post_id );
+
         $args = array(
             'before_title'  => '<h2>',
@@ -247 +267 @@
         );
         $widget = new WP_Widget_Text();
-        add_filter( 'widget_text', 'do_shortcode' );
         add_shortcode( 'example', array( $this, 'do_example_shortcode' ) );
 
         $base_instance = array(
             'title' => 'Example',
-            'text' => "This is an example:\n\n[example]",
-            'filter' => false,
-        );
-
-        // Legacy Text Widget.
-        $instance = array_merge( $base_instance, array(
-            'filter' => false,
-        ) );
-        ob_start();
-        $widget->widget( $args, $instance );
-        $output = ob_get_clean();
+            'text' => "This is an example:\n\n[example]\n\nHello.",
+            'filter' => false,
+        );
+
+        // Legacy Text Widget without wpautop.
+        $instance = array_merge( $base_instance, array(
+            'filter' => false,
+        ) );
+        $this->shortcode_render_count = 0;
+        ob_start();
+        $widget->widget( $args, $instance );
+        $output = ob_get_clean();
+        $this->assertEquals( 1, $this->shortcode_render_count );
+        $this->assertNotContains( '[example]', $output, 'Expected shortcode to be processed in legacy widget with plugin adding filter' );
         $this->assertContains( $this->example_shortcode_content, $output, 'Shortcode was applied without wpautop corrupting it.' );
-        $this->assertEquals( 10, has_filter( 'widget_text', 'do_shortcode' ), 'Filter was restored.' );
-
-        // Visual Text Widget.
-        $instance = array_merge( $base_instance, array(
-            'filter' => 'content',
-        ) );
-        ob_start();
-        $widget->widget( $args, $instance );
-        $output = ob_get_clean();
+        $this->assertNotContains( '<p>' . $this->example_shortcode_content . '</p>', $output, 'Expected shortcode_unautop() to have run.' );
+        $this->assertNull( $this->post_during_shortcode );
+
+        // Legacy Text Widget with wpautop.
+        $instance = array_merge( $base_instance, array(
+            'filter' => true,
+            'visual' => false,
+        ) );
+        $this->shortcode_render_count = 0;
+        ob_start();
+        $widget->widget( $args, $instance );
+        $output = ob_get_clean();
+        $this->assertEquals( 1, $this->shortcode_render_count );
+        $this->assertNotContains( '[example]', $output, 'Expected shortcode to be processed in legacy widget with plugin adding filter' );
         $this->assertContains( $this->example_shortcode_content, $output, 'Shortcode was applied without wpautop corrupting it.' );
-        $this->assertEquals( 10, has_filter( 'widget_text', 'do_shortcode' ), 'Filter was restored.' );
-        $this->assertFalse( has_filter( 'widget_text_content', 'do_shortcode' ), 'Filter was removed.' );
-
-        // Visual Text Widget with properly-used widget_text_content filter.
+        $this->assertNotContains( '<p>' . $this->example_shortcode_content . '</p>', $output, 'Expected shortcode_unautop() to have run.' );
+        $this->assertNull( $this->post_during_shortcode );
+
+        // Legacy text widget with plugin adding shortcode support as well.
+        add_filter( 'widget_text', 'do_shortcode' );
+        $this->shortcode_render_count = 0;
+        ob_start();
+        $widget->widget( $args, $instance );
+        $output = ob_get_clean();
+        $this->assertEquals( 1, $this->shortcode_render_count );
+        $this->assertNotContains( '[example]', $output, 'Expected shortcode to be processed in legacy widget with plugin adding filter' );
+        $this->assertContains( wpautop( $this->example_shortcode_content ), $output, 'Shortcode was applied *with* wpautop() applying to shortcode output since plugin used legacy filter.' );
+        $this->assertNull( $this->post_during_shortcode );
         remove_filter( 'widget_text', 'do_shortcode' );
-        add_filter( 'widget_text_content', 'do_shortcode', 11 );
-        $instance = array_merge( $base_instance, array(
-            'filter' => 'content',
-        ) );
-        ob_start();
-        $widget->widget( $args, $instance );
-        $output = ob_get_clean();
+
+        $instance = array_merge( $base_instance, array(
+            'filter' => true,
+            'visual' => true,
+        ) );
+
+        // Visual Text Widget with only core-added widget_text_content filter for do_shortcode.
+        $this->assertFalse( has_filter( 'widget_text', 'do_shortcode' ) );
+        $this->assertEquals( 11, has_filter( 'widget_text_content', 'do_shortcode' ), 'Expected core to have set do_shortcode as widget_text_content filter.' );
+        $this->shortcode_render_count = 0;
+        ob_start();
+        $widget->widget( $args, $instance );
+        $output = ob_get_clean();
+        $this->assertEquals( 1, $this->shortcode_render_count );
         $this->assertContains( $this->example_shortcode_content, $output, 'Shortcode was applied without wpautop corrupting it.' );
-        $this->assertFalse( has_filter( 'widget_text', 'do_shortcode' ), 'Filter was not erroneously restored.' );
+        $this->assertNotContains( '<p>' . $this->example_shortcode_content . '</p>', $output, 'Expected shortcode_unautop() to have run.' );
+        $this->assertFalse( has_filter( 'widget_text', 'do_shortcode' ), 'The widget_text filter still lacks do_shortcode handler.' );
+        $this->assertEquals( 11, has_filter( 'widget_text_content', 'do_shortcode' ), 'The widget_text_content filter still has do_shortcode handler.' );
+        $this->assertNull( $this->post_during_shortcode );
+
+        // Visual Text Widget with both filters applied added, one from core and another via plugin.
+        add_filter( 'widget_text', 'do_shortcode' );
+        $this->shortcode_render_count = 0;
+        ob_start();
+        $widget->widget( $args, $instance );
+        $output = ob_get_clean();
+        $this->assertEquals( 1, $this->shortcode_render_count );
+        $this->assertContains( $this->example_shortcode_content, $output, 'Shortcode was applied without wpautop corrupting it.' );
+        $this->assertNotContains( '<p>' . $this->example_shortcode_content . '</p>', $output, 'Expected shortcode_unautop() to have run.' );
+        $this->assertEquals( 10, has_filter( 'widget_text', 'do_shortcode' ), 'Expected do_shortcode to be restored to widget_text.' );
+        $this->assertNull( $this->post_during_shortcode );
+        $this->assertNull( $this->post_during_shortcode );
+        remove_filter( 'widget_text', 'do_shortcode' );
+
+        // Visual Text Widget with shortcode handling disabled via plugin removing filter.
+        remove_filter( 'widget_text_content', 'do_shortcode', 11 );
+        remove_filter( 'widget_text', 'do_shortcode' );
+        $this->shortcode_render_count = 0;
+        ob_start();
+        $widget->widget( $args, $instance );
+        $output = ob_get_clean();
+        $this->assertEquals( 0, $this->shortcode_render_count );
+        $this->assertContains( '[example]', $output );
+        $this->assertNotContains( $this->example_shortcode_content, $output );
+        $this->assertFalse( has_filter( 'widget_text', 'do_shortcode' ) );
+        $this->assertFalse( has_filter( 'widget_text_content', 'do_shortcode' ) );
     }