8
0
Fork 0
mirror of https://gitlab2.federez.net/re2o/re2o synced 2024-11-27 07:02:26 +00:00

Fix some email checks being bypassed for legacy users

This commit is contained in:
Jean-Romain Garnier 2020-04-21 18:40:19 +02:00 committed by Gabriel Detraz
parent 1755f6e624
commit 6235909a9a

View file

@ -564,11 +564,8 @@ class AdherentEditForm(AdherentForm):
original_email = self.user.email
new_email = self.cleaned_data.get("email")
# Allow empty emails if the user had an empty email before
if not original_email:
return new_email
if not new_email:
# Allow empty emails only if the user had an empty email before
if original_email and not new_email:
raise forms.ValidationError(
_("Email field cannot be empty.")
)
@ -880,11 +877,8 @@ class EmailSettingsForm(FormRevMixin, FieldPermissionFormMixin, ModelForm):
original_email = self.user.email
new_email = self.cleaned_data.get("email")
# Allow empty emails if the user had an empty email before
if not original_email:
return new_email
if not new_email:
# Allow empty emails only if the user had an empty email before
if original_email and not new_email:
raise forms.ValidationError(
_("Email field cannot be empty.")
)