Handle numRows sent by the server

[wolfram]

for making paging easier using the queryreadstore numRows can now be sent by the server, which the QueryReadStore? will store and provide as a property.

[jaredj]

I don't think this patch really helps solve the issue noted and it introduces another bug. Namely, it will fail if you have two different widgets doing queries against the same store. Different queries will have different # of rows returns. so, this introduces a timing window problem. Will propose an alternate patch which I think would be better.

[jaredj]

In reality, this should be passing back the data of #numRows as the parameter to the onBegin callback, which is explicitly defined as the size of match set, not just the current page size.

[jaredj]

Updtaed patch for handling numRows...

[jaredj]

Wolfram,

I've attached an updated patch which handles returning total # of rows in the manner defined by the dojo.data APIs (through the onBegin function), plus unit tests for it. Please take a look and tell me if you are also happy with this patch (as it's your store, effectively).

I've tested it on IE 6, Firefox 2.0.0.11, and Safari B3.

If you're satisfied with this, I'll commit it.

[wolfram]

Jaredj, this looks perfect and it definitely makes more sense then my suggestion, thank you! go ahead and commit it please! I am just a bit short in time lately, gotta finish a project until the end of the year :-(

[jaredj]

That's fine. I just wanted to clear it by you, since you are the primary author of it. I'll commit it in a few minutes. Thanks for all your work on this store, by the way. It is very appreciated.

[jaredj]

(In [11711]) Minor update for QueryReadStore?. fixes #5270