lib/arel/visitors/join_sql.rb in arel-2.0.10 vs lib/arel/visitors/join_sql.rb in arel-2.1.0

@@ -6,35 +6,14 @@
     #
     #   http://www.sqlite.org/syntaxdiagrams.html#join-source
     #
     # This visitor is used in SelectManager#join_sql and is for backwards
     # compatibility with Arel V1.0
-    class JoinSql < Arel::Visitors::ToSql
+    module JoinSql
       private
 
       def visit_Arel_Nodes_SelectCore o
-        [o.froms].grep(Nodes::Join).map { |x| visit x }.join ', '
-      end
-
-      def visit_Arel_Nodes_StringJoin o
-        [
-          (visit o.left if Nodes::Join === o.left),
-          visit(o.right)
-        ].compact.join ' '
-      end
-
-      def visit_Arel_Nodes_OuterJoin o
-        [
-          (visit o.left if Nodes::Join === o.left),
-          "LEFT OUTER JOIN #{visit o.right} #{visit o.constraint if o.constraint}"
-        ].compact.join ' '
-      end
-
-      def visit_Arel_Nodes_InnerJoin o
-        [
-          (visit o.left if Nodes::Join === o.left),
-          "INNER JOIN #{visit o.right} #{visit o.constraint if o.constraint}"
-        ].compact.join ' '
+        o.source.right.map { |j| visit j }.join ' '
       end
     end
   end
 end