diff options
author | Ryuta Kamizono <kamipo@gmail.com> | 2016-08-19 13:12:03 +0900 |
---|---|---|
committer | Ryuta Kamizono <kamipo@gmail.com> | 2016-08-19 13:12:03 +0900 |
commit | dea671b5e0369f2cd0e9f3c88e6eef7230176b0d (patch) | |
tree | 79adbd85c2b90aebffd6391b7cfb508e4bfde072 /activerecord/lib/active_record/associations/collection_association.rb | |
parent | b27acd0b69a093e1d9d91ddf52c9f6e1289e28e0 (diff) | |
download | rails-dea671b5e0369f2cd0e9f3c88e6eef7230176b0d.tar.gz rails-dea671b5e0369f2cd0e9f3c88e6eef7230176b0d.tar.bz2 rails-dea671b5e0369f2cd0e9f3c88e6eef7230176b0d.zip |
Remove unnecessary `length` method for collection proxy
`length` is delegated to `records` (`load_target`) by
`ActiveRecord::Delegation`.
https://github.com/rails/rails/blob/v5.0.0/activerecord/lib/active_record/relation/delegation.rb#L38
Diffstat (limited to 'activerecord/lib/active_record/associations/collection_association.rb')
-rw-r--r-- | activerecord/lib/active_record/associations/collection_association.rb | 9 |
1 files changed, 0 insertions, 9 deletions
diff --git a/activerecord/lib/active_record/associations/collection_association.rb b/activerecord/lib/active_record/associations/collection_association.rb index e17ca81867..a02ec032a7 100644 --- a/activerecord/lib/active_record/associations/collection_association.rb +++ b/activerecord/lib/active_record/associations/collection_association.rb @@ -270,15 +270,6 @@ module ActiveRecord end end - # Returns the size of the collection calling +size+ on the target. - # - # If the collection has been already loaded +length+ and +size+ are - # equivalent. If not and you are going to need the records anyway this - # method will take one less query. Otherwise +size+ is more efficient. - def length - load_target.size - end - # Returns true if the collection is empty. # # If the collection has been loaded |