Skip to content

Commit

Permalink
Remove useless select_values += select_values
Browse files Browse the repository at this point in the history
`select_values` is a local variable defined at previous line.
`select_values += select_values` is totally useless.
  • Loading branch information
kamipo committed Feb 26, 2017
1 parent 83f7f4f commit 97d46c1
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion activerecord/lib/active_record/relation/calculations.rb
Original file line number Diff line number Diff line change
Expand Up @@ -282,7 +282,6 @@ def execute_grouped_calculation(operation, column_name, distinct) #:nodoc:
operation,
distinct).as(aggregate_alias)
]
select_values += select_values unless having_clause.empty?

select_values.concat group_columns.map { |aliaz, field|
if field.respond_to?(:as)
Expand Down

0 comments on commit 97d46c1

Please sign in to comment.