Does not work with special characters. (ä, ü, ü). Is needed in European countries. Otherwise, good work!

Comments

sgerbino’s picture

sgerbino commented
Status: Active » Needs review

Ok, I've revised the regex to allow for accented characters, I'd just like some testing before I roll a stable release of it.

sgerbino’s picture

sgerbino commented
Status: Needs review » Fixed
sgerbino’s picture

sgerbino commented
Status: Fixed » Closed (fixed)
AlexDeRuyter’s picture

AlexDeRuyter commented

What about spaces in the lastname field ?

sw3b’s picture

sw3b commented
Status: Closed (fixed) » Needs work

The feature need re-work. In french language a lot of person have name like

Firstname : Jean-Francois
Lastname : Le Premier

If the Use Realname registration validation is check I got these errors:

•First name must only contain letters.
•Middle name must only contain letters.
•Last name must only contain letters.

Thanks for your work !

sgerbino’s picture

sgerbino commented
Assigned: Waldknoblauch » sgerbino
sgerbino’s picture

sgerbino commented

I'm open to suggestions on how to better validate real names. Perhaps looking for characters it may NOT contain would be a more versatile and language-friendly route.

sgerbino’s picture

sgerbino commented
Status: Needs work » Closed (fixed)

For now, I've modified the regular expression to allow for hyphens and spaces in the latest 7.x-2.x-dev branch.

sgerbino’s picture

sgerbino commented
Priority: Major » Normal
Status: Closed (fixed) » Needs work
sw3b’s picture

sw3b commented

Ok I'll take a look thanks !!!

sgerbino’s picture

sgerbino commented
Status: Needs work » Closed (fixed)

After doing a bit of research, I think the best route to take is validate the fields against a regex blacklist of symbols and numbers. I have an initial regex that will roll on the next dev branch. I'm marking this issue as closed, if the regex can be improved, feel free to re-open or create a new issue.