use grep over select for consistency and efficiency

pass block directly to grep

activerecord/lib/active_record/relation/query_methods.rb

@@ -1015,7 +1015,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