diff options
| author | Aaron Patterson <aaron.patterson@gmail.com> | 2010-12-09 15:52:12 -0800 |
|---|---|---|
| committer | Aaron Patterson <aaron.patterson@gmail.com> | 2010-12-09 15:52:12 -0800 |
| commit | a84c34fa08596c563226f3df216e3adf8868f112 (patch) | |
| tree | ad126306b7f857acb0e2757fb8130265a1fae760 | |
| parent | 30a0a9c23bc875eaca0c4b525696ed3ea074ba74 (diff) | |
| download | rails-a84c34fa08596c563226f3df216e3adf8868f112.tar.gz rails-a84c34fa08596c563226f3df216e3adf8868f112.tar.bz2 rails-a84c34fa08596c563226f3df216e3adf8868f112.zip | |
refactor SelectManager to use the create_join factory method
| -rw-r--r-- | lib/arel/select_manager.rb | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/arel/select_manager.rb b/lib/arel/select_manager.rb index f8b3089a82..09491c988b 100644 --- a/lib/arel/select_manager.rb +++ b/lib/arel/select_manager.rb @@ -89,10 +89,10 @@ module Arel case relation when String, Nodes::SqlLiteral raise if relation.blank? - from Nodes::StringJoin.new(@ctx.froms, relation) - else - from klass.new(@ctx.froms, relation, nil) + klass = Nodes::StringJoin end + + from create_join(@ctx.froms, relation, nil, klass) end def having expr |