[FIXED-811][3.1] TreeGrid discards loaders createNode implementation

[kristoph]

The TreeGrid implementation in svn has a bug (or perhaps it's a feature) around line 37 in TreeGrid.js. Also present in 3.1 I would guess.

The code as implemented always replaces the createNode in the loader which means, if a custom createNode was implemented for some reason, it is discarded.

Instead the code should look something like this ...



else if ( l )
{
var _createNode = l.createNode;
l.createNode = function( attr )
{
if ( !attr.uiProvider )
{
attr.uiProvider = Ext.ux.tree.TreeGridNodeUI;
}
return _createNode.call( this, attr );
}
}

[Condor]

I would also say that this is a bug (e.g. it makes it impossible to use an XmlTreeLoader in a TreeGrid).

I'll move this thread to the bugs section.

[MuratCorlu]

Is there any roadmap to solve this problem? Or, we must edit plugin code manually?

[evant]

I've changed this behaviour in SVN. If you pass in a loader instance it will use it without modification.