public
Last active — forked from carlosantoniodasilva/gist:599566

  • Download Gist
gistfile1.rb
Ruby
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65
## Original
def reverse_sql_order(order_query)
order_query.join(', ').split(',').collect { |s|
if s.match(/\s(asc|ASC)$/)
s.gsub(/\s(asc|ASC)$/, ' DESC')
elsif s.match(/\s(desc|DESC)$/)
s.gsub(/\s(desc|DESC)$/, ' ASC')
else
s + ' DESC'
end
}
end
 
## Modified
def reverse_sql_order(order_query)
order_query.join(', ').split(',').collect { |s|
if s.match(/\s(asc|ASC)$/)
s.gsub!(/\s(asc|ASC)$/, ' DESC')
elsif s.match(/\s(desc|DESC)$/)
s.gsub!(/\s(desc|DESC)$/, ' ASC')
else
s.concat(' DESC')
end
 
s
}
end
 
## Modified - don't create match object
def reverse_sql_order(order_query)
order_query.join(', ').split(',').collect { |s|
if s =~ /\s(asc|ASC)$/
s.gsub!(/\s(asc|ASC)$/, ' DESC')
elsif s =~ /\s(desc|DESC)$/
s.gsub!(/\s(desc|DESC)$/, ' ASC')
else
s.concat(' DESC')
end
 
s
}
end
 
## Modified - case?
def reverse_sql_order(order_query)
order_query.join(', ').split(',').collect { |s|
case s
when /\s(asc|ASC)$/
s.gsub!(/\s(asc|ASC)$/, ' DESC')
when /\s(desc|DESC)$/
s.gsub!(/\s(desc|DESC)$/, ' ASC')
else
s.concat(' DESC')
end
 
s
}
end
 
## Lazy gsub?
def reverse_sql_order(order_query)
order_query.join(', ').split(',').collect do |s|
s.gsub!(/\sasc\Z/i, ' DESC') || s.gsub!(/\sdesc\Z/i, ' ASC') || s.concat(' DESC')
end
end

Please sign in to comment on this gist.

Something went wrong with that request. Please try again.