zydronium/manage.get.gov
1
0
Fork 0
mirror of https://github.com/cisagov/manage.get.gov.git synced 2025-07-23 11:16:07 +02:00
Code Issues Projects Releases Packages Wiki Activity

fixed bug with error message expanding add new nameserver form

Browse source
This commit is contained in:
David Kennedy 2025-03-04 11:03:41 -05:00
parent 35d0d11eca
commit 5489e65df1
No known key found for this signature in database
GPG key ID: 6528A5386E66B96B
1 changed files with 4 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

6
src/registrar/forms/domain.py
View file

@ -214,8 +214,10 @@ class BaseNameserverFormset(forms.BaseFormSet):
return form.is_valid() or list(form.errors.get("server", [])) == [error_message]
def _enforce_minimum_nameservers(self, valid_forms, empty_forms, error_message):
"""Ensure at least two nameservers are provided, adjusting error messages as needed."""
if len(valid_forms) < 2:
"""Ensure at least two nameservers are provided, adjusting error messages as needed. This accounts
for three forms, where the last form is empty and one of the first two forms has an error. In that case
will want to remove the 'at least two valid servers' error."""
if len(valid_forms) < 2 and len(self.forms) < 3:
self._add_required_error(empty_forms, error_message)
else:
self._remove_required_error_from_forms(error_message)