Display errors in dojox.grid.enhanced.plugins.Pagination

[Michael670]

short story: in row 940 of Pagination.js (function: _onNew) this.grid.rowCount should be changed to this.grid.get('rowCount').

long story: if new items are added to the store of the grid the property rowCount returns a value that is not valid, what results in a wrong page size for the paginted grid and display errors. The correct value would be invalidated.rowCount. The getter for rowCount returns always the correct value.

thx.

[Evan]

Could you post a test case for this?

It's safe for that change, but I don't quite understand the issue you mentioned, I didn't see any logic related with invalidated.rowCount in DataGrid?.get() so isn't grid.get('rowCount') the same as grid.rowCount? Anything I missed?

[Michael670]

Take a look at line 460 in _Grid.js:

_getRowCountAttr: function(){

return this.updating && this.invalidated && this.invalidated.rowCount != undefined ?

this.invalidated.rowCount : this.rowCount;

}

this method gets called if one use grid.get('rowCount') instead of grid.rowCount.

[Evan]

In [27451]:

Fixes #14506 by using grid.get('rowCount') instead of grid.rowCount, also applied to 1.7 branch, !strict