range values clamped by config'd min and max, to avoid solr error

Author: jrochkind

lib/blacklight_range_limit.rb

@@ -31,7 +31,9 @@ def self.default_range_config
         chart_segment_border_color: 'rgb(54, 162, 235)',
         chart_segment_bg_color: 'rgba(54, 162, 235, 0.5)',
         chart_aspect_ratio: 2,
-        assumed_boundaries: nil
+        assumed_boundaries: nil,
+        min_value: -2_147_483_648, # solr intfield min and max
+        max_value: 2_147_483_648
       },
       filter_class: BlacklightRangeLimit::FilterField,
       presenter: BlacklightRangeLimit::FacetFieldPresenter,

lib/blacklight_range_limit/range_limit_builder.rb

@@ -26,13 +26,7 @@ def add_range_limit_params(solr_params)
 
         selected_value = search_state.filter(config.key).values.first
 
-        range = if selected_value.is_a? Range
-          selected_value
-        elsif range_config[:assumed_boundaries]
-          Range.new(*range_config[:assumed_boundaries])
-        else
-          nil
-        end
+        range = bl_create_selected_range_value(selected_value, config)
 
         # If we have both ends of a range
         add_range_segments_to_solr!(solr_params, field_key, range.begin, range.end) if range && range.count != Float::INFINITY
@@ -98,5 +92,30 @@ def facet_value_to_fq_string(facet_field, value, use_local_params: true)
       end
     end
 
+    # @returns Range or nil
+    #
+    # Range created from a range value or from assumed boundaries if present, and clamped between min and max
+    def bl_create_selected_range_value(selected_value, field_config)
+      range_config = field_config.range_config
+
+      range = if selected_value.is_a? Range
+        selected_value
+      elsif range_config[:assumed_boundaries]
+        Range.new(*range_config[:assumed_boundaries])
+      else
+        nil
+      end
+
+      # clamp between config'd min and max
+      min = range_config[:min_value]
+      max = range_config[:max_value]
+
+      range = Range.new(
+        (range.begin.clamp(min, max) if range.begin),
+        (range.end.clamp(min, max) if range.end),
+      ) if range
+
+      range
+    end
   end
 end