Changeset 7213 for branches/2.6/src/includes/common/template.php

Timestamp:

10/12/2021 10:05:17 PM (3 years ago)

Author:

johnjamesjacoby

Message:

Users: add some error checking to get_userdata() calls in profile link functions.

This change ensures that when attempting to link to user profiles when the display_name is the inner HTML of the link, that the "Anonymous" alternate is used whenever the user cannot be found.

It also bails early – returning false – if the user account simply does not exist, consistent with upstream calls to bbp_get_user_id(). (Functions calling these functions have a responsibility to appropriately handle boolean return values.)

In branches/2.6, for 2.6.7.

See #3440.

File:

• branches/2.6/src/includes/common/template.php (modified) (3 diffs)

branches/2.6/src/includes/common/template.php

@@ -2152 +2152 @@
  * @return bool True if match, false if not
  */
-function bbp_is_query_name( $name = '' )  {
+function bbp_is_query_name( $name = '' ) {
     return (bool) ( bbp_get_query_name() === $name );
 }
@@ -2163 +2163 @@
  * @return string To return the query var value
  */
-function bbp_get_query_name()  {
+function bbp_get_query_name() {
     return get_query_var( '_bbp_query_name' );
 }
@@ -2174 +2174 @@
  * @param string $name What to set the query var to
  */
-function bbp_set_query_name( $name = '' )  {
+function bbp_set_query_name( $name = '' ) {
     set_query_var( '_bbp_query_name', $name );
 }