Opened 11 years ago

Last modified 11 years ago

#12761 closed enhancement

ItemFileReadStore is not read-only — at Initial Version

Reported by: bht Owned by: Jared Jurkiewicz
Priority: high Milestone: 1.6
Component: Data Version: 1.6.0
Keywords: Cc:
Blocked By: Blocking:

Description

http://docs.dojocampus.org/dojo/data/ItemFileReadStore

For data, the documentation contains a disclaimer:

"The data object passed in is directly used and modified. If you want to reuse a data object, make sure you copy it before you pass it in."

However, with the effort required to use the implementation, one expects to deal with a MVC architecture that leaves the business objects alone, as other frameworks do.

With that I mean that the implementation expects a Model, with implementation specific requirements, which contains in it the Model Object.

It appears that dojo converts attributes of the model objects into arrays containing the attributes at index 0.

I think that the whole scenario is actually quite rude, to apply such bad practice and then add a disclaimer to the docs like, deal with it, we told you so.

So this issue is about changing the ItemFileReadStore? in such a way that it does not mess around with the model objects. If it really needs its own data to play with then it should create it.

Change History (0)

Note: See TracTickets for help on using tickets.