X-Git-Url: http://git.roojs.org/?p=roojs1;a=blobdiff_plain;f=Roo%2Fform%2FComboBoxArray.js;h=75ce301d7f8710bf4b1e417c88312f6d2c71021d;hp=b920f13a79390a462932bb8fabe179ff0d102e41;hb=61bc45258e16a779856dd2ad0862630b489e4583;hpb=6982a57767b997d74dfd88e45de421634c42b049 diff --git a/Roo/form/ComboBoxArray.js b/Roo/form/ComboBoxArray.js index b920f13a79..75ce301d7f 100644 --- a/Roo/form/ComboBoxArray.js +++ b/Roo/form/ComboBoxArray.js @@ -7,7 +7,7 @@ /** * @class Roo.form.ComboBoxArray - * @extends Roo.form.ComboBox + * @extends Roo.form.TextField * A facebook style adder... for lists of email / people / countries etc... * pick multiple items from a combo box, and shows each one. * @@ -36,6 +36,24 @@ Roo.form.ComboBoxArray = function(config) { + this.addEvents({ + /** + * @event beforeremove + * Fires before remove the value from the list + * @param {Roo.form.ComboBoxArray} _self This combo box array + * @param {Roo.form.ComboBoxArray.Item} item removed item + */ + 'beforeremove' : true, + /** + * @event remove + * Fires when remove the value from the list + * @param {Roo.form.ComboBoxArray} _self This combo box array + * @param {Roo.form.ComboBoxArray.Item} item removed item + */ + 'remove' : true + + + }); Roo.form.ComboBoxArray.superclass.constructor.call(this, config); @@ -73,16 +91,19 @@ Roo.extend(Roo.form.ComboBoxArray, Roo.form.TextField, */ name : false, /** - * @cfg {String} name The hidden name of the field, often contains an comma seperated list of names + * @cfg {String} hiddenName The hidden name of the field, often contains an comma seperated list of names */ hiddenName : false, - + /** + * @cfg {String} seperator The value seperator normally ',' + */ + seperator : ',', // private the array of items that are displayed.. items : false, // private - the hidden field el. hiddenEl : false, - + // private - the filed el.. el : false, //validateValue : function() { return true; }, // all values are ok! @@ -97,10 +118,15 @@ Roo.extend(Roo.form.ComboBoxArray, Roo.form.TextField, // give fake names to child combo; this.combo.hiddenName = this.hiddenName ? (this.hiddenName+'-subcombo') : this.hiddenName; - this.combo.name = this.name? (this.name+'-subcombo') : this.name; + this.combo.name = this.name ? (this.name+'-subcombo') : this.name; this.combo = Roo.factory(this.combo, Roo.form); this.combo.onRender(ct, position); + if (typeof(this.combo.width) != 'undefined') { + this.combo.onResize(this.combo.width,0); + } + + this.combo.initEvents(); // assigned so form know we need to do this.. this.store = this.combo.store; @@ -185,7 +211,7 @@ Roo.extend(Roo.form.ComboBoxArray, Roo.form.TextField, return; // not sure if this is needed.. - this.combo.onResize(w,h); + //this.combo.onResize(w,h); if(typeof w != 'number'){ // we do not handle it!?!? @@ -211,9 +237,10 @@ Roo.extend(Roo.form.ComboBoxArray, Roo.form.TextField, addItem: function(rec) { - var idField = this.combo.valueField; + var valueField = this.combo.valueField; var displayField = this.combo.displayField; - if (this.items.indexOfKey(rec[idField]) > -1) { + + if (this.items.indexOfKey(rec[valueField]) > -1) { //console.log("GOT " + rec.data.id); return; } @@ -221,12 +248,12 @@ Roo.extend(Roo.form.ComboBoxArray, Roo.form.TextField, var x = new Roo.form.ComboBoxArray.Item({ //id : rec[this.idField], data : rec, - nameField : displayField , + displayField : displayField , tipField : displayField , cb : this }); // use the - this.items.add(rec[idField],x); + this.items.add(rec[valueField],x); // add it before the element.. this.updateHiddenEl(); x.render(this.outerWrap, this.wrap.dom); @@ -244,18 +271,20 @@ Roo.extend(Roo.form.ComboBoxArray, Roo.form.TextField, this.items.each(function(f) { ar.push(f.data[idField]); - }); - this.hiddenEl.dom.value = ar.join(','); + this.hiddenEl.dom.value = ar.join(this.seperator); this.validate(); }, reset : function() { - Roo.form.ComboBoxArray.superclass.reset.call(this); - this.items.each(function(f) { - f.remove(); + this.items.clear(); + + Roo.each(this.outerWrap.select('.x-cbarray-item', true).elements, function(el){ + el.remove(); }); + + this.el.dom.value = ''; if (this.hiddenEl) { this.hiddenEl.dom.value = ''; } @@ -267,42 +296,111 @@ Roo.extend(Roo.form.ComboBoxArray, Roo.form.TextField, }, setValue: function(v) // not a valid action - must use addItems.. { - - if (this.store.isLocal) { + this.reset(); + + if (this.store.isLocal && (typeof(v) == 'string')) { // then we can use the store to find the values.. // comma seperated at present.. this needs to allow JSON based encoding.. - this.hiddenField.setValue(v); + this.hiddenEl.value = v; var v_ar = []; - Roo.each(v.split(','), function(k) { - var li = this.store.query(this.store.valueField, k); + Roo.each(v.split(this.seperator), function(k) { + Roo.log("CHECK " + this.valueField + ',' + k); + var li = this.store.query(this.valueField, k); if (!li.length) { return; } - add = {}; - add[this.store.valueField] = k; - add[this.store.displayField] = li.item(0).data[this.store.displayField]; + var add = {}; + add[this.valueField] = k; + add[this.displayField] = li.item(0).data[this.displayField]; this.addItem(add); }, this) - - - + + } + if (typeof(v) == 'object' ) { + // then let's assume it's an array of objects.. + Roo.each(v, function(l) { + var add = l; + if (typeof(l) == 'string') { + add = {}; + add[this.valueField] = k; + add[this.displayField] = k + } + this.addItem(add); + }, this); + } }, setFromData: function(v) { - Roo.log(v); + // this recieves an object, if setValues is called. + this.reset(); + this.el.dom.value = v[this.displayField]; + this.hiddenEl.dom.value = v[this.valueField]; + if (typeof(v[this.valueField]) != 'string' || !v[this.valueField].length) { + return; + } + var kv = v[this.valueField]; + var dv = v[this.displayField]; + kv = typeof(kv) != 'string' ? '' : kv; + dv = typeof(dv) != 'string' ? '' : dv; + + + var keys = kv.split(this.seperator); + var display = dv.split(this.seperator); + for (var i = 0 ; i < keys.length; i++) { + add = {}; + add[this.valueField] = keys[i]; + add[this.displayField] = display[i]; + this.addItem(add); + } }, + /** + * Validates the combox array value + * @return {Boolean} True if the value is valid, else false + */ + validate : function(){ + if(this.disabled || this.validateValue(this.processValue(this.getValue()))){ + this.clearInvalid(); + return true; + } + return false; + }, validateValue : function(value){ return Roo.form.ComboBoxArray.superclass.validateValue.call(this, this.getValue()); + }, + + /*@ + * overide + * + */ + isDirty : function() { + if(this.disabled) { + return false; + } + + try { + var d = Roo.decode(String(this.originalValue)); + } catch (e) { + return String(this.getValue()) !== String(this.originalValue); + } + + var originalValue = []; + + for (var i = 0; i < d.length; i++){ + originalValue.push(d[i][this.valueField]); + } + + return String(this.getValue()) !== String(originalValue.join(this.seperator)); + } }); @@ -328,7 +426,7 @@ Roo.form.ComboBoxArray.Item = function(config) { Roo.extend(Roo.form.ComboBoxArray.Item, Roo.BoxComponent, { data : {}, cb: false, - nameField : false, + displayField : false, tipField : false, @@ -362,7 +460,7 @@ Roo.extend(Roo.form.ComboBoxArray.Item, Roo.BoxComponent, { this.el.child('div').dom.innerHTML = this.cb.renderer ? this.cb.renderer(this.data) : - String.format('{0}',this.data[this.nameField]); + String.format('{0}',this.data[this.displayField]); this.el.child('div').dom.setAttribute('qtip', @@ -375,12 +473,18 @@ Roo.extend(Roo.form.ComboBoxArray.Item, Roo.BoxComponent, { remove : function() { + if(this.cb.disabled){ + return; + } + + if(false !== this.cb.fireEvent('beforeremove', this.cb, this)){ + this.cb.items.remove(this); + this.el.child('img').un('click', this.remove, this); + this.el.remove(); + this.cb.updateHiddenEl(); + + this.cb.fireEvent('remove', this.cb, this); + } - this.cb.items.remove(this); - this.el.child('img').un('click', this.remove, this); - this.el.remove(); - this.cb.updateHiddenEl(); } - - }); \ No newline at end of file