Removed mapping of id to _id with order option [skip ci]

Andrew Kane
1 parent 9623c51e
Showing 3 changed files with 2 additions and 19 deletions Show diff stats
CHANGELOG.md
lib/searchkick/query.rb
test/order_test.rb
@@ -4,6 +4,7 @@
 - Raise error when `search` called on relations
 - Raise `ArgumentError` (instead of warning) for invalid regular expression modifiers
 - Raise `ArgumentError` instead of `RuntimeError` for unknown operators
+- Removed mapping of `id` to `_id` with `order` option
 - Dropped support for Ruby < 2.6 and Active Record < 5.2
 - Dropped support for NoBrainer and Cequel
 - Dropped support for `faraday_middleware-aws-signers-v4` (use `faraday_middleware-aws-sigv4` instead)
@@ -873,11 +873,7 @@ module Searchkick
     end
     def set_order(payload)
-      order = options[:order].is_a?(Enumerable) ? options[:order] : {options[:order] => :asc}
-      id_field = :_id
-      # TODO no longer map id to _id in Searchkick 5
-      # since sorting on _id is deprecated in Elasticsearch
-      payload[:sort] = order.is_a?(Array) ? order : Hash[order.map { |k, v| [k.to_s == "id" ? id_field : k, v] }]
+      payload[:sort] = options[:order].is_a?(Enumerable) ? options[:order] : {options[:order] => :asc}
     end
     def where_filters(where)
@@ -11,20 +11,6 @@ class OrderTest &lt; Minitest::Test
     assert_order "product", ["Product A", "Product B", "Product C", "Product D"], order: "name"
   end
-  # TODO no longer map id to _id in Searchkick 5
-  # since sorting on _id is deprecated in Elasticsearch
-  def test_id
-    store_names ["Product A", "Product B"]
-    product_a = Product.where(name: "Product A").first
-    product_b = Product.where(name: "Product B").first
-    _, stderr = capture_io do
-      assert_order "product", [product_a, product_b].sort_by { |r| r.id.to_s }.map(&:name), order: {id: :asc}
-    end
-    unless Searchkick.server_below?("7.6.0")
-      assert_match "Loading the fielddata on the _id field is deprecated", stderr
-    end
-  end
-
   def test_multiple
     store [
       {name: "Product A", color: "blue", store_id: 1},
	@@ -11,20 +11,6 @@ class OrderTest < Minitest::Test		@@ -11,20 +11,6 @@ class OrderTest < Minitest::Test
11	assert_order "product", ["Product A", "Product B", "Product C", "Product D"], order: "name"	11	assert_order "product", ["Product A", "Product B", "Product C", "Product D"], order: "name"
12	end	12	end
13		13
14	- # TODO no longer map id to _id in Searchkick 5
15	- # since sorting on _id is deprecated in Elasticsearch
16	- def test_id
17	- store_names ["Product A", "Product B"]
18	- product_a = Product.where(name: "Product A").first
19	- product_b = Product.where(name: "Product B").first
20	- _, stderr = capture_io do
21	- assert_order "product", [product_a, product_b].sort_by { \|r\| r.id.to_s }.map(&:name), order: {id: :asc}
22	- end
23	- unless Searchkick.server_below?("7.6.0")
24	- assert_match "Loading the fielddata on the _id field is deprecated", stderr
25	- end
26	- end
27	-
28	def test_multiple	14	def test_multiple
29	store [	15	store [
30	{name: "Product A", color: "blue", store_id: 1},	16	{name: "Product A", color: "blue", store_id: 1},