Changeset 13897

Timestamp:

06/02/2024 05:52:42 PM (11 months ago)

Author:

espellcaste

Message:

Use wp_strip_all_tags instead of strip_tags.

Props imath
Closes ​https://github.com/buddypress/buddypress/pull/306
See #7228

Location:

trunk/src

Files:

• bp-activity/bp-activity-functions.php (modified) (1 diff)
• bp-activity/bp-activity-template.php (modified) (3 diffs)
• bp-activity/classes/class-bp-activity-activity.php (modified) (1 diff)
• bp-activity/classes/class-bp-activity-feed.php (modified) (1 diff)
• bp-core/bp-core-template.php (modified) (4 diffs)
• bp-core/classes/class-bp-media-extractor.php (modified) (1 diff)
• bp-groups/screens/single/request-membership.php (modified) (1 diff)
• bp-members/bp-members-functions.php (modified) (1 diff)
• bp-members/bp-members-template.php (modified) (2 diffs)
• bp-messages/bp-messages-template.php (modified) (1 diff)
• bp-templates/bp-nouveau/includes/messages/ajax.php (modified) (1 diff)
• bp-xprofile/classes/class-bp-xprofile-field-type-url.php (modified) (1 diff)

trunk/src/bp-activity/bp-activity-functions.php

@@ -173 +173 @@
 
     // Try to find mentions.
-    $usernames = bp_activity_find_mentions( strip_tags( $activity->content ) );
+    $usernames = bp_activity_find_mentions( wp_strip_all_tags( $activity->content ) );
 
     // Still empty? Stop now.

trunk/src/bp-activity/bp-activity-template.php

@@ -3205 +3205 @@
          * @param int    $user_id ID of the queried user.
          */
