aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/lib/active_record/associations/preloader
diff options
context:
space:
mode:
authorSean Griffin <sean@thoughtbot.com>2014-12-26 16:10:47 -0700
committerSean Griffin <sean@thoughtbot.com>2014-12-26 16:12:56 -0700
commitb98668decb9712f26118de57623fd15d7d28646d (patch)
tree7051a3854ea8da9a357566deb14c4a0daadf9d7c /activerecord/lib/active_record/associations/preloader
parent6d4a19cd503b39bdbefae7c1c527b24ad2e3a36a (diff)
downloadrails-b98668decb9712f26118de57623fd15d7d28646d.tar.gz
rails-b98668decb9712f26118de57623fd15d7d28646d.tar.bz2
rails-b98668decb9712f26118de57623fd15d7d28646d.zip
Go through normal `where` logic when preloading associations
This will allow eager type casting to take place as needed. There doesn't seem to be any particular reason that the `in` statement was forced for single values, and the commit message where it was introduced gives no context. See https://github.com/rails/rails/commit/d90b4e2615e8048fdeffc6dffe3246704adee01f
Diffstat (limited to 'activerecord/lib/active_record/associations/preloader')
-rw-r--r--activerecord/lib/active_record/associations/preloader/association.rb2
1 files changed, 1 insertions, 1 deletions
diff --git a/activerecord/lib/active_record/associations/preloader/association.rb b/activerecord/lib/active_record/associations/preloader/association.rb
index 7d6523dbc4..a6e1a24360 100644
--- a/activerecord/lib/active_record/associations/preloader/association.rb
+++ b/activerecord/lib/active_record/associations/preloader/association.rb
@@ -33,7 +33,7 @@ module ActiveRecord
end
def query_scope(ids)
- scope.where(association_key.in(ids))
+ scope.where(association_key_name => ids)
end
def table