Add SimpleStore class. #386
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR proposes to add a
SimpleStore
class, which is an intermediate class betweenContainer
andDataStore
.DataStore
stores records in instances ofLinkedCollection
and adds ES5 getters/setters to record classes for automatically linking relations upon insertion into the store as well as when foreign keys change, whileSimpleStore
just stores records in instances ofCollection
and does not add the ES5 getters/setters.SimpleStore
is perhaps easier to understand, but does not come with the in-memory relation linking features.This change does not affect
DataStore
in any way other than it now extendsSimpleStore
.For example, let's say
company
has manymember
andmember
belongs tocompany
:With
DataStore
:With
SimpleStore
:I welcome any thoughts or feedback.