From b5432046134e8c8aa8cafa5960b387b4befff889 Mon Sep 17 00:00:00 2001 From: Jon Leighton Date: Fri, 13 Jul 2012 11:09:47 +0100 Subject: 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. --- lib/arel/nodes/window.rb | 3 +-- 1 file changed, 1 insertion(+), 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 -- cgit v1.2.3