Ticket #8562: add_recognition_for_symbol_parameters.diff

actionpack/test/controller/url_rewriter_test.rb

@@ -210 +210 @@
-    assert_equal params[3], { 'query[person][position][]' => 'art director' }.to_query
-  end
-
+  def test_path_generation_for_symbol_parameter_keys
+    assert_generates("/image", :controller=> :image)
+  end
+
-  private
-    def extract_params(url)
-      url.split('?', 2).last.split('&')

actionpack/lib/action_controller/routing.rb

@@ -1239 +1239 @@
-        # great fun, eh?
-
-        options_as_params = options.clone
+
+        options.each do |k, value|
+          options_as_params[k] = value.to_param
+        end
+
-        options_as_params[:action] ||= 'index' if options[:controller]
-        options_as_params[:action] = options_as_params[:action].to_s if options_as_params[:action]
-        options_as_params