diff options
author | Sean Griffin <sean@seantheprogrammer.com> | 2014-11-26 14:41:44 -0700 |
---|---|---|
committer | Sean Griffin <sean@seantheprogrammer.com> | 2014-11-26 14:41:44 -0700 |
commit | b40dfdde690ea32d658393e9e81c6ad36d23891f (patch) | |
tree | ed6dcd74bb8e1a780dc62fb9546da95c9e8d1c5e | |
parent | 1fefe71b1872c0a83f09231164863cd8dbb57174 (diff) | |
parent | 8572455d4cf7a61462bde7cef2d3724685c1c44c (diff) | |
download | rails-b40dfdde690ea32d658393e9e81c6ad36d23891f.tar.gz rails-b40dfdde690ea32d658393e9e81c6ad36d23891f.tar.bz2 rails-b40dfdde690ea32d658393e9e81c6ad36d23891f.zip |
Merge pull request #177 from NARKOZ/patch-0
remove extra space from select statement
-rw-r--r-- | lib/arel/visitors/to_sql.rb | 1 | ||||
-rw-r--r-- | test/visitors/test_to_sql.rb | 9 |
2 files changed, 8 insertions, 2 deletions
diff --git a/lib/arel/visitors/to_sql.rb b/lib/arel/visitors/to_sql.rb index 884076d987..30c8634119 100644 --- a/lib/arel/visitors/to_sql.rb +++ b/lib/arel/visitors/to_sql.rb @@ -211,7 +211,6 @@ module Arel } unless o.orders.empty? - collector << SPACE collector << ORDER_BY len = o.orders.length - 1 o.orders.each_with_index { |x, i| diff --git a/test/visitors/test_to_sql.rb b/test/visitors/test_to_sql.rb index 2e3f7f8624..04fca8024b 100644 --- a/test/visitors/test_to_sql.rb +++ b/test/visitors/test_to_sql.rb @@ -181,6 +181,13 @@ module Arel assert_match(/LIMIT 'omg'/, compile(sc)) end + it "should contain a single space before ORDER BY" do + table = Table.new(:users) + test = table.order(table[:name]) + sql = compile test + assert_match(/"users" ORDER BY/, sql) + end + it "should quote LIMIT without column type coercion" do table = Table.new(:users) sc = table.where(table[:name].eq(0)).take(1).ast @@ -291,7 +298,7 @@ module Arel end it "should visit_Arel_Nodes_Assignment" do - column = @table["id"] + column = @table["id"] node = Nodes::Assignment.new( Nodes::UnqualifiedColumn.new(column), Nodes::UnqualifiedColumn.new(column) |