Changeset 7335

Timestamp:

07/02/2025 02:52:12 PM (5 months ago)

Author:

johnjamesjacoby

Message:

Tools - Warnings/Notices: begin phasing out of utf8_encode() for PHP 8.2 and higher.

This change moves the existing utf8_encode() usages into a new common formatting function: bbp_format_user_display_name(), and uses this new function to abstract & encapsulate checking that mbstring equivalents are loaded & supported.

Props philipjohn.

In trunk, for 2.7.

Fixes #3585.

Location:

trunk/src/includes

Files:

• common/formatting.php (modified) (1 diff)
• core/filters.php (modified) (1 diff)
• replies/template.php (modified) (1 diff)
• topics/template.php (modified) (1 diff)

trunk/src/includes/common/formatting.php

@@ -783 +783 @@
     return $reason;
 }
+
+/** Users *********************************************************************/
+
+/**
+ * Format the display name of a user.
+ *
+ * Abstracts mbstring library check and fallback to utf8_encode().
+ *
+ * @link https://bbpress.trac.wordpress.org/ticket/2141
+ *
+ * @since 2.6.14
+ *
+ * @param string $display_name The author display
+ * @return string
+ */
+function bbp_format_user_display_name( $display_name = '' ) {
+
+    // Default return value
+    $retval = $display_name;
+
+    // Use the mbstring library if possible
+    if ( function_exists( 'mb_check_encoding' ) && ! mb_check_encoding( $display_name, 'UTF-8' ) ) {
+        $retval = mb_convert_encoding( $display_name, 'UTF-8', 'ISO-8859-1' );
+
+    // Fallback to function that (deprecated in PHP8.2)
+    } elseif ( seems_utf8( $display_name ) === false ) {
+        $retval = utf8_encode( $display_name );
+    }
+
+    // Return
+    return $retval;
+}

trunk/src/includes/core/filters.php

@@ -310 +310 @@
 
 // Topic and reply author display names
-add_filter( 'bbp_get_topic_author_display_name', 'wptexturize'   );
-add_filter( 'bbp_get_topic_author_display_name', 'convert_chars' );
-add_filter( 'bbp_get_topic_author_display_name', 'esc_html'      );
-add_filter( 'bbp_get_reply_author_display_name', 'wptexturize'   );
-add_filter( 'bbp_get_reply_author_display_name', 'convert_chars' );
-add_filter( 'bbp_get_reply_author_display_name', 'esc_html'      );
+add_filter( 'bbp_get_topic_author_display_name', 'bbp_format_user_display_name' );
+add_filter( 'bbp_get_topic_author_display_name', 'wptexturize'                  );
+add_filter( 'bbp_get_topic_author_display_name', 'convert_chars'                );
+add_filter( 'bbp_get_topic_author_display_name', 'esc_html'                     );
+add_filter( 'bbp_get_reply_author_display_name', 'bbp_format_user_display_name' );
+add_filter( 'bbp_get_reply_author_display_name', 'wptexturize'                  );
+add_filter( 'bbp_get_reply_author_display_name', 'convert_chars'                );
+add_filter( 'bbp_get_reply_author_display_name', 'esc_html'                     );
 
 /**

trunk/src/includes/replies/template.php

@@ -1100 +1100 @@
         }
 
-        // Encode possible UTF8 display names
-        if ( seems_utf8( $author_name ) === false ) {
-            $author_name = utf8_encode( $author_name );
-        }
-
         // Filter & return
         return apply_filters( 'bbp_get_reply_author_display_name', $author_name, $reply_id );

trunk/src/includes/topics/template.php

@@ -1336 +1336 @@
         }
 
-        // Encode possible UTF8 display names
-        if ( seems_utf8( $author_name ) === false ) {
-            $author_name = utf8_encode( $author_name );
-        }
-
         // Filter & return
         return apply_filters( 'bbp_get_topic_author_display_name', $author_name, $topic_id );