From ef46276acda7db7b33ee8b84a39cd7d10c3e2628 Mon Sep 17 00:00:00 2001
From: Aaron Patterson <aaron.patterson@gmail.com>
Date: Tue, 14 Dec 2010 20:31:49 -0800
Subject: cleaning up more nodes

---
 Manifest.txt                     |  3 ---
 lib/arel/nodes.rb                |  8 +++-----
 lib/arel/nodes/binary.rb         |  9 +++++++++
 lib/arel/nodes/does_not_match.rb |  6 ------
 lib/arel/nodes/join.rb           | 13 -------------
 lib/arel/nodes/join_source.rb    |  6 ------
 lib/arel/nodes/not_in.rb         |  6 ------
 test/test_factory_methods.rb     |  2 +-
 8 files changed, 13 insertions(+), 40 deletions(-)
 delete mode 100644 lib/arel/nodes/does_not_match.rb
 delete mode 100644 lib/arel/nodes/join.rb
 delete mode 100644 lib/arel/nodes/not_in.rb

diff --git a/Manifest.txt b/Manifest.txt
index 7d9010e783..82c2c4f61a 100644
--- a/Manifest.txt
+++ b/Manifest.txt
@@ -22,7 +22,6 @@ lib/arel/nodes/avg.rb
 lib/arel/nodes/binary.rb
 lib/arel/nodes/count.rb
 lib/arel/nodes/delete_statement.rb
-lib/arel/nodes/does_not_match.rb
 lib/arel/nodes/equality.rb
 lib/arel/nodes/exists.rb
 lib/arel/nodes/function.rb
@@ -32,14 +31,12 @@ lib/arel/nodes/having.rb
 lib/arel/nodes/in.rb
 lib/arel/nodes/inner_join.rb
 lib/arel/nodes/insert_statement.rb
-lib/arel/nodes/join.rb
 lib/arel/nodes/join_source.rb
 lib/arel/nodes/lock.rb
 lib/arel/nodes/max.rb
 lib/arel/nodes/min.rb
 lib/arel/nodes/node.rb
 lib/arel/nodes/not.rb
-lib/arel/nodes/not_in.rb
 lib/arel/nodes/offset.rb
 lib/arel/nodes/on.rb
 lib/arel/nodes/ordering.rb
diff --git a/lib/arel/nodes.rb b/lib/arel/nodes.rb
index ef1db0cfa2..83977c356a 100644
--- a/lib/arel/nodes.rb
+++ b/lib/arel/nodes.rb
@@ -1,17 +1,16 @@
 require 'arel/nodes/node'
 require 'arel/nodes/unary'
 require 'arel/nodes/binary'
+
 require 'arel/nodes/equality'
 require 'arel/nodes/and'
+require 'arel/nodes/in'
+require 'arel/nodes/lock'
 
 require 'arel/nodes/not'
-require 'arel/nodes/does_not_match'
 require 'arel/nodes/join_source'
 
-require 'arel/nodes/in'
-require 'arel/nodes/not_in'
 require 'arel/nodes/ordering'
-require 'arel/nodes/lock'
 require 'arel/nodes/function'
 require 'arel/nodes/count'
 require 'arel/nodes/values'
@@ -30,7 +29,6 @@ require 'arel/nodes/update_statement'
 require 'arel/nodes/delete_statement'
 require 'arel/nodes/unqualified_column'
 require 'arel/nodes/table_alias'
-require 'arel/nodes/join'
 require 'arel/nodes/group'
 require 'arel/nodes/grouping'
 require 'arel/nodes/inner_join'
diff --git a/lib/arel/nodes/binary.rb b/lib/arel/nodes/binary.rb
index 43979f6f2e..1285af9447 100644
--- a/lib/arel/nodes/binary.rb
+++ b/lib/arel/nodes/binary.rb
@@ -7,6 +7,12 @@ module Arel
         @left  = left
         @right = right
       end
+
+      def initialize_copy other
+        super
+        @left  = @left.clone if @left
+        @right = @right.clone if @right
+      end
     end
 
     %w{
@@ -20,6 +26,9 @@ module Arel
       LessThan
       LessThanOrEqual
       Matches
+      DoesNotMatch
+      NotIn
+      Join
     }.each do |name|
       const_set(name, Class.new(Binary))
     end
diff --git a/lib/arel/nodes/does_not_match.rb b/lib/arel/nodes/does_not_match.rb
deleted file mode 100644
index 33bdeab005..0000000000
--- a/lib/arel/nodes/does_not_match.rb
+++ /dev/null
@@ -1,6 +0,0 @@
-module Arel
-  module Nodes
-    class DoesNotMatch < Arel::Nodes::Binary
-    end
-  end
-end
diff --git a/lib/arel/nodes/join.rb b/lib/arel/nodes/join.rb
deleted file mode 100644
index d3307fe0e0..0000000000
--- a/lib/arel/nodes/join.rb
+++ /dev/null
@@ -1,13 +0,0 @@
-module Arel
-  module Nodes
-    class Join < Arel::Nodes::Binary
-
-      alias :single_source :left
-      alias :constraint :right
-
-      def initialize single_source, constraint
-        super
-      end
-    end
-  end
-end
diff --git a/lib/arel/nodes/join_source.rb b/lib/arel/nodes/join_source.rb
index 7b739c19ad..c57ad0e930 100644
--- a/lib/arel/nodes/join_source.rb
+++ b/lib/arel/nodes/join_source.rb
@@ -9,12 +9,6 @@ module Arel
       def initialize single_source, joinop = []
         super
       end
-
-      def initialize_copy other
-        super
-        @left  = @left.clone if @left
-        @right = @right.clone if @right
-      end
     end
   end
 end
diff --git a/lib/arel/nodes/not_in.rb b/lib/arel/nodes/not_in.rb
deleted file mode 100644
index 6c01921a46..0000000000
--- a/lib/arel/nodes/not_in.rb
+++ /dev/null
@@ -1,6 +0,0 @@
-module Arel
-  module Nodes
-    class NotIn < Arel::Nodes::Binary
-    end
-  end
-end
diff --git a/test/test_factory_methods.rb b/test/test_factory_methods.rb
index c40615564f..6506d3c472 100644
--- a/test/test_factory_methods.rb
+++ b/test/test_factory_methods.rb
@@ -14,7 +14,7 @@ module Arel
       def test_create_join
         join = @factory.create_join :one, :two
         assert_kind_of Nodes::Join, join
-        assert_equal :two, join.constraint
+        assert_equal :two, join.right
       end
 
       def test_create_on
-- 
cgit v1.2.3