Changeset 6380 for trunk/src/includes/common/formatting.php

Timestamp:

03/17/2017 05:55:58 PM (8 years ago)

Author:

johnjamesjacoby

Message:

Mentions: Balance tags before making things clickable.

Then, make mentions clickable without "requiring" prepending whitespace. This allows other types of characters to be immediately before the @ control character.

See #2963.

File:

• trunk/src/includes/common/formatting.php (modified) (3 diffs)

trunk/src/includes/common/formatting.php

@@ -460 +460 @@
  */
 function bbp_make_mentions_clickable( $text = '' ) {
-    return preg_replace_callback( '#([\s>])@([0-9a-zA-Z-_]+)#i', 'bbp_make_mentions_clickable_callback', $text );
+    return preg_replace_callback( '#@([0-9a-zA-Z-_]+)#i', 'bbp_make_mentions_clickable_callback', $text );
 }
 
@@ -468 +468 @@
  * @since 2.6.0 (r6014)
  *
- * @param array $matches Single Regex Match.
- *
- * @return string HTML A tag with link to user profile.
+ * @param array $matches Regular expression matches in the current text blob.
+ *
+ * @return string Original text if no user exists, or link to user profile.
  */
 function bbp_make_mentions_clickable_callback( $matches = array() ) {
 
     // Get user; bail if not found
-    $user = get_user_by( 'slug', $matches[2] );
+    $user = get_user_by( 'slug', $matches[1] );
     if ( empty( $user ) || bbp_is_user_inactive( $user->ID ) ) {
         return $matches[0];
@@ -488 +488 @@
     // Escape & implode if not empty, otherwise an empty string
     $class_str = ! empty( $classes )
-        ? implode( ' ', array_map( 'esc_attr', $classes ) )
+        ? implode( ' ', array_map( 'sanitize_html_class', $classes ) )
         : '';
 
     // Create the link to the user's profile
     $url    = bbp_get_user_profile_url( $user->ID );
-    $clicky = '<a href="%1$s" class="' . $class_str . '">@%2$s</a>';
-    $anchor = sprintf( $clicky, esc_url( $url ), esc_html( $user->user_nicename ) );
+    $clicky = '<a href="%1$s" class="' . esc_attr( $class_str ) . '">%2$s</a>';
+    $anchor = sprintf( $clicky, esc_url( $url ), esc_html( $matches[0] ) );
     $link   = bbp_rel_nofollow( $anchor );
 
-    return $matches[1] . $link;
+    return $link;
 }