diff options
| author | Sean Griffin <sean@seantheprogrammer.com> | 2016-08-31 11:49:36 -0400 |
|---|---|---|
| committer | Sean Griffin <sean@seantheprogrammer.com> | 2016-08-31 11:50:53 -0400 |
| commit | 03d3f036a71d433c661d167596989ae6896e911c (patch) | |
| tree | e24867cde3a00b985bfaeb94cf8215cedb65cd33 /activerecord/lib/active_record/relation | |
| parent | 4ba40bc2d6c31c1f5c38fe079cc6aaa2ef7ffa33 (diff) | |
| download | rails-03d3f036a71d433c661d167596989ae6896e911c.tar.gz rails-03d3f036a71d433c661d167596989ae6896e911c.tar.bz2 rails-03d3f036a71d433c661d167596989ae6896e911c.zip | |
Override `respond_to_missing?` instead of `respond_to?` when possible
This was almost every case where we are overriding `respond_to?` in a
way that mirrors a parallel implementation of `method_missing`. There is
one remaining case in Active Model that should probably do the same
thing, but had a sufficiently strange implementation that I want to
investigate it separately.
Fixes #26333.
Diffstat (limited to 'activerecord/lib/active_record/relation')
| -rw-r--r-- | activerecord/lib/active_record/relation/delegation.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/activerecord/lib/active_record/relation/delegation.rb b/activerecord/lib/active_record/relation/delegation.rb index e1c36982dd..d16de4b06c 100644 --- a/activerecord/lib/active_record/relation/delegation.rb +++ b/activerecord/lib/active_record/relation/delegation.rb @@ -108,7 +108,7 @@ module ActiveRecord end end - def respond_to?(method, include_private = false) + def respond_to_missing?(method, include_private = false) super || @klass.respond_to?(method, include_private) || arel.respond_to?(method, include_private) end |