fix clipped extents in #41

src/app/collections/city_buildings.js

@@ -53,8 +53,13 @@ define([
     if (!_.isNumber(_value)) {
       return false;
     }
-
-    return (_value >= range.min && _value <= range.max);
+    // Handle situations where there's only a min or only a max
+    if (range.min && range.max)
+      return (_value >= range.min && _value <= range.max);
+    if (range.min)
+      return (_value >= range.min);
+    if (range.max)
+      return (_value <= range.max);
   }
 
   function cityBuildingsFilterizer(buildings, categories, ranges) {
@@ -92,7 +97,13 @@ define([
 
   CityBuildingQuery.prototype.toRangeSql = function() {
     return _.map(this.ranges, function(range){
-      return range.field + " BETWEEN " + range.min + " AND " + range.max;
+      // Handle situations where there's only a min or only a max
+      if (range.min && range.max)
+        return range.field + " BETWEEN " + range.min + " AND " + range.max;
+      if (range.min)
+        return range.field + " >= " + range.min;
+      if (range.max)
+        return range.field + " <= " + range.max;
     });
   };
 

src/app/views/map/filter.js

@@ -143,7 +143,13 @@ define([
       filters = _.reject(filters, function(f){ return f.field == fieldName; });
 
       if (rangeSlider.from !== rangeSlider.min || rangeSlider.to !== rangeSlider.max){
-        filters.push({field: fieldName, min: rangeSlider.from, max: rangeSlider.to});
+        var newFilter = {field: fieldName};
+        // Only include min or max in the filter if it is different from the rangeSlider extent.
+        // This is important to the rangeSlider can clip the extreme values off, but we don't
+        // want to use the rangeSlider extents to filter the data on the map.
+        if (rangeSlider.from !== rangeSlider.min) newFilter.min = rangeSlider.from;
+        if (rangeSlider.to   !== rangeSlider.max) newFilter.max = rangeSlider.to;
+        filters.push(newFilter);
       }
 
       // fire event for other non Filter.js listeners