Avoid using respond_to? on a relation, since it memoizes an arel object

shioyama · May 27, 2017 · d3e9748 · d3e9748
1 parent fcdf5aa
commit d3e9748
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/lib/mobility/backend/active_record/query_methods.rb b/lib/mobility/backend/active_record/query_methods.rb
@@ -20,7 +20,7 @@ def initialize(attributes, **_)
         # @note Only want to define this once, even if multiple QueryMethods
         #   modules are included, so define it here in extended method
         def extended(relation)
-          unless relation.respond_to?(:mobility_where_chain)
+          unless relation.methods(false).include?(:mobility_where_chain)
             relation.define_singleton_method(:mobility_where_chain) do
               @mobility_where_chain ||= Class.new(::ActiveRecord::QueryMethods::WhereChain)
             end