Add nodeNames to all filters, as this causes serialization issues

NCEAS · Nov 21, 2019 · 8733075 · 8733075
1 parent f833baa
commit 8733075
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 4 deletions.
diff --git a/src/js/models/filters/ChoiceFilter.js b/src/js/models/filters/ChoiceFilter.js
@@ -14,7 +14,8 @@ define(['jquery', 'underscore', 'backbone', 'models/filters/Filter'],
       return _.extend(Filter.prototype.defaults(), {
         chooseMultiple: true,
         //@type {object} - A literal JS object with a "label" and "value" attribute
-        choices: []
+        choices: [],
+        nodeName: "choiceFilter"
       });
     },
 

diff --git a/src/js/models/filters/DateFilter.js b/src/js/models/filters/DateFilter.js
@@ -16,7 +16,8 @@ define(['jquery', 'underscore', 'backbone', 'models/filters/Filter'],
         max: (new Date()).getUTCFullYear(),
         rangeMin: 0,
         rangeMax: (new Date()).getUTCFullYear(),
-        matchSubstring: false
+        matchSubstring: false,
+        nodeName: "dateFilter"
       });
     },
 

diff --git a/src/js/models/filters/ToggleFilter.js b/src/js/models/filters/ToggleFilter.js
@@ -15,7 +15,8 @@ define(['jquery', 'underscore', 'backbone', 'models/filters/Filter'],
         trueLabel: "On",
         trueValue: null,
         falseLabel: "Off",
-        falseValue: null
+        falseValue: null,
+        nodeName: "toggleFilter"
       });
     },
 
@@ -76,7 +77,7 @@ define(['jquery', 'underscore', 'backbone', 'models/filters/Filter'],
 
             // Remove the node if it exists in the DOM already
             $(objectDOM).find(nodeName).remove();
-            
+
             // Don't serialize falsey or default values
             if((value || value === false) && value != this.defaults()[nodeName]){