PDA

View Full Version : [FIXED-1480] Ext.ux.grid.LockingGridView does not include scope in getColumnData.



mankz
2 Jan 2011, 11:04 AM
getColumnData should include scope config for each column as done by the standard gridview.

LockingGridView version


getColumnData : function(){
var cs = [], cm = this.cm, colCount = cm.getColumnCount();
for(var i = 0; i < colCount; i++){
var name = cm.getDataIndex(i);
cs[i] = {
name : (!Ext.isDefined(name) ? this.ds.fields.get(i).name : name),
renderer : cm.getRenderer(i),
id : cm.getColumnId(i),
style : this.getColumnStyle(i),
locked : cm.isLocked(i)
};
}
return cs;
},



Regular gridview version:


getColumnData : function() {
var columns = [],
colModel = this.cm,
colCount = colModel.getColumnCount(),
fields = this.ds.fields,
i, name;

for (i = 0; i < colCount; i++) {
name = colModel.getDataIndex(i);

columns[i] = {
name : Ext.isDefined(name) ? name : (fields.get(i) ? fields.get(i).name : undefined),
renderer: colModel.getRenderer(i),
scope : colModel.getRendererScope(i),
id : colModel.getColumnId(i),
style : this.getColumnStyle(i)
};
}

return columns;
},

meroy
4 Jan 2011, 11:39 AM
Fixed in SVN (3.3.x branch). Thanks for the report.