Changeset 41086

Timestamp:

07/18/2017 10:10:41 PM (8 years ago)

Author:

westonruter

Message:

Widgets: Replace adding balanceTags on widget_custom_html_content filter in favor of just applying widget_text filters in the Custom HTML widget.

Ensures that users who copy HTML from the Text widget in legacy mode over to the Custom HTML widget will continue to get all of the same filters applied, including tag balancing and shortcodes, if a plugin added support. Plugins still have the widget_text_content and widget_custom_html_content filters they can use to target the specific widget types.

Amends [40893].
See #40951.
Fixes #40907 for trunk.

Location:

trunk

Files:

• src/wp-includes/default-filters.php (modified) (1 diff)
• src/wp-includes/widgets/class-wp-widget-custom-html.php (modified) (1 diff)
• src/wp-includes/widgets/class-wp-widget-text.php (modified) (1 diff)
• tests/phpunit/tests/widgets/custom-html-widget.php (modified) (4 diffs)

trunk/src/wp-includes/default-filters.php

@@ -170 +170 @@
 add_filter( 'widget_text_content', 'convert_smilies',  20 );
 add_filter( 'widget_text_content', 'wpautop'              );
-
-add_filter( 'widget_custom_html_content', 'balanceTags' );
 
 add_filter( 'date_i18n', 'wp_maybe_decline_date' );

trunk/src/wp-includes/widgets/class-wp-widget-custom-html.php

@@ -62 +62 @@
         $title = apply_filters( 'widget_title', $instance['title'], $instance, $this->id_base );
 
-        $content = $instance['content'];
+        /** This filter is documented in wp-includes/widgets/class-wp-widget-text.php */
+        $content = apply_filters( 'widget_text', $instance['content'], $instance, $this );
 
         /**

trunk/src/wp-includes/widgets/class-wp-widget-text.php

@@ -214 +214 @@
          * @since 2.3.0
          * @since 4.4.0 Added the `$this` parameter.
+         * @since 4.8.1 The `$this` param may now be a `WP_Widget_Custom_HTML` object in addition to a `WP_Widget_Text` object.
          *
-         * @param string         $text     The widget content.
-         * @param array          $instance Array of settings for the current widget.
-         * @param WP_Widget_Text $this     Current Text widget instance.
+         * @param string                               $text     The widget content.
+         * @param array                                $instance Array of settings for the current widget.
+         * @param WP_Widget_Text|WP_Widget_Custom_HTML $this     Current Text widget instance.
          */
         $text = apply_filters( 'widget_text', $text, $instance, $this );

trunk/tests/phpunit/tests/widgets/custom-html-widget.php

@@ -20 +20 @@
      */
     protected $widget_custom_html_content_args;
+
+    /**
+     * Args passed to the widget_text filter.
+     *
+     * @var array
+     */
+    protected $widget_text_args;
 
     /**
@@ -53 +60 @@
         );
 
-        $this->assertEquals( 10, has_filter( 'widget_custom_html_content', 'balanceTags' ) );
-
         update_option( 'use_balanceTags', 0 );
         add_filter( 'widget_custom_html_content', array( $this, 'filter_widget_custom_html_content' ), 5, 3 );
+        add_filter( 'widget_text', array( $this, 'filter_widget_text' ), 10, 3 );
         ob_start();
         $this->widget_custom_html_content_args = null;
+        $this->widget_text_args = null;
         $widget->widget( $args, $instance );
         $output = ob_get_clean();
         $this->assertNotEmpty( $this->widget_custom_html_content_args );
-        $this->assertContains( '[filter:widget_custom_html_content]', $output );
+        $this->assertNotEmpty( $this->widget_text_args );
+        $this->assertContains( '[filter:widget_text][filter:widget_custom_html_content]', $output );
         $this->assertNotContains( '<p>', $output );
         $this->assertNotContains( '<br>', $output );
         $this->assertNotContains( '</u>', $output );
+        $this->assertEquals( $instance, $this->widget_text_args[1] );
         $this->assertEquals( $instance, $this->widget_custom_html_content_args[1] );
+        $this->assertSame( $widget, $this->widget_text_args[2] );
         $this->assertSame( $widget, $this->widget_custom_html_content_args[2] );
         remove_filter( 'widget_custom_html_content', array( $this, 'filter_widget_custom_html_content' ), 5, 3 );
+        remove_filter( 'widget_text', array( $this, 'filter_widget_text' ), 10 );
 
         update_option( 'use_balanceTags', 1 );
@@ -78 +89 @@
 
     /**
-     * Filters the content of the Custom HTML widget.
+     * Filters the content of the Custom HTML widget using the legacy widget_text filter.
+     *
+     * @param string                $text     The widget content.
+     * @param array                 $instance Array of settings for the current widget.
+     * @param WP_Widget_Custom_HTML $widget   Current widget instance.
+     * @return string Widget content.
+     */
+    function filter_widget_text( $text, $instance, $widget ) {
+        $this->widget_text_args = array( $text, $instance, $widget );
+        $text .= '[filter:widget_text]';
+        return $text;
+    }
+
+    /**
+     * Filters the content of the Custom HTML widget using the dedicated widget_custom_html_content filter.
      *
      * @param string                $widget_content The widget content.
@@ -86 +111 @@
      */
     function filter_widget_custom_html_content( $widget_content, $instance, $widget ) {
-        $this->widget_custom_html_content_args = func_get_args();
-
+        $this->widget_custom_html_content_args = array( $widget_content, $instance, $widget );
         $widget_content .= '[filter:widget_custom_html_content]';
         return $widget_content;