Ticket #2542 (closed defect: fixed)
[dojo.data] change datastores to use SimpleBaseStore
| Reported by: | skinner | Owned by: | jaredj |
|---|---|---|---|
| Priority: | normal | Milestone: | 1.0 |
| Component: | Data | Version: | 0.4.1 |
| Severity: | minor | Keywords: | |
| Cc: |
Description
We should convert a lot of the dojo.data datastores to inherit from dojo.data.core.SimpleBaseStore?. A number of our older datastore implementations were written back before SimpleBaseStore? existed. Now that we have SimpleBaseStore?, we should use it as a standard abstract superclass for most of those older datastores. The datastores implementations will be simpler and more consistent after the change.
Here's what that current inheritance hierarchy looks like:
+ OpmlStore extends nothing + SimpleBaseStore extends nothing + JsonItemStore extends SimpleBaseStore + RemoteStore extends nothing + CsvStore extends RemoteStore + DeliciousStore extends RemoteStore + YahooStore extends RemoteStore + RdfStore extends RemoteStore
Here's what I think a better inheritance hierarchy would look like:
+ SimpleBaseStore extends nothing
+ JsonItemStore extends SimpleBaseStore
+ OpmlStore extends SimpleBaseStore
+ CsvStore extends SimpleBaseStore
+ DeliciousStore extends SimpleBaseStore
+ YahooStore extends SimpleBaseStore
+ RemoteStore extends SimpleBaseStore
+ RdfStore extends RemoteStore
Attachments
Change History
Note: See
TracTickets for help on using
tickets.