Merge pull request #112 from RyanONeill1970/main

haacked · web-flow · commit 36a24f13b177 · 2024-05-08T12:42:04.000-07:00
Fix for client side 'required' validator which allowed whitespace
diff --git a/src/index.ts b/src/index.ts
@@ -153,8 +153,9 @@ export class MvcValidationProviders {
 
             return false;
         }
-        // Default behavior otherwise.
-        return Boolean(value);
+
+        // Default behavior otherwise (trim ensures whitespace only is not seen as valid).
+        return Boolean(value?.trim());
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -153,8 +153,9 @@ export class MvcValidationProviders {`
`153`	`153`
`154`	`154`	`return false;`
`155`	`155`	`}`
`156`		`- // Default behavior otherwise.`
`157`		`- return Boolean(value);`
	`156`	`+`
	`157`	`+ // Default behavior otherwise (trim ensures whitespace only is not seen as valid).`
	`158`	`+ return Boolean(value?.trim());`
`158`	`159`	`}`
`159`	`160`
`160`	`161`	`/**`