Skip to content

Commit ed3088f

Browse files
committed
netteForms.js: used form.elements.namedItem(…) instead of form.elements[…] [Closes #72]
elements['item'] and elements['namedItem'] contains native methods
1 parent 4969582 commit ed3088f

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

src/assets/netteForms.js

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -60,8 +60,8 @@ Nette.getValue = function(elem) {
6060
}
6161
return multi ? res : null;
6262

63-
} else if (elem.name && !elem.form.elements[elem.name].tagName) { // multi element
64-
return Nette.getValue(elem.form.elements[elem.name]);
63+
} else if (elem.name && !elem.form.elements.namedItem(elem.name).tagName) { // multi element
64+
return Nette.getValue(elem.form.elements.namedItem(elem.name));
6565

6666
} else if (elem.type === 'file') {
6767
return elem.files || elem.value;
@@ -125,7 +125,7 @@ Nette.validateControl = function(elem, rules, onlyCheck, value) {
125125
for (var id = 0, len = rules.length; id < len; id++) {
126126
var rule = rules[id],
127127
op = rule.op.match(/(~)?([^?]+)/),
128-
curElem = rule.control ? elem.form.elements[rule.control] : elem;
128+
curElem = rule.control ? elem.form.elements.namedItem(rule.control) : elem;
129129

130130
if (!curElem) {
131131
continue;
@@ -156,7 +156,7 @@ Nette.validateControl = function(elem, rules, onlyCheck, value) {
156156
if (!onlyCheck) {
157157
var arr = Nette.isArray(rule.arg) ? rule.arg : [rule.arg],
158158
message = rule.msg.replace(/%(value|\d+)/g, function(foo, m) {
159-
return Nette.getValue(m === 'value' ? curElem : elem.form.elements[arr[m].control]);
159+
return Nette.getValue(m === 'value' ? curElem : elem.form.elements.namedItem(arr[m].control));
160160
});
161161
Nette.addError(curElem, message);
162162
}
@@ -215,7 +215,7 @@ Nette.validateForm = function(sender) {
215215
*/
216216
Nette.isDisabled = function(elem) {
217217
if (elem.type === 'radio') {
218-
elem = elem.form.elements[elem.name].tagName ? [elem] : elem.form.elements[elem.name];
218+
elem = elem.form.elements.namedItem(elem.name).tagName ? [elem] : elem.form.elements.namedItem(elem.name);
219219
for (var i = 0; i < elem.length; i++) {
220220
if (!elem[i].disabled) {
221221
return false;
@@ -245,7 +245,7 @@ Nette.addError = function(elem, message) {
245245
*/
246246
Nette.expandRuleArgument = function(form, arg) {
247247
if (arg && arg.control) {
248-
arg = Nette.getEffectiveValue(form.elements[arg.control]);
248+
arg = Nette.getEffectiveValue(form.elements.namedItem(arg.control));
249249
}
250250
return arg;
251251
};
@@ -442,7 +442,7 @@ Nette.toggleControl = function(elem, rules, success, firsttime, value) {
442442
for (var id = 0, len = rules.length; id < len; id++) {
443443
var rule = rules[id],
444444
op = rule.op.match(/(~)?([^?]+)/),
445-
curElem = rule.control ? elem.form.elements[rule.control] : elem;
445+
curElem = rule.control ? elem.form.elements.namedItem(rule.control) : elem;
446446

447447
if (!curElem) {
448448
continue;

0 commit comments

Comments
 (0)