diff options
| author | Ryuta Kamizono <kamipo@gmail.com> | 2018-01-10 07:08:42 +0900 |
|---|---|---|
| committer | Ryuta Kamizono <kamipo@gmail.com> | 2018-01-10 07:08:42 +0900 |
| commit | b8e5d4f9dffb35b7b0ca2f583cb1c9258a7d7cbe (patch) | |
| tree | 2de3b65e9ed7ac2a2d99847c4cb526a7c7953c4e /activerecord/lib/active_record/associations | |
| parent | d749b5d0e8a659f2018a3acdd0061ec6b57b7f7e (diff) | |
| download | rails-b8e5d4f9dffb35b7b0ca2f583cb1c9258a7d7cbe.tar.gz rails-b8e5d4f9dffb35b7b0ca2f583cb1c9258a7d7cbe.tar.bz2 rails-b8e5d4f9dffb35b7b0ca2f583cb1c9258a7d7cbe.zip | |
Bring back passing single record support for `Preloader`
I removed redundant `Array.wrap(records)` since `Preloader` is nodoc
class and Active Record always pass `records` as an array to
`Preloader`.
But if users relies on that behavior, it is not worth dropping its
behavior.
Fixes #31661.
Diffstat (limited to 'activerecord/lib/active_record/associations')
| -rw-r--r-- | activerecord/lib/active_record/associations/preloader.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/activerecord/lib/active_record/associations/preloader.rb b/activerecord/lib/active_record/associations/preloader.rb index e1087be9b3..59320431ee 100644 --- a/activerecord/lib/active_record/associations/preloader.rb +++ b/activerecord/lib/active_record/associations/preloader.rb @@ -83,7 +83,7 @@ module ActiveRecord # { author: :avatar } # [ :books, { author: :avatar } ] def preload(records, associations, preload_scope = nil) - records = records.compact + records = Array.wrap(records).compact if records.empty? [] |