Merge pull request #10767 from jmondo/master

Use grep instead of select with === in QueryMethods
2013-05-29 05:05:54 -07:00 · 2013-05-29 05:05:54 -07:00 · c2f3efffd9
commit c2f3efffd9
parent e1b59a268b c3ec0dbdd4
1 changed files with 1 additions and 1 deletions
--- a/activerecord/lib/active_record/relation/query_methods.rb
+++ b/activerecord/lib/active_record/relation/query_methods.rb
@ -1010,7 +1010,7 @@ def build_order(arel)
    end

    def validate_order_args(args)
-      args.select { |a| Hash === a  }.each do |h|
+      args.grep(Hash) do |h|
        unless (h.values - [:asc, :desc]).empty?
          raise ArgumentError, 'Direction should be :asc or :desc'
        end