Preprocess association query handling in predicate builder

Currently association query is handled as a postprocess. This has two
problems.

1. When `value` is a `Hash`, we need to skip the postprocess using
`next`.

2. `can_be_bound?` should return false if
`table.associated_with?(column_name)` is true (pass to the postprocess).

These are unneeded if preprocessing association query handling.

1 files changed, 8 insertions, 11 deletions

diff --git a/activerecord/lib/active_record/relation/predicate_builder.rb b/activerecord/lib/active_record/relation/predicate_builder.rb
index 18ae10a652..ec1861bd85 100644
--- a/activerecord/lib/active_record/relation/predicate_builder.rb
+++ b/activerecord/lib/active_record/relation/predicate_builder.rb
@@ -92,7 +92,14 @@ module ActiveRecord
             attrs, bvs = associated_predicate_builder(column_name).create_binds_for_hash(value)
             result[column_name] = attrs
             binds += bvs
-            next
+          when table.associated_with?(column_name)
+            # Find the foreign key when using queries such as:
+            # Post.where(author: author)
+            #
+            # For polymorphic relationships, find the foreign key and type:
+            # PriceEstimate.where(estimate_of: treasure)
+            result[column_name] = AssociationQueryHandler.value_for(table, column_name, value)
+            binds.concat(value.bound_attributes) if value.is_a?(Relation)
           when value.is_a?(Relation)
             binds += value.bound_attributes
           when value.is_a?(Range) && !table.type(column_name).respond_to?(:subtype)
@@ -114,15 +121,6 @@ module ActiveRecord
               binds << build_bind_param(column_name, value)
             end
           end
-
-          # Find the foreign key when using queries such as:
-          # Post.where(author: author)
-          #
-          # For polymorphic relationships, find the foreign key and type:
-          # PriceEstimate.where(estimate_of: treasure)
-          if table.associated_with?(column_name)
-            result[column_name] = AssociationQueryHandler.value_for(table, column_name, value)
-          end
         end
 
         [result, binds]
@@ -155,7 +153,6 @@ module ActiveRecord
       end
 
       def can_be_bound?(column_name, value)
-        return if table.associated_with?(column_name)
         case value
         when Array, Range
           table.type(column_name).respond_to?(:subtype)