diff options
author | Pratik Naik <pratiknaik@gmail.com> | 2010-06-02 14:42:25 +0100 |
---|---|---|
committer | Pratik Naik <pratiknaik@gmail.com> | 2010-06-02 14:42:25 +0100 |
commit | 4b91daff13be43ed913a97ffc8ad1b3f77fd9690 (patch) | |
tree | 8fa53e49312c54f4d6e600c7505db55894e8c30a /activerecord/lib/active_record | |
parent | 0570720d667b5fe55aeebfba087064788d4d2ba4 (diff) | |
download | rails-4b91daff13be43ed913a97ffc8ad1b3f77fd9690.tar.gz rails-4b91daff13be43ed913a97ffc8ad1b3f77fd9690.tar.bz2 rails-4b91daff13be43ed913a97ffc8ad1b3f77fd9690.zip |
Special treatement for Relation#select { with block }
Diffstat (limited to 'activerecord/lib/active_record')
-rw-r--r-- | activerecord/lib/active_record/relation/query_methods.rb | 15 |
1 files changed, 14 insertions, 1 deletions
diff --git a/activerecord/lib/active_record/relation/query_methods.rb b/activerecord/lib/active_record/relation/query_methods.rb index 6782554854..7a48a6596a 100644 --- a/activerecord/lib/active_record/relation/query_methods.rb +++ b/activerecord/lib/active_record/relation/query_methods.rb @@ -9,7 +9,7 @@ module ActiveRecord (ActiveRecord::Relation::ASSOCIATION_METHODS + ActiveRecord::Relation::MULTI_VALUE_METHODS).each do |query_method| attr_accessor :"#{query_method}_values" - next if [:where, :having].include?(query_method) + next if [:where, :having, :select].include?(query_method) class_eval <<-CEVAL, __FILE__, __LINE__ + 1 def #{query_method}(*args, &block) new_relation = clone @@ -21,6 +21,19 @@ module ActiveRecord CEVAL end + class_eval <<-CEVAL, __FILE__, __LINE__ + 1 + def select(*args, &block) + if block_given? + to_a.select(&block) + else + new_relation = clone + value = Array.wrap(args.flatten).reject {|x| x.blank? } + new_relation.select_values += value if value.present? + new_relation + end + end + CEVAL + [:where, :having].each do |query_method| class_eval <<-CEVAL, __FILE__, __LINE__ + 1 def #{query_method}(*args, &block) |