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

[#264] Fix for #264 from ckan-updates branch.

Browse Source 
Uses _.defaults rather than performing an explicit
deep copy.
This commit is contained in:
John Glover
2012-12-10 16:57:44 +01:00
parent 7f35cffe3c
commit 9bfbce776b
1 changed files with 1 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/model.js
View File

@@ -499,11 +499,7 @@ my.Query = Backbone.Model.extend({
var ourfilter = JSON.parse(JSON.stringify(filter)); var ourfilter = JSON.parse(JSON.stringify(filter));
// not fully specified so use template and over-write // not fully specified so use template and over-write
if (_.keys(filter).length <= 3) { if (_.keys(filter).length <= 3) {
ourfilter = _.extend( ourfilter = _.defaults(ourfilter, this._filterTemplates[filter.type]);
// crude deep copy
JSON.parse(JSON.stringify(this._filterTemplates[filter.type])),
ourfilter
);
} }
var filters = this.get('filters'); var filters = this.get('filters');
filters.push(ourfilter); filters.push(ourfilter);