Edit email doesn't work, says that email already exists [#704564]

I'm not sure that this screen is even useful, so I'm just going to hide the link in the D7 version for now, but in the D6 version this is a bug.

If it should be there, I'll fix the logic in the D7 version

Comment	File	Size	Author
#1	mailing_list_edit_email.patch	1.07 KB	fangel

Comments

Comment #1

fangel commented 13 August 2010 at 07:55

Status	File	Size
new	mailing_list_edit_email.patch	1.07 KB

Yes. I just came across this bug as well in the D6 version. The reason for this bug is that the same validation function is used for signups and for edits - and the requirement for signup is that the email doesn't exist, but for edit the requirement is that the `eid` exists..

So I've created a small patch that checks if `eid` is set in the form, and if it is checks if it's valid - if `eid` doesn't exist, it checks the email for uniqueness..
(The coding-style and/or logic is probably a bit off, I'm new to Drupal development)

-Morten