From 416ad003716bcc883c319ec30644bab8a3cf52c7 Mon Sep 17 00:00:00 2001 From: Sergey Alekseev Date: Mon, 26 May 2014 16:27:57 +0300 Subject: Wrap nested Nodes::Grouping in brackets only once --- lib/arel/visitors/to_sql.rb | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) (limited to 'lib/arel/visitors') diff --git a/lib/arel/visitors/to_sql.rb b/lib/arel/visitors/to_sql.rb index 22f7943ab9..264521f4c6 100644 --- a/lib/arel/visitors/to_sql.rb +++ b/lib/arel/visitors/to_sql.rb @@ -429,8 +429,12 @@ module Arel end def visit_Arel_Nodes_Grouping o, collector - collector << "(" - visit(o.expr, collector) << ")" + if o.expr.is_a? Nodes::Grouping + visit(o.expr, collector) + else + collector << "(" + visit(o.expr, collector) << ")" + end end def visit_Arel_SelectManager o, collector -- cgit v1.2.3