Opened 12 years ago
Closed 12 years ago
#5270 closed defect (fixed)
Handle numRows sent by the server
| Reported by: | wolfram | Owned by: | Jared Jurkiewicz |
|---|---|---|---|
| Priority: | high | Milestone: | |
| Component: | DojoX Data | Version: | 1.0 |
| Keywords: | Cc: | ||
| Blocked By: | Blocking: |
Description
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.
Attachments (2)
Change History (8)
Changed 12 years ago by
| Attachment: | patch5270.diff added |
|---|
comment:1 Changed 12 years ago by
comment:2 Changed 12 years ago by
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.
Changed 12 years ago by
| Attachment: | dojox.data.QueryReadStore_20071205.patch added |
|---|
Updtaed patch for handling numRows...
comment:3 Changed 12 years ago by
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.
comment:4 Changed 12 years ago by
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 :-(
comment:5 Changed 12 years ago by
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.
comment:6 Changed 12 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
(In [11711]) Minor update for QueryReadStore?. fixes #5270
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.