aboutsummaryrefslogtreecommitdiffstats
path: root/railties/lib/rails
diff options
context:
space:
mode:
authorPiotr Sarnacki <drogus@gmail.com>2012-05-27 12:28:26 +0200
committerPiotr Sarnacki <drogus@gmail.com>2012-05-27 14:13:45 +0200
commit0e69705b0fc7501bada74b3ca023ae7f7b2b8592 (patch)
tree8a3343074b09e1fa2584887257efb153e1cb1525 /railties/lib/rails
parent81ad94e0ac199cbae6c7bdc0380b2c821c3c56a5 (diff)
downloadrails-0e69705b0fc7501bada74b3ca023ae7f7b2b8592.tar.gz
rails-0e69705b0fc7501bada74b3ca023ae7f7b2b8592.tar.bz2
rails-0e69705b0fc7501bada74b3ca023ae7f7b2b8592.zip
Fix railties_order when application object is passed
railites_order method, introduced in 40b19e0, had a bug that was causing loading application instance twice in initializers if railties_order already included application instance. So for example railties_order = [Foo::Engine, :main_app, Bar::Engine] would result in such railties array: [MyApp::Application, Foo::Engine, MyAppApplication, Bar::Engine] In order to fix it, we need to check for existence of application in both railties_order and railties arrays.
Diffstat (limited to 'railties/lib/rails')
-rw-r--r--railties/lib/rails/application.rb2
1 files changed, 1 insertions, 1 deletions
diff --git a/railties/lib/rails/application.rb b/railties/lib/rails/application.rb
index 6d3349c61b..d92117f97e 100644
--- a/railties/lib/rails/application.rb
+++ b/railties/lib/rails/application.rb
@@ -188,7 +188,7 @@ module Rails
end
all = (railties.all - order)
- all.push(self) unless all.include?(self)
+ all.push(self) unless (all + order).include?(self)
order.push(:all) unless order.include?(:all)
index = order.index(:all)