Skip to content

Commit

Permalink
[#34,view][s]: query state in url (store into and read from).
Browse files Browse the repository at this point in the history
* Also change so that query values are not automatically JSON.stringify-ied (bad idea since it leads to repeated jsonification of same value ...)
  • Loading branch information
rufuspollock committed Mar 14, 2012
1 parent 58c06e5 commit 733a9d3
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 6 deletions.
2 changes: 1 addition & 1 deletion src/model.js
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ my.Dataset = Backbone.Model.extend({
query: function(queryObj) {
this.trigger('query:start');
var self = this;
this.queryState.set(queryObj, {silent: true});
this.queryState.set(queryObj);
var dfd = $.Deferred();
this.backend.query(this, this.queryState.toJSON()).done(function(rows) {
var docs = _.map(rows, function(row) {
Expand Down
2 changes: 1 addition & 1 deletion src/view-flot-graph.js
Original file line number Diff line number Diff line change
Expand Up @@ -123,7 +123,7 @@ my.FlotGraph = Backbone.View.extend({
this._getEditorData();
// update navigation
var qs = my.parseHashQueryString();
qs['graph'] = this.chartConfig;
qs['graph'] = JSON.stringify(this.chartConfig);
my.setHashQueryString(qs);
this.redraw();
},
Expand Down
13 changes: 10 additions & 3 deletions src/view.js
Original file line number Diff line number Diff line change
Expand Up @@ -111,6 +111,10 @@ my.DataExplorer = Backbone.View.extend({
my.clearNotifications();
self.el.find('.doc-count').text(self.model.docCount || 'Unknown');
my.notify('Data loaded', {category: 'success'});
// update navigation
var qs = my.parseHashQueryString();
qs['reclineQuery'] = JSON.stringify(self.model.queryState.toJSON());
my.setHashQueryString(qs);
});
this.model.bind('query:fail', function(error) {
my.clearNotifications();
Expand All @@ -134,8 +138,11 @@ my.DataExplorer = Backbone.View.extend({
// note this.model and dataset returned are the same
this.model.fetch()
.done(function(dataset) {
self.el.find('.doc-count').text(self.model.docCount || 'Unknown');
self.model.query();
var queryState = my.parseHashQueryString().reclineQuery;
if (queryState) {
queryState = JSON.parse(queryState);
}
self.model.query(queryState);
})
.fail(function(error) {
my.notify(error.message, {category: 'error', persist: true});
Expand Down Expand Up @@ -296,7 +303,7 @@ my.composeQueryString = function(queryParams) {
var queryString = '?';
var items = [];
$.each(queryParams, function(key, value) {
items.push(key + '=' + JSON.stringify(value));
items.push(key + '=' + value);
});
queryString += items.join('&');
return queryString;
Expand Down
2 changes: 1 addition & 1 deletion test/util.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ test('composeQueryString', function () {
a: 'b'
};
var out = recline.View.composeQueryString(params);
equal(out, '?x="y"&a="b"');
equal(out, '?x=y&a=b');
});

})(this.jQuery);

0 comments on commit 733a9d3

Please sign in to comment.