When Use two e-mail fields on registration form is enabled, the user must input the email address exactly the same twice.

This means that if you enter thisisanexample@example.com and THISISANEXAMPLE@EXAMPLE.COM, it will throw the "Your e-mail address and confirmed e-mail address must match." error.

It also doesn't trim the input, so if a stray space character finds its way to the end of one of the email addresses, it will cause problems.

Patch forthcoming.

CommentFileSizeAuthor
#1 logintoboggan-1843066-01.patch867 bytesBrockBoland

Comments

BrockBoland’s picture

BrockBoland commented
Status: Active » Needs review
StatusFileSize
new logintoboggan-1843066-01.patch867 bytes

Patched

stevecowie’s picture

stevecowie commented
Status: Needs review » Needs work

I think the point about trim makes good sense, but I'm less convinced about the case sensitive issue. I don't see that as an unreasonable requirement. Happy to commit the trim, if you want to re-roll the patch.

  • stevecowie committed bd47d77 on 7.x-1.x 
    Issue #1843066 by md2: prevent an existing user's email address being...

  • md2 committed 585299b on 7.x-1.x 
    Issue #1843066 by BrockBoland: Trimming email input of any stray space...
dooug’s picture

dooug commented
Issue summary: View changes
Status: Needs work » Fixed

I agree with @stevecowie on #2. Ensuring that the Email and Confirmation email are the same case is reasonable, though maybe not entirely necessary.

I don't see any follow-up required here. Marking as fixed.

Status: Fixed » Closed (fixed)

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