Replace filter factory function with explicit targetFilter and
authorMike Taylor <mike@indexdata.com>
Thu, 24 Apr 2014 09:38:58 +0000 (10:38 +0100)
committerMike Taylor <mike@indexdata.com>
Thu, 24 Apr 2014 09:38:58 +0000 (10:38 +0100)
fieldFilter factories. These objects know their own types.
Currently that type is not used, but that will change.

src/mkws-filter.js

index f19edcb..c2bc711 100644 (file)
@@ -85,15 +85,21 @@ function filterSet(team) {
 }
 
 
-// Factory function for filters. These can be of several types.
-function filter(id, name, field, value) {
-    var res;
-
-    if (id) {
-       res = { id: id, name: name };
-    } else {
-       res = { field: field, value: value };
-    }
+// Factory functions for filters. These can be of several types.
+function targetFilter(id, name) {
+    return {
+        type: 'target',
+        id: id,
+        name: name
+    };
 
     return res;
 }
+
+function fieldFilter(field, value) {
+    return {
+        type: 'field',
+        field: field,
+        value: value
+    };
+}