Fixed a bug that would cause an ApplicationController to require itself three times and hence cause filters to be run three times [evl]

git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@201 5ecf4fe2-1ee6-0310-87b1-e25e094e27de

1 files changed, 5 insertions, 4 deletions

diff --git a/actionpack/lib/action_controller/dependencies.rb b/actionpack/lib/action_controller/dependencies.rb
index d17369e5e1..14fb0147b6 100644
--- a/actionpack/lib/action_controller/dependencies.rb
+++ b/actionpack/lib/action_controller/dependencies.rb
@@ -32,8 +32,9 @@ module ActionController #:nodoc:
     #     observer :project_change_observer
     #   end
     #
-    # Please note that a controller like ApplicationController will automatically attempt to require_dependency on a model of its name and a helper
-    # of its name. If nothing is found, no error is raised. This is especially useful for concrete controllers like PostController:
+    # Please note that a controller like ApplicationController will automatically attempt to require_dependency on a model of its 
+    # singuralized name and a helper of its name. If nothing is found, no error is raised. This is especially useful for concrete 
+    # controllers like PostController:
     #
     #   class PostController < ApplicationController
     #     # model  :post (already required)
@@ -94,8 +95,8 @@ module ActionController #:nodoc:
 
         def inherited(child)
           inherited_without_model(child)
+          return if child.controller_name == "application" # otherwise the ApplicationController in Rails will include itself
           begin
-            child.model(child.controller_name)
             child.model(Inflector.singularize(child.controller_name))
           rescue LoadError
             # No neither singular or plural model available for this controller
@@ -103,4 +104,4 @@ module ActionController #:nodoc:
         end        
     end
   end
-end
+end
\ No newline at end of file