When you've chosen not to include spaces in usernames, it would make sense not to include spaces existing within the fields that build the username.

CommentFileSizeAuthor
#1 user_import-534958.patch788 bytesmlsamuelson
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mlsamuelson’s picture

mlsamuelson CreditAttribution: mlsamuelson commented
Status: Active » Needs review
FileSize
user_import-534958.patch788 bytes

I don't know if stripping spaces from fields is best handled in user_import_create_username() when we have _user_import_sanitise_username() but the proper variable scope existed there...

mlsamuelson’s picture

mlsamuelson CreditAttribution: mlsamuelson commented
Title: When not allowing spaces username parts, remove all spaces » When not allowing spaces between username parts, remove all spaces

Fixing typo in issue title.

gisle’s picture

gisle
he/him
Primary language Norwegian Bokmål
Location Norway
CreditAttribution: gisle at Hannemyr Nye Medier AS commented
Version: master » 7.x-3.x-dev
Issue summary: View changes
Status: Needs review » Needs work

Bumping version. This should be fixed in concert with #631408: On duplicate usernames concatenate the suffix without a space.