Skip to content

Commit 8c6fe71

Browse files
committed
Remove jQuery from isMarkedForDestroy
1 parent a2089a5 commit 8c6fe71

File tree

4 files changed

+16
-8
lines changed

4 files changed

+16
-8
lines changed

dist/client-side-validations.esm.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -661,10 +661,12 @@ var executeValidators = function executeValidators(validatorFunctions, $element,
661661
return true;
662662
};
663663
var isMarkedForDestroy = function isMarkedForDestroy($element) {
664-
var elementName = $element.attr('name');
664+
var element = $element[0];
665+
var elementName = element.name;
665666
if (/\[([^\]]*?)\]$/.test(elementName)) {
666667
var destroyInputName = elementName.replace(/\[([^\]]*?)\]$/, '[_destroy]');
667-
if (jQuery("input[name=\"".concat(destroyInputName, "\"]")).val() === '1') {
668+
var destroyInputElement = document.querySelector("input[name=\"".concat(destroyInputName, "\"]"));
669+
if (destroyInputElement && destroyInputElement.value === '1') {
668670
return true;
669671
}
670672
}

dist/client-side-validations.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -665,10 +665,12 @@
665665
return true;
666666
};
667667
var isMarkedForDestroy = function isMarkedForDestroy($element) {
668-
var elementName = $element.attr('name');
668+
var element = $element[0];
669+
var elementName = element.name;
669670
if (/\[([^\]]*?)\]$/.test(elementName)) {
670671
var destroyInputName = elementName.replace(/\[([^\]]*?)\]$/, '[_destroy]');
671-
if (jQuery("input[name=\"".concat(destroyInputName, "\"]")).val() === '1') {
672+
var destroyInputElement = document.querySelector("input[name=\"".concat(destroyInputName, "\"]"));
673+
if (destroyInputElement && destroyInputElement.value === '1') {
672674
return true;
673675
}
674676
}

src/index.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -168,12 +168,14 @@ const executeValidators = (validatorFunctions, $element, validators) => {
168168
}
169169

170170
const isMarkedForDestroy = ($element) => {
171-
const elementName = $element.attr('name')
171+
const element = $element[0]
172+
const elementName = element.name
172173

173174
if (/\[([^\]]*?)\]$/.test(elementName)) {
174175
const destroyInputName = elementName.replace(/\[([^\]]*?)\]$/, '[_destroy]')
176+
const destroyInputElement = document.querySelector(`input[name="${destroyInputName}"]`)
175177

176-
if (jQuery(`input[name="${destroyInputName}"]`).val() === '1') {
178+
if (destroyInputElement && destroyInputElement.value === '1') {
177179
return true
178180
}
179181
}

vendor/assets/javascripts/rails.validations.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -665,10 +665,12 @@
665665
return true;
666666
};
667667
var isMarkedForDestroy = function isMarkedForDestroy($element) {
668-
var elementName = $element.attr('name');
668+
var element = $element[0];
669+
var elementName = element.name;
669670
if (/\[([^\]]*?)\]$/.test(elementName)) {
670671
var destroyInputName = elementName.replace(/\[([^\]]*?)\]$/, '[_destroy]');
671-
if (jQuery("input[name=\"".concat(destroyInputName, "\"]")).val() === '1') {
672+
var destroyInputElement = document.querySelector("input[name=\"".concat(destroyInputName, "\"]"));
673+
if (destroyInputElement && destroyInputElement.value === '1') {
672674
return true;
673675
}
674676
}

0 commit comments

Comments
 (0)