Changeset 6669

Timestamp:

05/06/07 04:10:26 (1 year ago)

Author:

nzkoz

Message:

Bug fix for the filter implementation

Files:

• trunk/actionpack/lib/action_controller/filters.rb (modified) (4 diffs)
• trunk/actionpack/test/controller/filters_test.rb (modified) (3 diffs)

trunk/actionpack/lib/action_controller/filters.rb

@@ -528 +528 @@
-
-        def find_filter_append_position(filters, filter_type)
+          # appending an after filter puts it at the end of the call chain
+          # before and around filters goe before the first after filter in the chain
-          unless filter_type == :after
-            filter_chain.each_with_index do |f,i|
@@ -537 +539 @@
-
-        def find_filter_prepend_position(filters, filter_type)
+          # prepending a before or around filter puts it at the front of the call chain
+          # after filters go before the first after filter in the chain
-          if filter_type == :after
-            filter_chain.each_with_index do |f,i|
-              return i if f.after?
-            end
+            return -1
-          end
-          return 0
@@ -685 +690 @@
-      def call_filters(chain, index, nesting)
-        # run before filters until we find an after filter or around filter
-true
+chain[index]
-          filter, index = skip_excluded_filters(chain, index)
-
+ # end of call chain reached
-          case filter.type
-          when :before
@@ -711 +716 @@
-
-        # run after filters, if any
-filter = 
+
-          filter, index = skip_excluded_filters(chain, index)
+          break unless filter
-          case filter.type
-          when :after

trunk/actionpack/test/controller/filters_test.rb

@@ -132 +132 @@
-  end
-
+  class EmptyFilterChainController < TestController
+    self.filter_chain.clear
+    def show
+      @action_executed = true
+      render :text => "yawp!"
+    end
+  end
+
-  class PrependingController < TestController
-    prepend_before_filter :wonderful_life
@@ -292 +300 @@
-        self.action_name = params[:choose]
-      end
+  end
+
+  class PrependingBeforeAndAfterController < ActionController::Base
+    prepend_before_filter :before_all
+    prepend_after_filter :after_all
+    before_filter :between_before_all_and_after_all
+
+    def before_all
+      @ran_filter ||= []
+      @ran_filter << 'before_all'
+    end
+
+    def after_all
+      @ran_filter ||= []
+      @ran_filter << 'after_all'
+    end
+
+    def between_before_all_and_after_all
+      @ran_filter ||= []
+      @ran_filter << 'between_before_all_and_after_all'
+    end
+    def show
+      render :text => 'hello'
+    end
+  end
+
+  def test_empty_filter_chain
+    assert_equal 0, EmptyFilterChainController.filter_chain.size
+    assert test_process(EmptyFilterChainController).template.assigns['action_executed']
-  end
-
@@ -425 +462 @@
-      assert_equal action, response.body
-    end
+  end
+
+  def test_running_prepended_before_and_after_filter
+    assert_equal 3, PrependingBeforeAndAfterController.filter_chain.length
+    response = test_process(PrependingBeforeAndAfterController)
+    assert_equal %w( before_all between_before_all_and_after_all after_all ), response.template.assigns["ran_filter"]
-  end
-