Fix array/object mixup

This commit is contained in:
Dennis Eichhorn 2016-03-26 21:57:35 +01:00
parent 8b608d7d4c
commit 199d27f686

View File

@ -34,16 +34,21 @@
return this.ignore.indexOf(id) !== -1; return this.ignore.indexOf(id) !== -1;
}; };
jsOMS.UI.FormManager.prototype.unbind = function(id)
{
};
jsOMS.UI.FormManager.prototype.bind = function(id) jsOMS.UI.FormManager.prototype.bind = function(id)
{ {
if (typeof id !== 'undefined' && this.ignore.indexOf(id) === -1) { if (typeof id !== 'undefined' && typeof this.ignore[id] === 'undefined') {
this.bindForm(id) this.bindForm(id)
} else { } else {
let forms = document.getElementsByTagName('form'), let forms = document.getElementsByTagName('form'),
length = forms.length; length = forms.length;
for (var i = 0; i < length; i++) { for (var i = 0; i < length; i++) {
if (this.ignore.indexOf(forms[i].id) === -1) { if (typeof this.ignore[forms[i].id] === 'undefined') {
this.bindForm(forms[i].id); this.bindForm(forms[i].id);
} }
} }
@ -52,12 +57,16 @@
jsOMS.UI.FormManager.prototype.bindForm = function(id) jsOMS.UI.FormManager.prototype.bindForm = function(id)
{ {
if(typeof id === 'undefined' || !id) {
throw false;
}
let self = this; let self = this;
this.unbind(id); this.unbind(id);
if(this.ignore.indexOf(id) === -1) { if(typeof this.ignore[id] === 'undefined') {
this.forms[id] = new FormView(id); this.forms[id] = new jsOMS.Views.FormView(id);
} }
this.forms[id].getSubmit().addEventListener('click', function(event) { this.forms[id].getSubmit().addEventListener('click', function(event) {
@ -71,7 +80,7 @@
// todo: do i need the findex? can't i just use id? // todo: do i need the findex? can't i just use id?
let findex = 0; let findex = 0;
if((findex = this.forms.indexOf(id)) !== -1) { if((findex = this.forms[id]) !== 'undefined') {
this.forms[id].unbind(); this.forms[id].unbind();
this.forms.splice(findex, 1); this.forms.splice(findex, 1);