diff --git a/src/registrar/forms/portfolio.py b/src/registrar/forms/portfolio.py index 4d6ec81cb..5309f7263 100644 --- a/src/registrar/forms/portfolio.py +++ b/src/registrar/forms/portfolio.py @@ -234,7 +234,6 @@ class NewMemberForm(forms.ModelForm): if email_value: cleaned_data["email"] = email_value.lower() - ########################################## # TODO: future ticket # (invite new member) @@ -257,7 +256,7 @@ class NewMemberForm(forms.ModelForm): if field in self.errors and field != "email" and field != "member_access_level": del self.errors[field] return cleaned_data - + basic_dom_req_error = "basic_org_domain_request_permissions" admin_dom_req_error = "admin_org_domain_request_permissions" admin_member_error = "admin_org_members_permissions" @@ -267,8 +266,8 @@ class NewMemberForm(forms.ModelForm): del self.errors[basic_dom_req_error] elif member_access_level == "basic": # remove the error messages pertaining to admin permission inputs - if admin_dom_req_error in self.errors: + if admin_dom_req_error in self.errors: del self.errors[admin_dom_req_error] - if admin_member_error in self.errors: + if admin_member_error in self.errors: del self.errors[admin_member_error] return cleaned_data diff --git a/src/registrar/views/portfolios.py b/src/registrar/views/portfolios.py index 9e4d2c5a6..1dbab2913 100644 --- a/src/registrar/views/portfolios.py +++ b/src/registrar/views/portfolios.py @@ -423,7 +423,6 @@ class NewMemberView(PortfolioMembersPermissionView, FormMixin): else: return self.form_invalid(form) - def form_invalid(self, form): """Handle the case when the form is invalid.""" return self.render_to_response(self.get_context_data(form=form))