diff options
author | Aaron Patterson <aaron.patterson@gmail.com> | 2010-12-14 20:31:49 -0800 |
---|---|---|
committer | Aaron Patterson <aaron.patterson@gmail.com> | 2010-12-14 20:31:49 -0800 |
commit | ef46276acda7db7b33ee8b84a39cd7d10c3e2628 (patch) | |
tree | bb0159ba410a0ae75d15b8fdf27afe1c17472878 | |
parent | a7507d845a3b560895bbb0df3ce94fa033caa141 (diff) | |
download | rails-ef46276acda7db7b33ee8b84a39cd7d10c3e2628.tar.gz rails-ef46276acda7db7b33ee8b84a39cd7d10c3e2628.tar.bz2 rails-ef46276acda7db7b33ee8b84a39cd7d10c3e2628.zip |
cleaning up more nodes
-rw-r--r-- | Manifest.txt | 3 | ||||
-rw-r--r-- | lib/arel/nodes.rb | 8 | ||||
-rw-r--r-- | lib/arel/nodes/binary.rb | 9 | ||||
-rw-r--r-- | lib/arel/nodes/does_not_match.rb | 6 | ||||
-rw-r--r-- | lib/arel/nodes/join.rb | 13 | ||||
-rw-r--r-- | lib/arel/nodes/join_source.rb | 6 | ||||
-rw-r--r-- | lib/arel/nodes/not_in.rb | 6 | ||||
-rw-r--r-- | test/test_factory_methods.rb | 2 |
8 files changed, 13 insertions, 40 deletions
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 |