diff options
| author | Sean Griffin <sean@thoughtbot.com> | 2015-01-27 09:52:54 -0700 |
|---|---|---|
| committer | Sean Griffin <sean@thoughtbot.com> | 2015-01-27 09:52:54 -0700 |
| commit | aac9da257f291ad8d2d4f914528881c240848bb2 (patch) | |
| tree | 4ba06fbdc5f8f2d2a533abc7e7b3c0f14ad5d1aa /lib/arel/select_manager.rb | |
| parent | d36a769234911c8374e09069eb054d4c60eb1b99 (diff) | |
| download | rails-aac9da257f291ad8d2d4f914528881c240848bb2.tar.gz rails-aac9da257f291ad8d2d4f914528881c240848bb2.tar.bz2 rails-aac9da257f291ad8d2d4f914528881c240848bb2.zip | |
Change the interface of `having` to match that of `where`
These two clauses have nearly identical semantics with regards to how
they would be constructed as an AST. It doesn't make sense for their
interfaces to be separate.
Diffstat (limited to 'lib/arel/select_manager.rb')
| -rw-r--r-- | lib/arel/select_manager.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/arel/select_manager.rb b/lib/arel/select_manager.rb index dd1ae37b65..e5fdbc887c 100644 --- a/lib/arel/select_manager.rb +++ b/lib/arel/select_manager.rb @@ -118,8 +118,8 @@ module Arel join(relation, Nodes::OuterJoin) end - def having *exprs - @ctx.having = Nodes::Having.new(collapse(exprs, @ctx.having)) + def having expr + @ctx.havings << expr self end |