Skip to content

Commit 9dd5ba3

Browse files
miguelffkarmi
authored andcommitted
[MODEL] Do not require models to respond to ancestors
Related: elastic#10, elastic#30, elastic#50, elastic#129, elastic#346
1 parent 9ae94f2 commit 9dd5ba3

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

elasticsearch-model/lib/elasticsearch/model/adapters/active_record.rb

+1-1
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ module Adapter
77
module ActiveRecord
88

99
Adapter.register self,
10-
lambda { |klass| !!defined?(::ActiveRecord::Base) && klass.ancestors.include?(::ActiveRecord::Base) }
10+
lambda { |klass| !!defined?(::ActiveRecord::Base) && klass.respond_to?(:ancestors) && klass.ancestors.include?(::ActiveRecord::Base) }
1111

1212
module Records
1313
# Returns an `ActiveRecord::Relation` instance

elasticsearch-model/lib/elasticsearch/model/adapters/mongoid.rb

+1-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ module Adapter
99
module Mongoid
1010

1111
Adapter.register self,
12-
lambda { |klass| !!defined?(::Mongoid::Document) && klass.ancestors.include?(::Mongoid::Document) }
12+
lambda { |klass| !!defined?(::Mongoid::Document) && klass.respond_to?(:ancestors) && klass.ancestors.include?(::Mongoid::Document) }
1313

1414
module Records
1515

0 commit comments

Comments
 (0)