diff options
author | Jack <aquajach@gmail.com> | 2016-07-29 19:00:45 +0800 |
---|---|---|
committer | Jack <aquajach@gmail.com> | 2016-07-29 19:00:45 +0800 |
commit | c0eb542e57fe6eaa959218c1a7026e53673ba0cb (patch) | |
tree | 88d2edbe8862b109a5e05744e0bef17f33f1ccef /activerecord/lib/active_record | |
parent | 3916656f8e9700eb5f1cfc441ff66e1f12173683 (diff) | |
download | rails-c0eb542e57fe6eaa959218c1a7026e53673ba0cb.tar.gz rails-c0eb542e57fe6eaa959218c1a7026e53673ba0cb.tar.bz2 rails-c0eb542e57fe6eaa959218c1a7026e53673ba0cb.zip |
have does_not_support_reverse? support sql functions with commas in nested brackets
Diffstat (limited to 'activerecord/lib/active_record')
-rw-r--r-- | activerecord/lib/active_record/relation/query_methods.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/activerecord/lib/active_record/relation/query_methods.rb b/activerecord/lib/active_record/relation/query_methods.rb index 0749bb30b5..97acf26516 100644 --- a/activerecord/lib/active_record/relation/query_methods.rb +++ b/activerecord/lib/active_record/relation/query_methods.rb @@ -1137,7 +1137,7 @@ module ActiveRecord def does_not_support_reverse?(order) # Uses SQL function with multiple arguments. - /\([^()]*,[^()]*\)/.match?(order) || + order.split(',').find { |section| section.count('(') != section.count(')')} || # Uses "nulls first" like construction. /nulls (first|last)\Z/i.match?(order) end |