From 7e745f3aa5de4e117758ee92e2ca92aa265c6deb Mon Sep 17 00:00:00 2001
From: David Heinemeier Hansson <david@loudthinking.com>
Date: Sun, 20 Mar 2005 15:42:40 +0000
Subject: Added to_param call for parameters when composing an url using
 url_for from something else than strings #812 [Sam Stephenson]

git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@935 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
---
 actionpack/lib/action_controller/routing.rb      | 9 +++++++--
 actionpack/lib/action_controller/url_rewriter.rb | 2 +-
 2 files changed, 8 insertions(+), 3 deletions(-)

(limited to 'actionpack/lib/action_controller')

diff --git a/actionpack/lib/action_controller/routing.rb b/actionpack/lib/action_controller/routing.rb
index c021d88ca9..20bc6c8cae 100644
--- a/actionpack/lib/action_controller/routing.rb
+++ b/actionpack/lib/action_controller/routing.rb
@@ -64,9 +64,9 @@ module ActionController
       	    if value.nil? || item == :controller
               value
             elsif collection
-	            CGI.escape(value.to_s).gsub(/%2F/, "/")
+	            Routing.extract_parameter_value(value).gsub(/%2F/, "/")
             else
-              CGI.escape(value.to_s)
+              Routing.extract_parameter_value(value)
             end
           else
             item
@@ -316,6 +316,11 @@ module ActionController
       end
     end
     
+    def self.extract_parameter_value(parameter)
+      value = parameter.respond_to?(:to_param) ? parameter.to_param : parameter.to_s
+      CGI.escape(value)
+    end
+
     def self.draw(*args, &block) #:nodoc:
       Routes.draw(*args) {|*args| block.call(*args)}
     end
diff --git a/actionpack/lib/action_controller/url_rewriter.rb b/actionpack/lib/action_controller/url_rewriter.rb
index 6c51ef3f3e..bbfa6009a4 100644
--- a/actionpack/lib/action_controller/url_rewriter.rb
+++ b/actionpack/lib/action_controller/url_rewriter.rb
@@ -95,7 +95,7 @@ module ActionController
           key = CGI.escape key
           key += '[]' if value.class == Array
           value = [ value ] unless value.class == Array
-          value.each { |val| elements << "#{key}=#{CGI.escape(val.to_s)}" }
+          value.each { |val| elements << "#{key}=#{Routing.extract_parameter_value(val)}" }
         end
         
         query_string << ("?" + elements.join("&")) unless elements.empty?
-- 
cgit v1.2.3