[4.1.0] Treepanel LoadMask: cannot call method 'hasCls' of null

[zombeerose]

REQUIRED INFORMATION

Ext version tested:

• Ext 4.1.0 GA

Browser versions tested against:

• Chrome 18

DOCTYPE tested against:

• Strict

Description:

• Loading a tree will fail with an uncaught exception when a loadmask is defined for a treepanel.
• This currently relies upon an override to enable loadMask support for tree panels. See forum post.

Steps to reproduce the problem:

• Run the code.

The result that was expected:

• No error and a loadMask should appear when the tree is refreshed.

The result that occurs instead:

• The process dies.

Test Case:

//BUG: http://www.sencha.com/forum/showthread.php?198341-4.1-Loading-mask
Ext.define('Ext.view.AbstractViewOverride',{
    override: 'Ext.view.AbstractView',
    
    onRender: function() {
        var me = this;
        me.callOverridden(arguments);
        if (me.loadMask && Ext.isObject(me.panel.store)) {
            me.setMaskBind(me.panel.store);
        }
    }
});


    var store = Ext.create('Ext.data.TreeStore', {
        proxy: {
            type: 'ajax',
            url: '/includes/library/extjs/ext/examples/tree/get-nodes.php'
        },
        root: {
            text: 'Ext JS',
            id: 'src',
            expanded: true
        },
        folderSort: true,
        sorters: [{
            property: 'text',
            direction: 'ASC'
        }]
    });


    var tree = Ext.create('Ext.tree.Panel', {
        loadMask: true,
        store: store,
        renderTo: Ext.getBody(),
        height: 300,
        width: 250,
        title: 'Files',
        useArrows: true,
        dockedItems: [{
            xtype: 'toolbar',
            items: [{
                text: 'Refresh',
                handler: function(){
                    store.load();
                }
            }]
        }]
    });

HELPFUL INFORMATION

Additional CSS used:

• only default ext-all.css

Operating System:

• WinXP Pro

Debugging already done:

• In Ext.view.AbstractView#onMaskBeforeShow, the "all" collection is cleared before displaying the mask.
• The value of the "loading" field changes when the response is received, which triggers a call to Ext.view.Table#onUpdate.
• The "all" collection is used in Ext.view.Table#onUpdate to look up the oldRow reference but will return null because the collection was cleared when the mask was shown.

[mitchellsimoens]

Thanks for the report.