github / searchkick

Download as
Browse Code ยป

Commit 917d0ef305e6d556477c34c6b1a9b403e2c9bf60

Authored by Andrew Kane
1 parent 13f47854
Exists in master and in 21 other branches allow_missing, conversions_period, conversions_v2, eager_load, find_in_batches, highlighted, index_name, lazy_index_name, less_deps, nested, reindex_async_threads, reindex_refactor, relation, relation_search, routing_queue, scope_attributes, search_as_you_type, synonyms_ci, test_language, threads, wildcard

Fix for load: false behavior - closes #156

Showing 4 changed files with 8 additions and 1 deletions   Show diff stats
lib/searchkick.rb
  Diff comments   View file @917d0ef
1 1 require "active_model"
2 2 require "patron"
3 3 require "elasticsearch"
  4 +require "hashie"
4 5 require "searchkick/version"
5 6 require "searchkick/index"
6 7 require "searchkick/reindex"
... ...
lib/searchkick/results.rb
  Diff comments   View file @917d0ef
... ... @@ -25,7 +25,7 @@ module Searchkick
25 25 hit_ids = hit_ids.map(&:to_s)
26 26 records.sort_by{|r| hit_ids.index(r.id.to_s) }
27 27 else
28   - hits
  28 + hits.map{|hit| Hashie::Mash.new(hit["_source"]) }
29 29 end
30 30 end
31 31 end
... ...
searchkick.gemspec
  Diff comments   View file @917d0ef
... ... @@ -21,6 +21,7 @@ Gem::Specification.new do |spec|
21 21 spec.add_dependency "activemodel"
22 22 spec.add_dependency "elasticsearch"
23 23 spec.add_dependency "patron" # persistent http connections for performance
  24 + spec.add_dependency "hashie"
24 25  
25 26 spec.add_development_dependency "bundler", "~> 1.3"
26 27 spec.add_development_dependency "rake"
... ...
test/sql_test.rb
  Diff comments   View file @917d0ef
... ... @@ -250,6 +250,11 @@ class TestSql < Minitest::Unit::TestCase
250 250 assert_kind_of Hash, Product.search("product", load: false).first
251 251 end
252 252  
  253 + def test_load_false_methods
  254 + store_names ["Product A"]
  255 + assert_equal "Product A", Product.search("product", load: false).first.name
  256 + end
  257 +
253 258 def test_load_false_with_include
254 259 store_names ["Product A"]
255 260 assert_kind_of Hash, Product.search("product", load: false, include: [:store]).first
... ...