Merge pull request #178 from mgm09a/master

ghiscoding · web-flow · commit ce2237066cc7 · 2018-11-13T23:17:08.000-05:00
Fixed revalidateAndAttachOnBlur method to handle zeroes appropriately
diff --git a/src/validation-directive.js b/src/validation-directive.js
@@ -179,7 +179,7 @@
           }
 
           // invalidate field before doing any validation
-          if(!!value || commonObj.isFieldRequired() || _validateOnEmpty) {
+          if((value !== "" && value !== null && typeof value !== "undefined") || commonObj.isFieldRequired() || _validateOnEmpty) {
             ctrl.$setValidity('validation', false);
           }
 
@@ -403,7 +403,9 @@
         /** Re-evaluate the element and revalidate it, also re-attach the onBlur event on the element */
         function revalidateAndAttachOnBlur() {
           // Revalidate the input when enabled (without displaying the error)
-          var value = ctrl.$modelValue || '';
+          var value = ctrl.$modelValue !== null && typeof ctrl.$modelValue !== 'undefined'
+                        ? ctrl.$modelValue
+                        : '';
           if(!Array.isArray(value)) {
             ctrl.$setValidity('validation', commonObj.validate(value, false));
           }

Original file line number	Diff line number	Diff line change
`@@ -179,7 +179,7 @@`
`179`	`179`	`}`
`180`	`180`
`181`	`181`	`// invalidate field before doing any validation`
`182`		`- if(!!value \|\| commonObj.isFieldRequired() \|\| _validateOnEmpty) {`
	`182`	`+ if((value !== "" && value !== null && typeof value !== "undefined") \|\| commonObj.isFieldRequired() \|\| _validateOnEmpty) {`
`183`	`183`	`ctrl.$setValidity('validation', false);`
`184`	`184`	`}`
`185`	`185`
`@@ -403,7 +403,9 @@`
`403`	`403`	`/** Re-evaluate the element and revalidate it, also re-attach the onBlur event on the element */`
`404`	`404`	`function revalidateAndAttachOnBlur() {`
`405`	`405`	`// Revalidate the input when enabled (without displaying the error)`
`406`		`- var value = ctrl.$modelValue \|\| '';`
	`406`	`+ var value = ctrl.$modelValue !== null && typeof ctrl.$modelValue !== 'undefined'`
	`407`	`+ ? ctrl.$modelValue`
	`408`	`+ : '';`
`407`	`409`	`if(!Array.isArray(value)) {`
`408`	`410`	`ctrl.$setValidity('validation', commonObj.validate(value, false));`
`409`	`411`	`}`