[CLOSED] Prevent paging toolbar from calling doLoad using beforechange and returning false

[jlosi]

Hi,

First of all, thank you for the sample to reproduce.

Well, we would prefer to leave the things as they are. Here is a related discussion on Sencha:
http://www.sencha.com/forum/showthread.php?82196

I would agree with @Evan to do not consider it a bug.

Ok, I see his point and agree that its not a bug. But it's also not completely desired behavior. Like he said, it should be up to the developer and I would add that the developer should be able to reset the page count manually on the client side without having to set start and limit to 0 and make a request to the server just to have the page count updated. The store is empty.. it should be considered if no data was loaded from the server at all; for example: first page load with autoload=false on the store and requiring an action before the store populates. it's like me just saying "RESET to defaults" like clearing a form.

[Daniil]

You can use the scripts suggested by @Baidaly.

Or just this trick after a removeAll call.

PagingToolbar1.store.totalLength = 0;
PagingToolbar1.onLoad(null, null, { params: { start: 0 }});

[jlosi]

I did a little bit of both:

Ext.ux.PageResetPlugin = function (config) {
    Ext.apply(this, config);
}


Ext.extend(Ext.ux.PageResetPlugin, Ext.PagingToolbar, {
    init: function (tb) {
        Ext.apply(tb, {
            onClear: function (s, r) {
                this.onLoad(s, r, s.lastOptions.params);
            },
            initResetEvents: function () {
                this.store.un('clear', this.onClear, this);
                this.store.on('clear', this.onClear, this);
            },
            getPageData: function () {
                var total = this.store.getCount() > 0 ? this.store.getTotalCount() : 0;
                return {
                    total: total,
                    activePage: Math.ceil((this.cursor + this.pageSize) / this.pageSize),
                    pages: total < this.pageSize ? 1 : Math.ceil(total / this.pageSize)
                };
            }
        });
        tb.initResetEvents();
    }
});


Ext.preg('pageResetPlugin', Ext.ux.PageResetPlugin);

Please advise if you think anything here might be problematic.

[Daniil]

Well, if you won't need to use a bindStore method, then, I think, your solution should work well.

Could you clarify why you "un" a clear listener before "on"?

[jlosi]

Well, if you won't need to use a bindStore method, then, I think, your solution should work well.

Could you clarify why you "un" a clear listener before "on"?

Oops.. that was not intentional. Thanks for catching that :)

[Daniil]

No problem:)

Can we mark the thread closed?

[jlosi]

No problem:)

Can we mark the thread closed?

Yes. Thanks for the assistance :D