clarify logic in toString()

hammerlab · Mar 2, 2015 · 219c244 · 219c244
1 parent 30cea65
commit 219c244
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/cycledash/static/js/QueryLanguage.js b/cycledash/static/js/QueryLanguage.js
@@ -140,10 +140,11 @@ function toString(parsedQuery) {
   // e.g. {filters:[{type: '<', filterValue:'10', columnName:'A'}]}
   if (parsedQuery.filters) {
     filters = filters.concat(parsedQuery.filters.map(f => {
-      if (f.filterValue !== null && f.filterValue !== undefined) {
-        return `${f.columnName} ${f.type} ${maybeQuote(f.filterValue)}`;
-      } else {
+      var type = f.type.toLowerCase();
+      if (type == 'null' || type == 'not null') {
         return `${f.columnName} IS ${f.type}`;
+      } else {
+        return `${f.columnName} ${f.type} ${maybeQuote(f.filterValue)}`;
       }
     }));
   }