Skip to content

Commit 1288dea

Browse files
author
George Papanikolaou
committed
Fix existing methods override
Fix issue elastic#924 more: elastic#936
1 parent 5acfc78 commit 1288dea

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

elasticsearch-model/lib/elasticsearch/model.rb

+1-1
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@ def self.included(base)
112112
# Delegate common methods to the `__elasticsearch__` ClassMethodsProxy, unless they are defined already
113113
class << self
114114
METHODS.each do |method|
115-
delegate method, to: :__elasticsearch__ unless self.respond_to?(method)
115+
delegate method, to: :__elasticsearch__ unless self.public_instance_methods.include?(method)
116116
end
117117
end
118118
end

elasticsearch-model/spec/elasticsearch/model/module_spec.rb

+1
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,7 @@ def self.search(query, options={})
4848
end
4949

5050
DummyIncludingModel.__send__ :include, Elasticsearch::Model
51+
DummyIncludingModelWithSearchMethodDefined.__send__ :include, Elasticsearch::Model
5152
end
5253

5354
after(:all) do

0 commit comments

Comments
 (0)