After last commit in DEV, spaces are not allowed in names, middle names or lastnames.

This regex:

/^[-\pL']*$/u

Should be this (Addded whitespace):

/^[-\pL' ]*$/u

Of course any of these can have a whitespace....

CommentFileSizeAuthor
#3 realname_registration-added_space_to_validation_regex-2360101-3.patch1.35 KBphilltran
#1 added_space_to_validation_regex-2360101-2.patch1.35 KBphilltran

Comments

philltran’s picture

philltran commented
StatusFileSize
new added_space_to_validation_regex-2360101-2.patch1.35 KB

Rolled my first patch, every. Please be gentle if it's wrong :)

david_garcia’s picture

david_garcia commented

1. If you want a review, change status to "Needs review" after submitting patch. This will trigger automatic testing if available.

2. The patch is not properly made, it won't apply. Please read this carefuly: https://www.drupal.org/node/707484

Greetings and good luck!

philltran’s picture

philltran commented
Status: Needs work » Needs review
StatusFileSize
new realname_registration-added_space_to_validation_regex-2360101-3.patch1.35 KB

@david_garcia Thanks for the heads up. I missed a step.

This patch should work.

david_garcia’s picture

david_garcia commented
Status: Needs review » Reviewed & tested by the community

Sorry for that, your first patch was all OK, it was me trying to apply it over the wrong branch.

Applies over 7.x-2.x and tested.

philltran’s picture

philltran commented

@david_garcia no problem. Thanks for the update. It was my first patch.

sgerbino’s picture

sgerbino commented
Status: Reviewed & tested by the community » Closed (fixed)

Merged in to 7.x-2.x

https://github.com/sgerbino/realname_registration/pull/3