Prevent multiple emails from pointing to same collaborator (fixes #78)

This commit is contained in:
R. Miles McCain 2020-08-28 17:19:57 +00:00
parent c8dead4457
commit 4a07ab80ce
No known key found for this signature in database
GPG Key ID: F1053629E2905557

View File

@ -60,6 +60,7 @@ class ServiceForm(forms.ModelForm):
def clean_collaborators(self):
collaborators = []
users_to_emails = {} # maps users to the email they are listed under as a collaborator
for collaborator_email in self.cleaned_data["collaborators"].split(","):
email = collaborator_email.strip()
if email == "":
@ -69,6 +70,10 @@ class ServiceForm(forms.ModelForm):
).first()
if collaborator_email_linked is None:
raise forms.ValidationError(f"Email '{email}' is not registered")
user = collaborator_email_linked.user
if user in collaborators:
raise forms.ValidationError(f"The emails '{email}' and '{users_to_emails[user]}' both correspond to the same user")
users_to_emails[user] = email
collaborators.append(collaborator_email_linked.user)
return collaborators