0

I'm creating a short code that shows both first and last name of the user if he's logged in, or, if he's logged in, but hasn't configured any name, it will return a random name with a message telling him to edit his profile. I'm succeeding into getting the names, but not into showing the text that should appear in case the user has no name. The code:

// Show name if logged in - PT e ENG
function colaborador_nome($atts)
{
    if (is_user_logged_in() && !is_feed()) {
          return '&nbsp;' .    $username = '<a href="' . ( admin_url('profile.php') .'">' . get_user_meta(get_current_user_id(), 'first_name' , true) . '&nbsp;' .
        (($username = get_user_meta(get_current_user_id(), 'last_name', true)  . '</a>')  ? $username : '&nbsp;<a href="' . admin_url('profile.php') . '">Noe Name [edit profile]</a>"'));   
    }
}
add_shortcode('colaborador_nome', 'colaborador_nome');

The part that's not working:

 ? $username : '&nbsp;<a href="' . admin_url('profile.php') . '">Noe Name [edit profile]</a>"'

1 Answer 1

1

You absolutely did not understand how the code from my previous answer works, yeah? The PHP ternary operator $result = $a ? $b : $c works the same way as the following construction:

if ( $a ) {
    $result = $b;
} else {
    $result = $c;
}

So the whole idea was to check if the get_user_meta(get_current_user_id(), 'last_name', true) function returns an empty value. When you change it to

($username = get_user_meta(get_current_user_id(), 'last_name', true)  . '</a>') ? ...

you break the whole logic, because that

get_user_meta(get_current_user_id(), 'last_name', true)  . '</a>'

string would never have an empty value (that should be quite obvious, even if the get_user_meta() function returns an empty value, the whole string would be equal to '</a>'). Use this statement:

return '&nbsp;<a href="' . admin_url('profile.php') . '">' . ( ( $username = get_user_meta( get_current_user_id(), 'last_name', true ) ) ? $username : ">Noe Name [edit profile]" ) . '</a>';

Update

If you want to work with several user meta fields, I recommend to use get_userdata() function:

function colaborador_nome($atts)
{
    if (is_user_logged_in() && !is_feed()) {
        $userdata = get_userdata( get_current_user_id() );
        $username = trim( implode( ' ', array( $userdata->first_name, $userdata->last_name ) ) );
        return '&nbsp;<a href="' . admin_url('profile.php') . '">' . ( $username ? $username : ">Noe Name [edit profile]" ) . '</a>';
    }
}

I still don't understand why the shorthand form of ternary operator doesn't work for you, using it would help to simplify the return statement to

return '&nbsp;<a href="' . admin_url('profile.php') . '">' . ( $username ?: ">Noe Name [edit profile]" ) . '</a>';
2
  • I was already able to achieve results by returning only one name, but I'm not able to do the same when I use both first and lastname.
    – 023023
    Jul 9, 2020 at 7:25
  • @21321312 See the update. Jul 9, 2020 at 7:46

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Not the answer you're looking for? Browse other questions tagged or ask your own question.