-        $latest_update = apply_filters( 'bp_get_activity_latest_update_excerpt', trim( strip_tags( bp_create_excerpt( $update['content'], bp_activity_get_excerpt_length() ) ) ), $user_id );
+        $latest_update = apply_filters( 'bp_get_activity_latest_update_excerpt', wp_strip_all_tags( bp_create_excerpt( $update['content'], bp_activity_get_excerpt_length() ) ), $user_id );
 
         $latest_update = sprintf(
@@ -4196 +4196 @@
 
         $content = explode( '<span', $content );
-        $title   = strip_tags( ent2ncr( trim( convert_chars( $content[0] ) ) ) );
+        $title   = wp_strip_all_tags( ent2ncr( trim( convert_chars( $content[0] ) ) ) );
 
         if ( ':' === substr( $title, -1 ) ) {
@@ -4203 +4203 @@
 
         if ( 'activity_update' === $activities_template->activity->type ) {
-            $title .= ': ' . strip_tags( ent2ncr( trim( convert_chars( bp_create_excerpt( $activities_template->activity->content, 70, array( 'ending' => " [&#133;]" ) ) ) ) ) );
+            $title .= ': ' . wp_strip_all_tags( ent2ncr( trim( convert_chars( bp_create_excerpt( $activities_template->activity->content, 70, array( 'ending' => " [&#133;]" ) ) ) ) ) );
         }
 

trunk/src/bp-activity/classes/class-bp-activity-activity.php

@@ -1881 +1881 @@
         for ( $i = 0, $count = count( $activities ); $i < $count; ++$i ) {
             $title                            = explode( '<span', $activities[$i]['content'] );
-            $activity_feed[$i]['title']       = trim( strip_tags( $title[0] ) );
+            $activity_feed[$i]['title']       = wp_strip_all_tags( $title[0] );
             $activity_feed[$i]['link']        = $activities[$i]['primary_link'];
             $activity_feed[$i]['description'] = @sprintf( $activities[$i]['content'], '' );

trunk/src/bp-activity/classes/class-bp-activity-feed.php

@@ -201 +201 @@
     protected function setup_properties() {
         $this->id               = sanitize_title( $this->id );
-        $this->title            = strip_tags( $this->title );
+        $this->title            = wp_strip_all_tags( $this->title );
         $this->link             = esc_url_raw( $this->link );
-        $this->description      = strip_tags( $this->description );
+        $this->description      = wp_strip_all_tags( $this->description );
         $this->ttl              = (int) $this->ttl;
-        $this->update_period    = strip_tags( $this->update_period );
+        $this->update_period    = wp_strip_all_tags( $this->update_period );
         $this->update_frequency = (int) $this->update_frequency;
         $this->activity_args    = bp_parse_args(

trunk/src/bp-core/bp-core-template.php

@@ -817 +817 @@
         }
 
-        $totalLength = mb_strlen( strip_tags( $ending ) );
+        $totalLength = mb_strlen( wp_strip_all_tags( $ending ) );
         $openTags    = array();
         $truncate    = '';
@@ -867 +867 @@
         // Strip HTML tags if necessary.
         if ( ! empty( $r['strip_tags'] ) ) {
-            $text = strip_tags( $text );
+            $text = wp_strip_all_tags( $text );
         }
 
@@ -3059 +3059 @@
         // Use component name on member pages.
         } else {
-            $bp_title_parts = array_merge( $bp_title_parts, array_map( 'strip_tags', array(
+            $bp_title_parts = array_merge( $bp_title_parts, array_map( 'wp_strip_all_tags', array(
                 $displayed_user_name,
                 $component_name,
@@ -3066 +3066 @@
             // If we have a subnav name, add it separately for localization.
             if ( ! empty( $component_subnav_name ) ) {
-                $bp_title_parts[] = strip_tags( $component_subnav_name );
+                $bp_title_parts[] = wp_strip_all_tags( $component_subnav_name );
             }
         }

trunk/src/bp-core/classes/class-bp-media-extractor.php

@@ -933 +933 @@
      */
     protected function strip_markup( $richtext ) {
-        $plaintext = strip_shortcodes( html_entity_decode( strip_tags( $richtext ) ) );
+        $plaintext = strip_shortcodes( html_entity_decode( wp_strip_all_tags( $richtext ) ) );
 
         /**

trunk/src/bp-groups/screens/single/request-membership.php

@@ -52 +52 @@
         // If the member added a message to their request include it into the request arguments.
         if ( isset( $_POST['group-request-membership-comments'] ) && $_POST['group-request-membership-comments'] ) {
-            $request_args['content'] = strip_tags( wp_unslash( $_POST['group-request-membership-comments'] ) );
+            $request_args['content'] = wp_strip_all_tags( wp_unslash( $_POST['group-request-membership-comments'] ) );
         }
 

trunk/src/bp-members/bp-members-functions.php

@@ -515 +515 @@
     return apply_filters( 'bp_core_get_user_displayname', get_the_author_meta( 'display_name', $user_id ), $user_id );
 }
-add_filter( 'bp_core_get_user_displayname', 'strip_tags', 1 );
+add_filter( 'bp_core_get_user_displayname', 'wp_strip_all_tags', 1 );
 add_filter( 'bp_core_get_user_displayname', 'trim' );
 add_filter( 'bp_core_get_user_displayname', 'stripslashes' );

trunk/src/bp-members/bp-members-template.php

@@ -1013 +1013 @@
     add_filter( 'bp_get_member_name', 'wp_filter_kses' );
     add_filter( 'bp_get_member_name', 'stripslashes'   );
-    add_filter( 'bp_get_member_name', 'strip_tags'     );
+    add_filter( 'bp_get_member_name', 'wp_strip_all_tags' );
     add_filter( 'bp_get_member_name', 'esc_html'       );
 
@@ -1149 +1149 @@
 
         if ( ! empty( $update['content'] ) ) {
-            $excerpt           = trim( strip_tags( bp_create_excerpt( $update['content'], $length ) ) );
-            $update['content'] = trim( strip_tags( $update['content'] ) );
+            $excerpt           = wp_strip_all_tags( bp_create_excerpt( $update['content'], $length ) );
+            $update['content'] = wp_strip_all_tags( $update['content'] );
             $update['excerpt'] = $excerpt;
         } else {

trunk/src/bp-messages/bp-messages-template.php

@@ -206 +206 @@
          * @param string $value Excerpt of the current thread in the loop.
          */
-        return apply_filters( 'bp_get_message_thread_excerpt', strip_tags( bp_create_excerpt( $messages_template->thread->last_message_content, 75 ) ) );
+        return apply_filters( 'bp_get_message_thread_excerpt', wp_strip_all_tags( bp_create_excerpt( $messages_template->thread->last_message_content, 75 ) ) );
     }
 

trunk/src/bp-templates/bp-nouveau/includes/messages/ajax.php

@@ -503 +503 @@
         $thread->thread = array(
             'id'      => bp_get_the_thread_id(),
-            'subject' => strip_tags( bp_get_the_thread_subject() ),
+            'subject' => wp_strip_all_tags( bp_get_the_thread_subject() ),
         );
 

trunk/src/bp-xprofile/classes/class-bp-xprofile-field-type-url.php

@@ -188 +188 @@
      */
     public static function display_filter( $field_value, $field_id = '' ) {
-        $link      = strip_tags( $field_value );
+        $link      = wp_strip_all_tags( $field_value );
         $no_scheme = preg_replace( '#^https?://#', '', rtrim( $link, '/' ) );
         $url_text  = str_replace( $link, $no_scheme, $field_value );