Fix ValidationUtil helper typo

aabashkin · aabashkin · commit 50f8fe8d8399 · 2026-01-05T15:39:06.000-05:00
diff --git a/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidCreditCardValidator.java b/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidCreditCardValidator.java
@@ -27,7 +27,7 @@ public boolean isValid(String input, ConstraintValidatorContext constraintValida
         boolean valid = ESAPI.validator().isValidCreditCard(context, input, allowNull, errorList);
         
         if(!valid){
-            ValidationUtil.addViolatons(errorList, constraintValidatorContext);
+            ValidationUtil.addViolations(errorList, constraintValidatorContext);
         }
         
         return valid;
diff --git a/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidDateValidator.java b/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidDateValidator.java
@@ -37,7 +37,7 @@ public boolean isValid(String input, ConstraintValidatorContext constraintValida
         boolean valid = ESAPI.validator().isValidDate(context, input, dateFormat, allowNull, errorList);
         
         if(!valid){
-            ValidationUtil.addViolatons(errorList, constraintValidatorContext);
+            ValidationUtil.addViolations(errorList, constraintValidatorContext);
         }
         
         return valid;
diff --git a/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidDirectoryPathValidator.java b/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidDirectoryPathValidator.java
@@ -32,7 +32,7 @@ public boolean isValid(String input, ConstraintValidatorContext constraintValida
         boolean valid = ESAPI.validator().isValidDirectoryPath(context, input, parent, allowNull, errorList);
         
         if(!valid){
-            ValidationUtil.addViolatons(errorList, constraintValidatorContext);
+            ValidationUtil.addViolations(errorList, constraintValidatorContext);
         }
         
         return valid;
diff --git a/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidDoubleValidator.java b/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidDoubleValidator.java
@@ -31,7 +31,7 @@ public boolean isValid(String input, ConstraintValidatorContext constraintValida
         boolean valid = ESAPI.validator().isValidDouble(context, input, minValue, maxValue, allowNull, errorList);
         
         if(!valid){
-            ValidationUtil.addViolatons(errorList, constraintValidatorContext);
+            ValidationUtil.addViolations(errorList, constraintValidatorContext);
         }
         
         return valid;
diff --git a/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidFileContentValidator.java b/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidFileContentValidator.java
@@ -29,7 +29,7 @@ public boolean isValid(byte[] input, ConstraintValidatorContext constraintValida
         boolean valid = ESAPI.validator().isValidFileContent(context, input, maxBytes, allowNull, errorList);
         
         if(!valid){
-            ValidationUtil.addViolatons(errorList, constraintValidatorContext);
+            ValidationUtil.addViolations(errorList, constraintValidatorContext);
         }
         
         return valid;
diff --git a/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidFileNameValidator.java b/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidFileNameValidator.java
@@ -39,7 +39,7 @@ public boolean isValid(String input, ConstraintValidatorContext constraintValida
         }
         
         if(!valid){
-            ValidationUtil.addViolatons(errorList, constraintValidatorContext);
+            ValidationUtil.addViolations(errorList, constraintValidatorContext);
         }
         
         return valid;
diff --git a/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidFileUploadValidator.java b/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidFileUploadValidator.java
@@ -38,7 +38,7 @@ public boolean isValid(byte[] input, ConstraintValidatorContext constraintValida
         boolean valid = ESAPI.validator().isValidFileUpload(context, directoryPath, fileName, parent, input, maxBytes, allowNull, errorList);
         
         if(!valid){
-            ValidationUtil.addViolatons(errorList, constraintValidatorContext);
+            ValidationUtil.addViolations(errorList, constraintValidatorContext);
         }
         
         return valid;
diff --git a/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidHTTPRequestParameterSetValidator.java b/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidHTTPRequestParameterSetValidator.java
@@ -41,7 +41,7 @@ public boolean isValid(HttpServletRequest input, ConstraintValidatorContext cons
         boolean valid = ESAPI.validator().isValidHTTPRequestParameterSet(context, input, requiredNamesSet, optionalNamesSet, errorList);
         
         if(!valid){
-            ValidationUtil.addViolatons(errorList, constraintValidatorContext);
+            ValidationUtil.addViolations(errorList, constraintValidatorContext);
         }
         
         return valid;
diff --git a/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidIntegerValidator.java b/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidIntegerValidator.java
@@ -31,7 +31,7 @@ public boolean isValid(String input, ConstraintValidatorContext constraintValida
         boolean valid = ESAPI.validator().isValidInteger(context, input, minValue, maxValue, allowNull, errorList);
         
         if(!valid){
-            ValidationUtil.addViolatons(errorList, constraintValidatorContext);
+            ValidationUtil.addViolations(errorList, constraintValidatorContext);
         }
         
         return valid;
diff --git a/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidListItemValidator.java b/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidListItemValidator.java
@@ -36,7 +36,7 @@ public boolean isValid(String input, ConstraintValidatorContext constraintValida
         boolean valid = ESAPI.validator().isValidListItem(context, input, list, errorList);
         
         if(!valid){
-            ValidationUtil.addViolatons(errorList, constraintValidatorContext);
+            ValidationUtil.addViolations(errorList, constraintValidatorContext);
         }
         
         return valid;
diff --git a/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidNumberValidator.java b/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidNumberValidator.java
@@ -31,7 +31,7 @@ public boolean isValid(String input, ConstraintValidatorContext constraintValida
         boolean valid = ESAPI.validator().isValidNumber(context, input, minValue, maxValue, allowNull, errorList);
         
         if(!valid){
-            ValidationUtil.addViolatons(errorList, constraintValidatorContext);
+            ValidationUtil.addViolations(errorList, constraintValidatorContext);
         }
         
         return valid;
diff --git a/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidPrintableStringValidator.java b/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidPrintableStringValidator.java
@@ -29,7 +29,7 @@ public boolean isValid(String input, ConstraintValidatorContext constraintValida
         boolean valid = ESAPI.validator().isValidPrintable(context, input, maxLength, allowNull, errorList);
         
         if(!valid){
-            ValidationUtil.addViolatons(errorList, constraintValidatorContext);
+            ValidationUtil.addViolations(errorList, constraintValidatorContext);
         }
         
         return valid;
diff --git a/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidPrintableValidator.java b/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidPrintableValidator.java
@@ -29,7 +29,7 @@ public boolean isValid(char[] input, ConstraintValidatorContext constraintValida
         boolean valid = ESAPI.validator().isValidPrintable(context, input, maxLength, allowNull, errorList);
         
         if(!valid){
-            ValidationUtil.addViolatons(errorList, constraintValidatorContext);
+            ValidationUtil.addViolations(errorList, constraintValidatorContext);
         }
         
         return valid;
diff --git a/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidRedirectLocationValidator.java b/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidRedirectLocationValidator.java
@@ -27,7 +27,7 @@ public boolean isValid(String input, ConstraintValidatorContext constraintValida
         boolean valid = ESAPI.validator().isValidRedirectLocation(context, input, allowNull, errorList);
         
         if(!valid){
-            ValidationUtil.addViolatons(errorList, constraintValidatorContext);
+            ValidationUtil.addViolations(errorList, constraintValidatorContext);
         }
         
         return valid;
diff --git a/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidSafeHTMLValidator.java b/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidSafeHTMLValidator.java
@@ -29,7 +29,7 @@ public boolean isValid(String input, ConstraintValidatorContext constraintValida
         boolean valid = ESAPI.validator().isValidSafeHTML(context, input, maxLength, allowNull, errorList);
         
         if(!valid){
-            ValidationUtil.addViolatons(errorList, constraintValidatorContext);
+            ValidationUtil.addViolations(errorList, constraintValidatorContext);
         }
         
         return valid;
diff --git a/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidStringValidator.java b/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidStringValidator.java
@@ -33,7 +33,7 @@ public boolean isValid(String input, ConstraintValidatorContext constraintValida
         boolean valid = ESAPI.validator().isValidInput(context, input, type, maxLength, allowNull, canonicalize, errorList);
         
         if(!valid){
-            ValidationUtil.addViolatons(errorList, constraintValidatorContext);
+            ValidationUtil.addViolations(errorList, constraintValidatorContext);
         }
         
         return valid;
diff --git a/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidationUtil.java b/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidationUtil.java
@@ -9,7 +9,7 @@ public class ValidationUtil {
 
     private ValidationUtil(){}
 
-    public static void addViolatons(ValidationErrorList errorList, ConstraintValidatorContext constraintValidatorContext){
+    public static void addViolations(ValidationErrorList errorList, ConstraintValidatorContext constraintValidatorContext){
         constraintValidatorContext.disableDefaultConstraintViolation();
         for (ValidationException vex : errorList.errors()) {
             String errorMessage = vex.getUserMessage();

Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ public boolean isValid(String input, ConstraintValidatorContext constraintValida`
`27`	`27`	`boolean valid = ESAPI.validator().isValidCreditCard(context, input, allowNull, errorList);`
`28`	`28`
`29`	`29`	`if(!valid){`
`30`		`- ValidationUtil.addViolatons(errorList, constraintValidatorContext);`
	`30`	`+ ValidationUtil.addViolations(errorList, constraintValidatorContext);`
`31`	`31`	`}`
`32`	`32`
`33`	`33`	`return valid;`
Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,7 @@ public boolean isValid(String input, ConstraintValidatorContext constraintValida`
`37`	`37`	`boolean valid = ESAPI.validator().isValidDate(context, input, dateFormat, allowNull, errorList);`
`38`	`38`
`39`	`39`	`if(!valid){`
`40`		`- ValidationUtil.addViolatons(errorList, constraintValidatorContext);`
	`40`	`+ ValidationUtil.addViolations(errorList, constraintValidatorContext);`
`41`	`41`	`}`
`42`	`42`
`43`	`43`	`return valid;`
Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ public boolean isValid(String input, ConstraintValidatorContext constraintValida`
`32`	`32`	`boolean valid = ESAPI.validator().isValidDirectoryPath(context, input, parent, allowNull, errorList);`
`33`	`33`
`34`	`34`	`if(!valid){`
`35`		`- ValidationUtil.addViolatons(errorList, constraintValidatorContext);`
	`35`	`+ ValidationUtil.addViolations(errorList, constraintValidatorContext);`
`36`	`36`	`}`
`37`	`37`
`38`	`38`	`return valid;`
Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ public boolean isValid(String input, ConstraintValidatorContext constraintValida`
`31`	`31`	`boolean valid = ESAPI.validator().isValidDouble(context, input, minValue, maxValue, allowNull, errorList);`
`32`	`32`
`33`	`33`	`if(!valid){`
`34`		`- ValidationUtil.addViolatons(errorList, constraintValidatorContext);`
	`34`	`+ ValidationUtil.addViolations(errorList, constraintValidatorContext);`
`35`	`35`	`}`
`36`	`36`
`37`	`37`	`return valid;`
Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,7 @@ public boolean isValid(byte[] input, ConstraintValidatorContext constraintValida`
`29`	`29`	`boolean valid = ESAPI.validator().isValidFileContent(context, input, maxBytes, allowNull, errorList);`
`30`	`30`
`31`	`31`	`if(!valid){`
`32`		`- ValidationUtil.addViolatons(errorList, constraintValidatorContext);`
	`32`	`+ ValidationUtil.addViolations(errorList, constraintValidatorContext);`
`33`	`33`	`}`
`34`	`34`
`35`	`35`	`return valid;`
Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ public boolean isValid(String input, ConstraintValidatorContext constraintValida`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`if(!valid){`
`42`		`- ValidationUtil.addViolatons(errorList, constraintValidatorContext);`
	`42`	`+ ValidationUtil.addViolations(errorList, constraintValidatorContext);`
`43`	`43`	`}`
`44`	`44`
`45`	`45`	`return valid;`
Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,7 @@ public boolean isValid(byte[] input, ConstraintValidatorContext constraintValida`
`38`	`38`	`boolean valid = ESAPI.validator().isValidFileUpload(context, directoryPath, fileName, parent, input, maxBytes, allowNull, errorList);`
`39`	`39`
`40`	`40`	`if(!valid){`
`41`		`- ValidationUtil.addViolatons(errorList, constraintValidatorContext);`
	`41`	`+ ValidationUtil.addViolations(errorList, constraintValidatorContext);`
`42`	`42`	`}`
`43`	`43`
`44`	`44`	`return valid;`
Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,7 @@ public boolean isValid(HttpServletRequest input, ConstraintValidatorContext cons`
`41`	`41`	`boolean valid = ESAPI.validator().isValidHTTPRequestParameterSet(context, input, requiredNamesSet, optionalNamesSet, errorList);`
`42`	`42`
`43`	`43`	`if(!valid){`
`44`		`- ValidationUtil.addViolatons(errorList, constraintValidatorContext);`
	`44`	`+ ValidationUtil.addViolations(errorList, constraintValidatorContext);`
`45`	`45`	`}`
`46`	`46`
`47`	`47`	`return valid;`
Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ public boolean isValid(String input, ConstraintValidatorContext constraintValida`
`36`	`36`	`boolean valid = ESAPI.validator().isValidListItem(context, input, list, errorList);`
`37`	`37`
`38`	`38`	`if(!valid){`
`39`		`- ValidationUtil.addViolatons(errorList, constraintValidatorContext);`
	`39`	`+ ValidationUtil.addViolations(errorList, constraintValidatorContext);`
`40`	`40`	`}`
`41`	`41`
`42`	`42`	`return valid;`