Merge pull request #46 from markuspeter/main

Fix required validator for inputs other than checkbox and radio
haacked · Jun 21, 2023 · f74388e · f74388e
2 parents d6d4aff + 26ac977
commit f74388e
Show file tree

Hide file tree

Showing 6 changed files with 7 additions and 7 deletions.
diff --git a/dist/aspnet-validation.js b/dist/aspnet-validation.js
@@ -192,7 +192,7 @@ var MvcValidationProviders = /** @class */ (function () {
         this.required = function (value, element, params) {
             // Handle single and multiple checkboxes/radio buttons.
             var elementType = element.type.toLowerCase();
-            if (elementType === "checkbox" || elementType) {
+            if (elementType === "checkbox" || elementType === "radio") {
                 var allElementsOfThisName = Array.from(element.form.querySelectorAll("input[name='".concat(element.name, "'][type='").concat(elementType, "']")));
                 for (var _i = 0, allElementsOfThisName_1 = allElementsOfThisName; _i < allElementsOfThisName_1.length; _i++) {
                     var element_1 = allElementsOfThisName_1[_i];

diff --git a/dist/aspnet-validation.min.js b/dist/aspnet-validation.min.js
diff --git a/dist/aspnet-validation.min.js.map b/dist/aspnet-validation.min.js.map
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "aspnet-client-validation",
-  "version": "0.8.8",
+  "version": "0.8.9",
   "description": "Enables ASP.NET MVC client-side validation, without jQuery!",
   "main": "dist/aspnet-validation.js",
   "style": "dist/aspnet-validation.css",

diff --git a/src/index.ts b/src/index.ts
@@ -102,7 +102,7 @@ export class MvcValidationProviders {
     required: ValidationProvider = (value, element, params) => {
         // Handle single and multiple checkboxes/radio buttons.
         const elementType = element.type.toLowerCase();
-        if (elementType === "checkbox" || elementType) {
+        if (elementType === "checkbox" || elementType === "radio") {
             const allElementsOfThisName = Array.from(element.form.querySelectorAll(`input[name='${element.name}'][type='${elementType}']`));
             for (let element of allElementsOfThisName) {
                 if (element instanceof HTMLInputElement && element.checked === true) {