diff options
| author | George Claghorn <george.claghorn@gmail.com> | 2018-01-29 15:56:53 -0500 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-01-29 15:56:53 -0500 |
| commit | ca1d13a1587330b370d737be9acdefababe027b0 (patch) | |
| tree | 9fae267a7a83292a3739054dfa11426378947e92 /activerecord/lib/active_record/relation/calculations.rb | |
| parent | ef83c41724e7d359b6f8ffd5e0918754823bedf3 (diff) | |
| parent | 2e8c8d60e7568f9fb82fbed26d86c45d3cf82232 (diff) | |
| download | rails-ca1d13a1587330b370d737be9acdefababe027b0.tar.gz rails-ca1d13a1587330b370d737be9acdefababe027b0.tar.bz2 rails-ca1d13a1587330b370d737be9acdefababe027b0.zip | |
Merge pull request #31821 from composerinteralia/extra-to_s
Avoid extra calls to to_s
Diffstat (limited to 'activerecord/lib/active_record/relation/calculations.rb')
| -rw-r--r-- | activerecord/lib/active_record/relation/calculations.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/activerecord/lib/active_record/relation/calculations.rb b/activerecord/lib/active_record/relation/calculations.rb index decd60c87f..dba2f33dd4 100644 --- a/activerecord/lib/active_record/relation/calculations.rb +++ b/activerecord/lib/active_record/relation/calculations.rb @@ -242,7 +242,7 @@ module ActiveRecord def aggregate_column(column_name) return column_name if Arel::Expressions === column_name - if @klass.has_attribute?(column_name.to_s) || @klass.attribute_alias?(column_name.to_s) + if @klass.has_attribute?(column_name) || @klass.attribute_alias?(column_name) @klass.arel_attribute(column_name) else Arel.sql(column_name == :all ? "*" : column_name.to_s) |