aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/lib/active_record/relation/query_methods.rb
diff options
context:
space:
mode:
authorSantiago Pastorino <santiago@wyeworks.com>2010-06-24 20:36:01 -0300
committerSantiago Pastorino <santiago@wyeworks.com>2010-06-25 15:06:32 -0300
commit3d8ccb924084ecd341b2d9644e6e0b66903d8432 (patch)
tree0c2a92c6c2504e033a64c9bc272600cadef4dc3a /activerecord/lib/active_record/relation/query_methods.rb
parent82dc16b7d2b6313117a42f7bf69a097b5ab79d1f (diff)
downloadrails-3d8ccb924084ecd341b2d9644e6e0b66903d8432.tar.gz
rails-3d8ccb924084ecd341b2d9644e6e0b66903d8432.tar.bz2
rails-3d8ccb924084ecd341b2d9644e6e0b66903d8432.zip
Makes a build_select for the select part of build_arel
Diffstat (limited to 'activerecord/lib/active_record/relation/query_methods.rb')
-rw-r--r--activerecord/lib/active_record/relation/query_methods.rb23
1 files changed, 13 insertions, 10 deletions
diff --git a/activerecord/lib/active_record/relation/query_methods.rb b/activerecord/lib/active_record/relation/query_methods.rb
index adc56fbef0..1e570a569b 100644
--- a/activerecord/lib/active_record/relation/query_methods.rb
+++ b/activerecord/lib/active_record/relation/query_methods.rb
@@ -137,16 +137,7 @@ module ActiveRecord
arel = arel.order(*@order_values.uniq.select{|o| o.present?}) if @order_values.present?
- selects = @select_values.uniq
-
- if selects.present?
- selects.each do |s|
- @implicit_readonly = false
- arel = arel.project(s) if s.present?
- end
- else
- arel = arel.project(@klass.quoted_table_name + '.*')
- end
+ arel = build_select(arel, @select_values.uniq)
arel = arel.from(@from_value) if @from_value.present?
@@ -219,6 +210,18 @@ module ActiveRecord
relation.join(custom_joins)
end
+ def build_select(arel, selects)
+ if selects.present?
+ @implicit_readonly = false
+ selects.each do |s|
+ arel = arel.project(s) if s.present?
+ end
+ else
+ arel = arel.project(@klass.quoted_table_name + '.*')
+ end
+ arel
+ end
+
def apply_modules(modules)
values = Array.wrap(modules)
@extensions += values if values.present?