Ticket #883: route_paths_as_arrays.patch

actionpack/lib/action_controller/routing.rb

@@ -47 +47 @@
-        
-        used_names = @requirements.inject({}) {|hash, (k, v)| hash[k] = true; hash} # Mark requirements as used so they don't get put in the query params
-        components = @items.collect do |item|
+
-          if item.kind_of? Symbol
-            collection = false
-
@@ -69 +70 @@
-              else
-                value = Routing.extract_parameter_value(value).gsub(/%2F/, "/")
-              end
+              value
-            else
-              Routing.extract_parameter_value(value)
-            end
@@ -117 +119 @@
-            options[:controller] = controller_class.controller_path
-            return nil, requirements_for(:controller) unless passes_requirements?(:controller, options[:controller])
-          elsif /^\*/ =~ item.to_s
-
+? 
-            value.collect! {|c| CGI.unescape c}
-            components = []
-
+ end
-            options[item.to_s.sub(/^\*/,"").intern] = value
-          elsif item.kind_of? Symbol
-            value = components.shift || @defaults[item]

actionpack/test/controller/routing_tests.rb

@@ -329 +329 @@
-  def test_path_collection
-    route '*path_info', :controller => 'content', :action => 'fish'
-    verify_recognize'path/with/slashes',
-'path/with/slashes'
+%w(path with slashes)
-    verify_generate('path/with/slashes', {},
-       
+        
-        {})
-  end
+  def test_path_collection_with_array
+    route '*path_info', :controller => 'content', :action => 'fish'
+    verify_recognize'path/with/slashes',
+        :controller => 'content', :action => 'fish', :path_info => %w(path with slashes)
+    verify_generate('path/with/slashes', {},
+        {:controller => 'content', :action => 'fish', :path_info => %w(path with slashes)},
+        {})
+  end
+
+
-  
-  def test_special_characters
-    route ':id', :controller => 'content', :action => 'fish'