0
@@ -1,5 +1,7 @@
0
 *1.13.0 RC1* (November 19th, 2006)
0
 
0
+* Update Routing to complain when :controller is not specified by a route. Closes #6669. [Nicholas Seckar]
0
+
0
 * Ensure render_to_string cleans up after itself when an exception is raised.  #6658 [rsanheim]
0
 
0
 * Update to Prototype and script.aculo.us [5579]. [Sam Stephenson, Thomas Fuchs]

0
@@ -950,6 +950,10 @@ module ActionController
0
           route.significant_keys << :action
0
         end
0
 
0
+        if !route.significant_keys.include?(:controller)
0
+          raise ArgumentError, "Illegal route: the :controller must be specified!"
0
+        end
0
+
0
         route
0
       end
0
     end

0
@@ -853,6 +853,12 @@ class RouteTest < Test::Unit::TestCase
0
       { :controller => "users", :action => "show", :format => "html" },
0
       route.defaults)
0
   end
0
+  
0
+  def test_builder_complains_without_controller
0
+    assert_raises(ArgumentError) do
0
+      ROUTING::RouteBuilder.new.build '/contact', :contoller => "contact", :action => "index"
0
+    end
0
+  end
0
 
0
   def test_significant_keys_for_default_route
0
     keys = default_route.significant_keys.sort_by {|k| k.to_s }

0
@@ -78,7 +78,7 @@ class UrlWriterTests < Test::Unit::TestCase
0
   
0
   def test_named_route
0
     ActionController::Routing::Routes.draw do |map|
0
-      map.home '/home/sweet/home/:user'
0
+      map.home '/home/sweet/home/:user', :controller => 'home', :action => 'index'
0
       map.connect ':controller/:action/:id'
0
     end
0
     
0
@@ -96,7 +96,7 @@ class UrlWriterTests < Test::Unit::TestCase
0
   
0
   def test_only_path
0
     ActionController::Routing::Routes.draw do |map|
0
-      map.home '/home/sweet/home/:user'
0
+      map.home '/home/sweet/home/:user', :controller => 'home', :action => 'index'
0
       map.connect ':controller/:action/:id'
0
     end
0