Hi

In some use cases (as mine now) you want to validate that the mail entered in login form is ok and you want to force login only by email, this can be accomplished just with a custom validate handler that validates the email but email_registration doesn't keep this data.
I'm attaching a really little patch for keeping the email address in the validation phase.

CommentFileSizeAuthor
#6 1291908-keep_email_for_validate-6.patch726 bytesgreggles
#2 1291908-keep_email_for_validate-2.patch756 bytespcambra
#1 1291908-keep_email_for_validate-1.patch720 bytespcambra
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pcambra’s picture

pcambra
he/him
Primary language Spanish
Location Asturies
CreditAttribution: pcambra commented
Status: Active » Needs review
FileSize
1291908-keep_email_for_validate-1.patch720 bytes
pcambra’s picture

pcambra
he/him
Primary language Spanish
Location Asturies
CreditAttribution: pcambra commented
FileSize
1291908-keep_email_for_validate-2.patch756 bytes

It should be done before the user check.

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost commented

Great but this should be configurable or not? this adds an option #583152-13: email confirmation option

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles commented
Title: Keep the email login entered by the user » Keep the email login entered by the user so it can be validated

Makes sense to me.

Status: Needs review » Needs work

The last submitted patch, 1291908-keep_email_for_validate-2.patch, failed testing.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles commented
Status: Needs work » Needs review
FileSize
1291908-keep_email_for_validate-6.patch726 bytes

reroll.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles commented
Version: 7.x-1.x-dev » 6.x-1.x-dev
Status: Needs review » Patch (to be ported)

Now committed to 7.x - http://drupalcode.org/project/email_registration.git/commit/ceac58a

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles commented
Status: Patch (to be ported) » Fixed

Needed a slight re-roll but seems to work http://drupalcode.org/project/email_registration.git/commit/944b4c5

Status: Fixed » Closed (fixed)

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