Modern Tree Error When Removing Node Child - WORKAROUND REQUIRED PLEASE!

[gwyn_carwardine]

I raised this bug some time ago
https://fiddle.sencha.com/#fiddle/2k99

It's rather problematic as it occurs in a few different use cases - the fiddle just demonstrates one of them

It's recorded as bug EXTJS-27428 and it's quite serious but the "Remains of Sencha" have not seen fit to fix it yet - and possibly never will.

Any of the Sencha experts on here have a suitable workaround please?

[gwyn_carwardine]

Including this override seems to work but I have no idea what other implications it has

Ext.define(null, {
override: 'Ext.grid.cell.Tree',


privates: {
syncIndent: function () {
var me = this,
column = me.getColumn(),
indentSize, record, depth;
if (column) {
indentSize = column._indentSize;
record = me.getRecord();
if (!indentSize) {
column._indentSize = indentSize = parseInt(me.el.getStyle('background-position'), 10);
}
// replace this
// if (record) {
// with
if (record && record.getTreeStore()) {
depth = record.getTreeStore().rootVisible ? record.data.depth : record.data.depth - 1;

me.indentElement.dom.style.width = (depth * indentSize) + 'px';
}
}
},
}
});

[firefoxSafari]

Is explicitly removing the node from the tree store an option?

[gwyn_carwardine]

I tried that and whilst it solved that particular use case it didn't solve another one!

[evant]

Checking if the store exists or not is reasonable. The grid row is being returned to the pool of rows. So even if a record is attached at that point, assuming that it also has a related store is questionable.

[lovelyhibabe]

Ext.define(null, {  

    override: 'Ext.grid.cell.Tree',

    privates: {
        syncIndent: function () {
            var me = this,
                record = me.getRecord();
            if (!record || record.getTreeStore()) {
                me.callParent(arguments);
            }
        }
    }
});