Setting Accept New Registration to No restricts current users from logging in

Viewing 3 replies - 1 through 3 (of 3 total)
  • Thread Starter knsheely

    (@knsheely)

    The problem seems to be on line 704 of wsl.authentication.php

    // if user do not exist
			if( ! $hybridauth_user_id ){

    I think this is checking to see if the user has been linked with the provider. There are a few problems with this. 1st, what if the user has never used the provider to login to this site? 2nd, even if the user has used the provider to login to the site previously, the $hybridauth_user_id is 0. There seems to be a problem with the wsl_get_user_by_meta() function on line 597. What I did to fix this is to check for an existing email instead. I replaced line 704 with the following:

    if( ! email_exists($hybridauth_user_email) ){

    knsheely: thanks for this fix!!

    I submitted it as a pull request to WSL: https://github.com/hybridauth/WordPress-Social-Login/pull/33.

    B M

    (@bmiwc)

    Thanks!

    This fixed the issue I was having too.

Viewing 3 replies - 1 through 3 (of 3 total)
  • The topic ‘Setting Accept New Registration to No restricts current users from logging in’ is closed to new replies.

Tags