Fix required validator for inputs other than checkbox and radio

haacked · Jun 21, 2023 · aa5543a · aa5543a
1 parent d6d4aff
commit aa5543a
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 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/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) {