Merge pull request #1074 from scottgonzalez/age-group-code-validator

Age Group: Proper validation for code length
jflamy · Aug 22, 2024 · 48abaf9 · 48abaf9
2 parents a3a5efd + 43a1a8b
commit 48abaf9
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/owlcms/src/main/java/app/owlcms/nui/preparation/AgeGroupEditingFormFactory.java b/owlcms/src/main/java/app/owlcms/nui/preparation/AgeGroupEditingFormFactory.java
@@ -106,9 +106,10 @@ public Component buildNewForm(CrudOperation operation, AgeGroup aFromDb, boolean
 		formLayout.addFormItem(codeField, Translator.translate("AgeGroupCode"));
 		int maxLength = 5;
 		codeField.setRequired(true);
+		codeField.setMaxLength(maxLength);
 		this.binder.forField(codeField)
 		        .withValidator(
-		                new StringLengthValidator(Translator.translate("ThisFieldIsRequired", maxLength), 1, maxLength))
+		                new StringLengthValidator(Translator.translate("ThisFieldIsRequired", maxLength), 1, null))
 		        .withValidator(
 		                new StringLengthValidator(Translator.translate("CodeMustBeShort", maxLength), 0, maxLength))
 		        .bind(AgeGroup::getCode, AgeGroup::setCode);