Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Creating manually a user with PMO ID is not possible #165

Open
ice1e0 opened this issue Jun 1, 2024 · 3 comments
Open

Creating manually a user with PMO ID is not possible #165

ice1e0 opened this issue Jun 1, 2024 · 3 comments

Comments

@ice1e0
Copy link

ice1e0 commented Jun 1, 2024

Hi, I switched to the new plugin and users are now not anymore automatically created. I tried now to create a user manually, but a user name must include at least one letter when I want to create the user via the Wordpress admin page. How can I now with the new plugin create a user to assign e.g. role admin when he/she log in via BCC Login?

@ice1e0
Copy link
Author

ice1e0 commented Jun 3, 2024

Note: I figured out that the Wordpress validation rule in wp-includes/ms-functions.php creates this issue:

// All numeric?
if ( preg_match( '/^[0-9]*$/', $user_name ) ) {
	$errors->add( 'user_name', __( 'Sorry, usernames must have letters too!' ) );
}

One option is to manually disable this logic but I guess it is there for a reason. The Wordpress plugin should work here differently to link the PMO ID to the user by another field than the username.

@cs1m0n
Copy link
Member

cs1m0n commented Jun 4, 2024

The user can be linked to the BCC member also by the email address, totally independent of what username you are setting to the account.

@ice1e0
Copy link
Author

ice1e0 commented Jun 7, 2024

@cs1m0n This is great to hear!
But I think this would not work anymore when someone changes the email address, right? Then one would be locked out of Wordpress right?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants