Probably at the end of this day, you'll hate me. But I'm here again :)

In your ConfigForm you don't give possibility to avoid spaces in username. I really need that, and I made a patch for you :)

Comments

g0blin79 created an issue. See original summary.

g0blin79’s picture

This is the patch

g0blin79’s picture

StatusFileSize
new3.19 KB

Fixed the patch.

g0blin79’s picture

StatusFileSize
new4 KB

I noticed that in 1.x-dev release the ajax messages works well.

I added the ajax message also for my check for spaces. This could be finally the patch.

sprite’s picture

This feature, to configure this module to disallow spaces is very important.

There are a host of situations when users other module where user names generated with spaces will break the other modules.
Please provide a feature for this module, so that it can be configured to disallow spaces in user names.

Thanks you.

nkoporec’s picture

Status: Active » Reviewed & tested by the community

Tested latest patch from @g0blin79 and it's working.I also think this is a very important feature which should be implemented as soon as possible.Changing status to RTBC.

  • VladimirAus committed c36a99a on 8.x-1.x authored by g0blin79
    Issue #2873137 by g0blin79, nkoporec, VladimirAus: Add check to avoid...
vladimiraus’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.