neo/examples/stateProvider/Table.mjs invokes onStoreLoad _twice_ ??

[gplanansky]

Running neo/examples/stateProvider/Table.mjs invokes onStoreLoad twice, hence createViewData is invoked twice. Shouldn't this only load once?

https://github.com/neomjs/neo/blob/42029c4405b6a5c1189fb5ad641672db584eccc1c/src/table/Container.mjs#L476

    onStoreLoad(data) {
        let me = this;

        if (me.rendered) {
            me.createViewData();

neo/src/table/View.mjs

Line 325 in 42029c4

createViewData() {

    createViewData() {
        let me                    = this,
            {selectedRows, store} = me,
            countRecords          = store.getCount(),
            i                     = 0,
            rows                  = [];

        for (; i < countRecords; i++) {     /////////   this 
            rows.push(me.createRow({record: store.items[i], rowIndex: i}))
        }

[tobiu]

Hi @gplanansky,

Obviously it should not happen, although the vdom engine will catch it => no duplicate DOM manipulations.

I changed the store internally a bit:
https://github.com/neomjs/neo/blob/dev/src/data/Store.mjs#L435

• onConstructed() now triggers a slightly delayed load event, in case there is data.
• so, beforeSetStore() inside the table container no longer needs to to this as well.
• double-checked the grid, and it was already adjusted there.

[gplanansky]

Thanks. Debugging a data app's event-triggered reload logic, which I use a lot, gets confounded if core store / table have their own bug. I could see the trace going through beforeSetStore, but didn't sort it out.