Right now if a user attempts to register with an email address that another user has added to their email addresses but has not yet confirmed, the new user's registration is blocked. This behavior must be changed to one of the following:

  1. Cancel the pending confirmation and allocate the email address to the new user
  2. Take new registrant to another page confirming that they actually wish to register a new account, then proceed with item #1 above

Comments

shawn dearmond’s picture

shawn dearmond commented
Version: 5.x-1.x-dev » 6.x-1.x-dev
Assigned: Josh Benner » Unassigned

I'm going to bump this issue to 6.x because it still applies. It might be a good idea to have Cron remove unconfirmed emails after a given amount of time, say, 24 hours or so. That would help this issue.

Open to discussion.

shawn dearmond’s picture

shawn dearmond commented

@sdboyer recommends adding a "If you received this message in error, click this link and your email will be removed from our system." to the confirmation message.

shawn dearmond’s picture

shawn dearmond commented

Duh. There already is a setting for how long unconfirmed emails sit before they expire. That should solve the main problem.

Adding the removal language to the confirmation email is still a good idea.

cor3huis’s picture

cor3huis commented

Not fully the same issue. Situation: Someone registers and adds an email which is not his for whatever reason. This makes it impossible for the other user which owns the real email to register. It should be possible to register even if the email is already taken on an unconfirmed user. If the email gets confirmed only then the other email should be removed.

BTW if we think having duplicate emails as non primary should not be allowed, what would be the extensive rationale behind this decision?

lee20’s picture

lee20 commented

Subscribing. I concur with the behavior described by cor3huis in #5.

I think the CCK filters provided by this module should also allow you to filter by only confirmed addresses as well.

drumm’s picture

drumm
he/him
Location NY, US
commented
Version: 6.x-1.x-dev » 7.x-1.x-dev
Issue summary: View changes

I'm guessing this still applies to 7.x.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Status: Active » Closed (outdated)

I am closing this issue, since it is for a Drupal version that is no longer supported. Please re-open this issue if it is relevant for a project release for a supported Drupal version.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.