[FIXED] [2.3] Trunk r5130: PagingStore destroyStore Function Nulls Out data Causing clearData To Error Out

[ltctech]

PagingStore:

    destroyStore : function () {        
        this.data = this.allData = this.snapshot = null;
        this.callParent(arguments); //<-- data is null before calling
    },
    clearData : function (isLoad) {
        var me = this,
            records,
            i;

        if (me.allData) {
            me.data = me.allData;
            delete me.allData;
        }
        else if (me.snapshot) {
            me.data = me.snapshot;
            delete me.snapshot;
        }

        if (!me.buffered) {
            records = me.data.items; // <-- TypeError: me.data is null
            i = records.length;

            while (i--) {
                records[i].unjoin(me);
            }
        }

        this.data.clear();

        if (isLoad !== true || me.clearRemovedOnLoad) {
            me.removed.length = 0;
        }
    },

AbstractStore:

    destroyStore: function() {
        var me = this;

        if (!me.isDestroyed) {
            me.clearListeners();
            if (me.storeId) {
                Ext.data.StoreManager.unregister(me);
            }
            me.clearData(); // <-- this calls clearData with data being null
            me.data = me.tree = me.sorters = me.filters = me.groupers = null;
            if (me.reader) {
                me.reader.destroyReader();
            }
            me.proxy = me.reader = me.writer = null;
            me.isDestroyed = true;

            if (me.implicitModel) {
                Ext.destroy(me.model);
            } else {
                me.model = null;
            }
        }
    },

[Baidaly]

Hello!

I was able to reproduce. We are investigating!

Thank you!

[ltctech]

Maybe just reverse the two statements, call parent then set null?

I am using this workaround right now, calling super to bypass Ext.NET destroyStore:

Ext.data.PagingStore.override({
    destroyStore: function () {
        this.callSuper(arguments);
        this.data = this.allData = this.snapshot = null;
    }
});

[Daniil]

It has been fixed in SVN using your suggestion. Thank you!