knight/datahub
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[#88,model/state][xs]: can just specify a url instead of full dataset object when restoring from state (easier and good for backwards compatability).

Browse Source
This commit is contained in:
Rufus Pollock
2012-04-16 15:16:28 +01:00
parent 53e099beda
commit a42840cdf3
1 changed files with 7 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
src/model.js
View File

@@ -150,17 +150,22 @@ my.Dataset = Backbone.Model.extend({
// <pre> // <pre>
// { // {
// backend: {backend type - i.e. value of dataset.backend.__type__} // backend: {backend type - i.e. value of dataset.backend.__type__}
// dataset: {result of dataset.toJSON()} // dataset: {dataset info needed for loading -- result of dataset.toJSON() would be sufficient but can be simpler }
// // convenience - if url provided and dataste not this be used as dataset url
// url: {dataset url}
// ... // ...
// } // }
my.Dataset.restore = function(state) { my.Dataset.restore = function(state) {
// hack-y - restoring a memory dataset does not mean much ... // hack-y - restoring a memory dataset does not mean much ...
var dataset = null; var dataset = null;
if (state.url && !state.dataset) {
state.dataset = {url: state.url};
}
if (state.backend === 'memory') { if (state.backend === 'memory') {
dataset = recline.Backend.createDataset( dataset = recline.Backend.createDataset(
[{stub: 'this is a stub dataset because we do not restore memory datasets'}], [{stub: 'this is a stub dataset because we do not restore memory datasets'}],
[], [],
state.dataset state.dataset // metadata
); );
} else { } else {
dataset = new recline.Model.Dataset( dataset = new recline.Model.Dataset(