diff options
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/arel/visitors/oracle.rb | 25 |
1 files changed, 17 insertions, 8 deletions
diff --git a/lib/arel/visitors/oracle.rb b/lib/arel/visitors/oracle.rb index bb6ebd4ab8..00572213e6 100644 --- a/lib/arel/visitors/oracle.rb +++ b/lib/arel/visitors/oracle.rb @@ -70,19 +70,14 @@ module Arel /DISTINCT.*FIRST_VALUE/ === projection end end - # FIXME: previous version with join and split broke ORDER BY clause + # Previous version with join and split broke ORDER BY clause # if it contained functions with several arguments (separated by ','). - # Currently splitting is done only if there is no function calls # # orders = o.orders.map { |x| visit x }.join(', ').split(',') orders = o.orders.map do |x| string = visit x - # if there is function call - if string.include?('(') - string - # if no function call then comma splits several ORDER BY columns - elsif string.include?(',') - string.split(',') + if string.include?(',') + split_order_string(string) else string end @@ -94,6 +89,20 @@ module Arel end o end + + # Split string by commas but count opening and closing brackets + # and ignore commas inside brackets. + def split_order_string(string) + array = [] + i = 0 + string.split(',').each do |part| + array[i] ||= "" + array[i] << part + i += 1 if array[i].count('(') == array[i].count(')') + end + array + end + end end end |