Merge pull request #12586 from asineath-onpoint/fix/abide-validateInput-remove-cleared-errors

joeworkman · web-flow · commit 130e7f7735d3 · 2024-09-27T08:55:04.000-07:00
fix: Abide validateInput - remove existing error classes before adding error classes
diff --git a/js/foundation.abide.js b/js/foundation.abide.js
@@ -510,10 +510,9 @@ class Abide extends Plugin {
     }
 
     if (manageErrorClasses) {
+      this.removeErrorClasses($el);
       if (!goodToGo) {
-        this.addErrorClasses($el, failedValidators);
-      } else {
-        this.removeErrorClasses($el);
+          this.addErrorClasses($el, failedValidators);
       }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -510,10 +510,9 @@ class Abide extends Plugin {`
`510`	`510`	`}`
`511`	`511`
`512`	`512`	`if (manageErrorClasses) {`
	`513`	`+ this.removeErrorClasses($el);`
`513`	`514`	`if (!goodToGo) {`
`514`		`- this.addErrorClasses($el, failedValidators);`
`515`		`- } else {`
`516`		`- this.removeErrorClasses($el);`
	`515`	`+ this.addErrorClasses($el, failedValidators);`
`517`	`516`	`}`
`518`	`517`	`}`
`519`	`518`