Login redirect not working

[andreu]

Hi,

I’m trying to redirect the users to a custom page for each user after login. I’m trying to do this with the login_redirect filter this way:

function my_redirect( $to, $requested, $user ){
    if( !isset( $user->user_login ) ){ // we only want this to run when credentials have been supplied
        return $to;
    }
    return get_custom_user_url($user->ID);
}
add_filter('login_redirect', 'my_redirect', 10, 3);

I can’t make the filter work. I’ve tried with normal URLs and it’s not working either. It seems that th filter is not running.

Do you know how can I make it work?

Thanks

[Support]

Hello,

By default filter “login_redirect” are disabled for avoiding plugin conflict. To enable this please use dev version.

Dev Version (login to download)

download

and put following code to functions.php to enable the filter

function allow_login_redirect_hook( $isEnable, $hookName ) {
    if ( 'login_redirect' == $hookName )
        $isEnable = true;
    
    return $isEnable;
}
add_filter( 'user_meta_wp_hook', 'allow_login_redirect_hook', 10, 2 );

Thanks.

[andreu]

The redirection is working fine now. However, I can’t get the user ID. I’ve tried with $user->ID without succes. Also, I’ve tried with this, but it’s not working neither:

global $current_user;
get_currentuserinfo();
$user_id == $current_user->ID;

Thanks!

[Support]

Hi,

Can you get $user->user_login ?

[andreu]

No, I’ve tried to get $user->ID and $user->user_login without success.

[Khaled]

Could you please post your whole code here.

Thanks.

[andreu]

I want to return a page with the user ID as a GET var. This is my code:

function allow_login_redirect_hook( $isEnable, $hookName ) {
    if ( 'login_redirect' == $hookName )
        $isEnable = true;
    
    return $isEnable;
}
add_filter( 'user_meta_wp_hook', 'allow_login_redirect_hook', 10, 2 );

add_filter( 'login_redirect', 'redirectClient' );
function redirectClient( $redirect_to, $redirect, $user ){
	return get_permalink(24).'?user_id='.$user->ID; 
}

Thanks

[Khaled]

$user suppose to return WP_User or WP_Error object. So for success login $user->ID should work. If you have a look on file user-meta/models/pro/umSupportProModel.php line (300-325), you can see $user = wp_signon() and this $user object are feed to login_redirect hook. So it should work. However, is it possible to create a temp admin account for us to check the issue on your site?
Thanks.

[andreu]

Thanks Khaled, I could make work the $user var.

However, is it possible that get_term_link function doesn’t work fine inside the filter? I can return a hardcoded url, but if I try to get a term link a WP Error appears and admin-ajax.php doesn’t complete the request.

I’ve been trying alternatives during more than an hour without success 🙁

Thank you

[Khaled]

Hello,

get_term_link() function should work too. As you got WP Error, it means the function is calling and returns WP_error object. get_term_link() returns error when it doesn’t find the term. Please make sure you have provided correct arguments into get_term_link.

“admin-ajax.php doesn’t complete the request.”: “login_redirect” expects string (url) not object, and there are no other error handling after that, so it fails.

And earlier codes fails because of missing 4th arguments.

add_filter( 'login_redirect', 'redirectClient', 10 , 3 );

Thanks.

[andreu]

I’m sorry but I couldn’t make it work. get_term_link() and the rest of the function seem to be ok.
Finally, I made a little trick that let me do what I need. When the user login I use the login_redirect() filter to redirect him to the homepage with a get variable:

return site_url().'?goto=clientarea';

Then, with template_redirect() hook, I redirect him to the custom term page with a conditional, this way:

function my_page_template_redirect(){
    if( $_GET['goto'] == 'clientarea' ){
        wp_redirect( get_term_link(...) );
        exit();
    }
}
add_action( 'template_redirect', 'my_page_template_redirect' );

The get_term_link() function is working fine in the template_redirect() function. However, I couldn’t make it work in the login_redirect() one…

Thanks

[Author]

Posts