Wrong redirection on login when redirect_to is defined

[francescolaffi]

I found that when in the login request is present the redirect_to address, buddypress redirect to homepage if 'wp-admin' is not in the address, shuldn't it do the opposite?

in bp-core.php in the function bp_core_login_redirect i think that this line (2106)

if ( !empty( $_REQUEST['redirect_to'] ) && strpos( $_REQUEST['redirect_to'], 'wp-admin' ) )

should be

if ( !empty( $_REQUEST['redirect_to'] ) && false === strpos( $_REQUEST['redirect_to'], 'wp-admin' ) )

P.S.: sorry if i did something wrong, it's the first time i use a bug tracking system

[cnorris23]

Is this still an issue? Glancing at the code, it looks as if this is correct.

[boonebgorges]

(In [4898]) Ensures that bp_core_login_redirect() successfully directs users away from wp-admin. Fixes #2073. Props francescolaffi