aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJon Leighton <j@jonathanleighton.com>2012-07-13 11:09:47 +0100
committerJon Leighton <j@jonathanleighton.com>2012-07-13 11:09:47 +0100
commitb5432046134e8c8aa8cafa5960b387b4befff889 (patch)
treef4fed7a55f006a9de7927c37065d03af49d40ab3
parentcbff1bcf385aba876933b2b4569826e9bc46183c (diff)
downloadrails-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.
-rw-r--r--lib/arel/nodes/window.rb3
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