Skip to content

[ConfirmRegister] Error if username has only digits #88

Open

Description

@Skippidar

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'));

Activity

  1. changed the title [-][ConfirmRegister] Error if username was generated in preHook[/-] [+][ConfirmRegister] Error if username has only digits[/+] on Mar 21, 2017
  2. matdave commented on Sep 18, 2017

    @matdave
    Member

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

  3. zlobinberg commented on Jan 10, 2018

    @zlobinberg

    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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions