#5155 closed defect (fixed)
QueryReadStore does not implement dojo.data.api.Identity
| Reported by: | guest | Owned by: | Jared Jurkiewicz |
|---|---|---|---|
| Priority: | high | Milestone: | |
| Component: | DojoX Data | Version: | 1.0 |
| Keywords: | Identity QueryReadStore | Cc: | |
| Blocked By: | Blocking: |
Description
It looks like the QueryReadStore? has all the dojo.data.api.Identity methods implemented but it doesn't return dojo.data.api.Identity as a feature on getFeature()
Shouldn't getFeatures() be this?:
getFeatures: function()
{
return this._features;
},
Attachments (1)
Change History (6)
comment:1 Changed 12 years ago by
comment:2 Changed 12 years ago by
This should also have a UT test that verifies the features supported. That will also be added next week.
Changed 12 years ago by
| Attachment: | dojox.data.QueryReadStore_20071126.patch added |
|---|
Minor patch to fix return of getFeatures.
comment:3 Changed 12 years ago by
Tested on:
FireFox? 2.0.0.9 Safari B3 IE 6 IE 7 Firefox 1.5.0.12 Opera 9.2 Seamonkey 1.1.2
comment:4 Changed 12 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
(In [11631]) Minor fix to QueryReadStore?. fixes #5155
Note: See
TracTickets for help on using
tickets.
Yes, it should. Will check in a fix next week, when back from Thanksgiving vacation and dealing with relatives.