Split apply into two steps

varvet · Nov 1, 2012 · 5affb3e · 5affb3e
1 parent 9c49462
commit 5affb3e
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/lib/ar_outer_join/join.rb b/lib/ar_outer_join/join.rb
@@ -6,12 +6,14 @@ def initialize(klass)
       @klass = klass
     end
 
-    def apply(*args)
-      return klass if args.compact.blank?
-
-      args.inject(klass) do |scope, arg|
-        scope.joins(*JoinBuilder.new(klass.reflect_on_association(arg)).build)
+    def generate(*args)
+      args.compact.map do |arg|
+        JoinBuilder.new(klass.reflect_on_association(arg)).build
       end
     end
+
+    def apply(*args)
+      klass.joins(generate(*args))
+    end
   end
 end