[FIXED][2.1] Bug when a ComboBox is used as an EditorGridPanel editor

[mdissel]

The problem seems to be also in the onViewClick .. when using tab to exit the field will always select the record from the view, even if the this.el.dom.value is empty..

I think the up / down key on the view should set the this.el.dom.value with the selected value, then it's easier in the doForce to check for emptyText / allowBlank..

Thanks
Marco

(dup post of http://extjs.com/forum/showthread.php?t=23850)

[crungmungus]

Any progress????

[MaxT]

Original thread:

http://extjs.com/forum/showthread.php?t=39409

The basis is that when typeahead is enabled, the Combo's Store is filtered to only expose those items which match.

If the Combo is blurred, and therefore removed in that state, the filter is not cleared.

So next time the Combo is made visible in the document, and has its value set to the chosen cell value, the Store is still filtered and the value will not be found, so it gets used as the raw value.

A fix could be changing setValue so that it uses the store unfiltered:

Ext.override(Ext.ComboBox, {
    setValue : function(v){
        var text = v;
        if(this.valueField){
            this.store.clearFilter();
            var r = this.findRecord(this.valueField, v);
            if(r){
                text = r.data[this.displayField];
            }else if(this.valueNotFoundText !== undefined){
                text = this.valueNotFoundText;
            }
        }
        this.lastSelectionText = text;
        if(this.hiddenField){
            this.hiddenField.value = v;
        }
        Ext.form.ComboBox.superclass.setValue.call(this, text);
        this.value = v;
    }
});

I think I've just run into this as well. In my case the combo doesn't have a valueField. So maybe the clearFilter() should be before the if test?

Max

Ext.override(Ext.ComboBox, {
    setValue : function(v){
        var text = v;
        this.store.clearFilter();
        if(this.valueField){
            var r = this.findRecord(this.valueField, v);
            if(r){
                text = r.data[this.displayField];
            }else if(this.valueNotFoundText !== undefined){
                text = this.valueNotFoundText;
            }
        }
        this.lastSelectionText = text;
        if(this.hiddenField){
            this.hiddenField.value = v;
        }
        Ext.form.ComboBox.superclass.setValue.call(this, text);
        this.value = v;
    }
});

[meprakashg@gmail.com]

Solution:

For Combo Problem in Grid, override blur action like below code
Some partial code is:-
...........
new Ext.form.ComboBox({
typeAhead: true,
triggerAction: 'all',
transform:'light',
lazyRender:true,
listClass: 'x-combo-list-small',
forceSelection:true,
blur: function(){this.store.clearFilter()}
}),
renderer: function(v) {
var s = cm.getColumnById('light').editor.field.store;
var i = s.find('value', v);
return s.getAt(i).get('text');
}................

[mystix]

fixed by @evant in SVN rev 5233:
http://www.sencha.com/forum/showthread.php?p=379997