Fix issue with deprecation messing up #template_root= usage. Add...

Fix issue with deprecation messing up #template_root= usage.  Add #prepend_view_path and #append_view_path to allow modification of a copy of the
superclass' view_paths.  [Rick]

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

actionpack/CHANGELOG

 *SVN*
 
+* Fix issue with deprecation messing up #template_root= usage.  Add #prepend_view_path and #append_view_path to allow modification of a copy of the
+superclass' view_paths.  [Rick]
+
 * Allow Controllers to have multiple view_paths instead of a single template_root.  Closes #2754 [John Long]
 
 * Add much-needed html-scanner tests.  Fixed CDATA parsing bug. [Rick]

actionpack/lib/action_controller/base.rb

@@ -356,9 +356,9 @@ def hide_action(*names)
       
       # Deprecated. Use view_paths instead.
       def template_root=(path)
-        view_paths.unshift(path)
+        prepend_view_path path
+        template_root
       end
-      deprecate :template_root= => :view_paths
       
       # Deprecated. Use view_paths instead.
       def template_root
@@ -374,7 +374,7 @@ def template_root
       def view_paths=(value)
         @@view_paths[name] = value
       end
-      
+
       # View load paths for controller.
       def view_paths
         if paths = @@view_paths[name]
@@ -388,6 +388,22 @@ def view_paths
         end
       end
       
+      # Adds a view_path to the front of the view_paths array.
+      # If the current class has no view paths, copy them from 
+      # the superclass
+      def prepend_view_path(path)
+        self.view_paths = view_paths.dup if view_paths.frozen?
+        view_paths.unshift(path)
+      end
+      
+      # Adds a view_path to the end of the view_paths array.
+      # If the current class has no view paths, copy them from 
+      # the superclass
+      def append_view_path(path)
+        self.view_paths = view_paths.dup if view_paths.frozen?
+        view_paths << path
+      end
+      
       # Replace sensitive paramater data from the request log.
       # Filters paramaters that have any of the arguments as a substring.
       # Looks in all subhashes of the param hash for keys to filter.

actionpack/lib/action_controller/components.rb

@@ -77,8 +77,8 @@ def process_with_components(request, response, parent_controller = nil) #:nodoc:
       def uses_component_template_root
         path_of_calling_controller = File.dirname(caller[1].split(/:\d+:/, 2).first)
         path_of_controller_root    = path_of_calling_controller.sub(/#{Regexp.escape(File.dirname(controller_path))}$/, "")
-
-        self.template_root = path_of_controller_root
+        prepend_view_path path_of_controller_root
+        view_paths.first
       end
 
       deprecate :uses_component_template_root => 'Components are deprecated and will be removed in Rails 2.0.'

actionpack/test/controller/view_paths_test.rb

@@ -46,10 +46,11 @@ def test_view_paths_override
   
   def test_template_root_deprecated
     assert_deprecated(/template_root.*view_paths/) do
-      TestController.template_root = LOAD_PATH_ROOT
+      TestController.template_root = 'foo/bar'
     end
     assert_deprecated(/template_root.*view_paths/) do
-      assert_equal LOAD_PATH_ROOT, TestController.template_root
+      assert_equal 'foo/bar', TestController.template_root
+      assert_equal ['foo/bar', LOAD_PATH_ROOT], TestController.view_paths
     end
   end
   

actionwebservice/test/scaffolded_controller_test.rb

@@ -5,7 +5,7 @@
   map.connect ':controller/:action/:id'
 end
 
-ActionController::Base.template_root = '.'
+ActionController::Base.view_load_paths = [ '.' ]
 
 class ScaffoldPerson < ActionWebService::Struct
   member :id,     :int