diff options
author | Jon Leighton <j@jonathanleighton.com> | 2012-07-13 11:09:47 +0100 |
---|---|---|
committer | Jon Leighton <j@jonathanleighton.com> | 2012-07-13 11:09:47 +0100 |
commit | b5432046134e8c8aa8cafa5960b387b4befff889 (patch) | |
tree | f4fed7a55f006a9de7927c37065d03af49d40ab3 /lib/arel/nodes | |
parent | cbff1bcf385aba876933b2b4569826e9bc46183c (diff) | |
download | rails-b5432046134e8c8aa8cafa5960b387b4befff889.tar.gz rails-b5432046134e8c8aa8cafa5960b387b4befff889.tar.bz2 rails-b5432046134e8c8aa8cafa5960b387b4befff889.zip |
Remove ArgumentError.
It is untested. There is no `@frame` variable. Presumably it is supposed
to be `@framing`, but changing that shows that some of the tests *are*
setting frame twice.
I don't see why this level of strictness is necessary. If someone
disagrees, they should add a test for this behaviour and make the other
tests pass.
Diffstat (limited to 'lib/arel/nodes')
-rw-r--r-- | lib/arel/nodes/window.rb | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/lib/arel/nodes/window.rb b/lib/arel/nodes/window.rb index b54eb7fe64..383d6b8778 100644 --- a/lib/arel/nodes/window.rb +++ b/lib/arel/nodes/window.rb @@ -17,7 +17,6 @@ module Arel end def frame(expr) - raise ArgumentError, "Window frame cannot be set more than once" if @frame @framing = expr end @@ -75,4 +74,4 @@ module Arel end end end -end
\ No newline at end of file +end |