From 9d4225f1298bebacc04cc0055b180c6b2926c024 Mon Sep 17 00:00:00 2001 From: Rick Olson Date: Fri, 15 Jun 2007 23:28:51 +0000 Subject: [PATCH] Fixed that dispatcher preparation callbacks only run once in production mode. Mock Routes.reload so that dispatcher preparation callback tests run. [Rick] git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@7033 5ecf4fe2-1ee6-0310-87b1-e25e094e27de --- railties/CHANGELOG | 2 ++ railties/lib/dispatcher.rb | 3 +-- railties/test/dispatcher_test.rb | 7 +++++++ 3 files changed, 10 insertions(+), 2 deletions(-) diff --git a/railties/CHANGELOG b/railties/CHANGELOG index 99e8249b46..72d9361962 100644 --- a/railties/CHANGELOG +++ b/railties/CHANGELOG @@ -1,5 +1,7 @@ *SVN* +* Fixed that dispatcher preparation callbacks only run once in production mode. Mock Routes.reload so that dispatcher preparation callback tests run. [Rick] + * Fix syntax error in dispatcher than wrecked failsafe responses. #8625 [mtitorenko] * Scaffolded validation errors set the appropriate HTTP status for XML responses. #6946, #8622 [Manfred Stienstra, mmmultiworks] diff --git a/railties/lib/dispatcher.rb b/railties/lib/dispatcher.rb index fd39b1991b..83d73c4aa3 100644 --- a/railties/lib/dispatcher.rb +++ b/railties/lib/dispatcher.rb @@ -88,7 +88,6 @@ class Dispatcher private attr_accessor_with_default :preparation_callbacks, [] attr_accessor_with_default :preparation_callbacks_run, false - alias_method :preparation_callbacks_run?, :preparation_callbacks_run # CGI.new plus exception handling. CGI#read_multipart raises EOFError # if body.empty? or body.size != Content-Length and raises ArgumentError @@ -113,7 +112,7 @@ class Dispatcher end def run_preparation_callbacks - return if preparation_callbacks_run? + return if preparation_callbacks_run preparation_callbacks.each { |_, callback| callback.call } self.preparation_callbacks_run = true end diff --git a/railties/test/dispatcher_test.rb b/railties/test/dispatcher_test.rb index ad5a4d6275..3267314b03 100644 --- a/railties/test/dispatcher_test.rb +++ b/railties/test/dispatcher_test.rb @@ -27,11 +27,18 @@ class DispatcherTest < Test::Unit::TestCase Dispatcher.send(:preparation_callbacks_run=, false) Object.const_set :ApplicationController, nil + class << ActionController::Routing::Routes + alias_method :old_reload, :reload + def reload() end + end end def teardown Object.send :remove_const, :ApplicationController ENV['REQUEST_METHOD'] = nil + class << ActionController::Routing::Routes + alias_method :reload, :old_reload + end end def test_ac_subclasses_cleared_on_reset