Cast big decimals to floats due to Rails BigDecimal JSON serialization

Andrew Kane
1 parent b18c7b11
Showing 2 changed files with 29 additions and 0 deletions Show diff stats
lib/searchkick/model.rb
test/sql_test.rb
@@ -61,6 +61,28 @@ module Searchkick
             source[field] = source[field].map(&:to_f).reverse if source[field]
           end
  
+          # change all BigDecimal values to floats due to
+          # https://github.com/rails/rails/issues/6033
+          cast_big_decimal =
+            proc do |obj|
+              case obj
+              when BigDecimal
+                obj.to_f
+              when Hash
+                obj.each do |k, v|
+                  obj[k] = cast_big_decimal.call(v)
+                end
+              when Enumerable
+                obj.map! do |v|
+                  cast_big_decimal.call(v)
+                end
+              else
+                obj
+              end
+            end
+
+          cast_big_decimal.call(source)
+
           source.to_json
         end
  
@@ -155,6 +155,13 @@ class TestSql &lt; Minitest::Unit::TestCase
     assert_first "blue", "Blue B", fields: [:name, :color]
   end
  
+  def test_big_decimal
+    store [
+      {name: "Product", latitude: 100.0}
+    ]
+    assert_search "product", ["Product"], where: {latitude: {gt: 99}}
+  end
+
   # load
  
   def test_load_default
...	...	@@ -61,6 +61,28 @@ module Searchkick
61	61	source[field] = source[field].map(&:to_f).reverse if source[field]
62	62	end
63	63
	64	+ # change all BigDecimal values to floats due to
	65	+ # https://github.com/rails/rails/issues/6033
	66	+ cast_big_decimal =
	67	+ proc do \|obj\|
	68	+ case obj
	69	+ when BigDecimal
	70	+ obj.to_f
	71	+ when Hash
	72	+ obj.each do \|k, v\|
	73	+ obj[k] = cast_big_decimal.call(v)
	74	+ end
	75	+ when Enumerable
	76	+ obj.map! do \|v\|
	77	+ cast_big_decimal.call(v)
	78	+ end
	79	+ else
	80	+ obj
	81	+ end
	82	+ end
	83	+
	84	+ cast_big_decimal.call(source)
	85	+
64	86	source.to_json
65	87	end
66	88
...	...
...	...	@@ -155,6 +155,13 @@ class TestSql < Minitest::Unit::TestCase
155	155	assert_first "blue", "Blue B", fields: [:name, :color]
156	156	end
157	157
	158	+ def test_big_decimal
	159	+ store [
	160	+ {name: "Product", latitude: 100.0}
	161	+ ]
	162	+ assert_search "product", ["Product"], where: {latitude: {gt: 99}}
	163	+ end
	164	+
158	165	# load
159	166
160	167	def test_load_default
...	...