TEST_CODEOWNERS/rails
3
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Merge pull request #26394 from kamipo/remove_redundant_condition

Browse Source 
Remove redundant `!loaded?` condition
This commit is contained in:
Andrew White 2016-09-06 10:52:01 +01:00 committed by GitHub
commit 38fae35d53
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
activerecord/lib/active_record/associations/collection_association.rb
View File

@ -227,9 +227,9 @@ def size
else
target.size
end
elsif !loaded? && !association_scope.group_values.empty?
elsif !association_scope.group_values.empty?
load_target.size
elsif !loaded? && !association_scope.distinct_value && target.is_a?(Array)
elsif !association_scope.distinct_value && target.is_a?(Array)
unsaved_records = target.select(&:new_record?)
unsaved_records.size + count_records
else