Merge pull request #2199 from Zoddo/ticket/12275

[ticket/12275] Fix a bug on the event "core.modify_username_string"

* Zoddo/ticket/12275:
  [ticket/12275] Change if (empty) to if (!isset)
  [ticket/12275] Fix a bug on the event "core.modify_username_string"
This commit is contained in:
Joas Schilling 2014-05-24 14:18:43 +02:00
commit c7f51ca1a7

View file

@ -1348,7 +1348,8 @@ function get_username_string($mode, $user_id, $username, $username_colour = '',
// Return colour // Return colour
if ($mode == 'colour') if ($mode == 'colour')
{ {
return $username_colour; $username_string = $username_colour;
break;
} }
// no break; // no break;
@ -1368,7 +1369,8 @@ function get_username_string($mode, $user_id, $username, $username_colour = '',
// Return username // Return username
if ($mode == 'username') if ($mode == 'username')
{ {
return $username; $username_string = $username;
break;
} }
// no break; // no break;
@ -1389,12 +1391,15 @@ function get_username_string($mode, $user_id, $username, $username_colour = '',
// Return profile // Return profile
if ($mode == 'profile') if ($mode == 'profile')
{ {
return $profile_url; $username_string = $profile_url;
break;
} }
// no break; // no break;
} }
if (!isset($username_string))
{
if (($mode == 'full' && !$profile_url) || $mode == 'no_profile') if (($mode == 'full' && !$profile_url) || $mode == 'no_profile')
{ {
$username_string = str_replace(array('{USERNAME_COLOUR}', '{USERNAME}'), array($username_colour, $username), (!$username_colour) ? $_profile_cache['tpl_noprofile'] : $_profile_cache['tpl_noprofile_colour']); $username_string = str_replace(array('{USERNAME_COLOUR}', '{USERNAME}'), array($username_colour, $username), (!$username_colour) ? $_profile_cache['tpl_noprofile'] : $_profile_cache['tpl_noprofile_colour']);
@ -1403,6 +1408,7 @@ function get_username_string($mode, $user_id, $username, $username_colour = '',
{ {
$username_string = str_replace(array('{PROFILE_URL}', '{USERNAME_COLOUR}', '{USERNAME}'), array($profile_url, $username_colour, $username), (!$username_colour) ? $_profile_cache['tpl_profile'] : $_profile_cache['tpl_profile_colour']); $username_string = str_replace(array('{PROFILE_URL}', '{USERNAME_COLOUR}', '{USERNAME}'), array($profile_url, $username_colour, $username), (!$username_colour) ? $_profile_cache['tpl_profile'] : $_profile_cache['tpl_profile_colour']);
} }
}
/** /**
* Use this event to change the output of get_username_string() * Use this event to change the output of get_username_string()