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

[ConfirmRegister] Error if username has only digits #88

Open
Skippidar opened this issue Mar 21, 2017 · 2 comments
Open

[ConfirmRegister] Error if username has only digits #88

Skippidar opened this issue Mar 21, 2017 · 2 comments

Comments

@Skippidar
Copy link

Skippidar commented Mar 21, 2017

I'm trying to create registration with only email needed. I generate usernames like 4568245873 but when i try to confirm registration - i see 404 error page. I tried to create user using old registration - when i type username like "Skippidar" - everything is ok and i can verify my email, but if i type 0000000007 or any other digits - i see 404 error page again.

I solved it by adding "u" at the beginning when generating username. But i think that's MODX's registry bug, so it would be better to use not
$this->modx->registry->login->subscribe('/useractivation/'.$this->user->get('username'));
but
$this->modx->registry->login->subscribe('/useractivation/user'.$this->user->get('username'));

@Skippidar Skippidar changed the title [ConfirmRegister] Error if username was generated in preHook [ConfirmRegister] Error if username has only digits Mar 21, 2017
@matdave
Copy link

matdave commented Sep 18, 2017

I think I have a related issue. It appears if the password is all digits, it has the same result.

@zlobinberg
Copy link

I have the same problem. It is very important for me that users always enter only digits in their login.

Applied hack [[! LoginRegisterFix, but I need to be in the database, too, without the "u" - so it does not suit me.

What to do? There is a solution? Help please, for earlier thanks.

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

3 participants