diff options
author | Ryuta Kamizono <kamipo@gmail.com> | 2019-03-18 07:04:01 +0900 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-03-18 07:04:01 +0900 |
commit | aa225455a01e57016e0a28f5d88f6a751ef7b5d2 (patch) | |
tree | 0bed1c0321dbe226e6017c4f8ee9022fdc08a243 /activerecord/lib/arel/visitors/to_sql.rb | |
parent | 2d5d537d19d62e9c132cf49f7dbc9eb8ff9190e3 (diff) | |
parent | 187870db2fcc58aa0da8bb3f26711664fd5ed611 (diff) | |
download | rails-aa225455a01e57016e0a28f5d88f6a751ef7b5d2.tar.gz rails-aa225455a01e57016e0a28f5d88f6a751ef7b5d2.tar.bz2 rails-aa225455a01e57016e0a28f5d88f6a751ef7b5d2.zip |
Merge pull request #35653 from kamipo/remove_arel_nodes_values
Get rid of `Arel::Nodes::Values`
Diffstat (limited to 'activerecord/lib/arel/visitors/to_sql.rb')
-rw-r--r-- | activerecord/lib/arel/visitors/to_sql.rb | 21 |
1 files changed, 1 insertions, 20 deletions
diff --git a/activerecord/lib/arel/visitors/to_sql.rb b/activerecord/lib/arel/visitors/to_sql.rb index 72efac72bf..583f920290 100644 --- a/activerecord/lib/arel/visitors/to_sql.rb +++ b/activerecord/lib/arel/visitors/to_sql.rb @@ -159,7 +159,7 @@ module Arel # :nodoc: all when Nodes::SqlLiteral, Nodes::BindParam collector = visit(value, collector) else - collector << quote(value) + collector << quote(value).to_s end collector << COMMA unless k == row_len end @@ -169,25 +169,6 @@ module Arel # :nodoc: all collector end - def visit_Arel_Nodes_Values(o, collector) - collector << "VALUES (" - - len = o.expressions.length - 1 - o.expressions.each_with_index { |value, i| - case value - when Nodes::SqlLiteral, Nodes::BindParam - collector = visit value, collector - else - collector << quote(value).to_s - end - unless i == len - collector << COMMA - end - } - - collector << ")" - end - def visit_Arel_Nodes_SelectStatement(o, collector) if o.with collector = visit o.with, collector |