Move #to_query methods where they ought to belong. Closes #10395 [Chu Yeow]

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

activesupport/lib/active_support/core_ext/array/conversions.rb

@@ -29,6 +29,15 @@ module ActiveSupport #:nodoc:
           join '/'
         end
 
+        # Converts an array into a string suitable for use as a URL query string, using the given <tt>key</tt> as the
+        # param name.
+        #
+        # ==== Example:
+        #   ['Rails', 'coding'].to_query('hobbies') => "hobbies%5B%5D=Rails&hobbies%5B%5D=coding"
+        def to_query(key)
+          collect { |value| value.to_query("#{key}[]") } * '&'
+        end
+
         def self.included(base) #:nodoc:
           base.class_eval do
             alias_method :to_default_s, :to_s

activesupport/lib/active_support/core_ext/hash/conversions.rb

@@ -4,23 +4,6 @@ require 'base64'
 require 'builder'
 require 'xmlsimple'
 
-# Extensions needed for Hash#to_query
-class Object
-  def to_param #:nodoc:
-    to_s
-  end
-
-  def to_query(key) #:nodoc:
-    "#{CGI.escape(key.to_s)}=#{CGI.escape(to_param.to_s)}"
-  end
-end
-
-class Array
-  def to_query(key) #:nodoc:
-    collect { |value| value.to_query("#{key}[]") } * '&'
-  end
-end
-
 # Locked down XmlSimple#xml_in_string
 class XmlSimple
   # Same as xml_in but doesn't try to smartly shoot itself in the foot.
@@ -100,6 +83,13 @@ module ActiveSupport #:nodoc:
           klass.extend(ClassMethods)
         end
 
+        # Converts a hash into a string suitable for use as a URL query string. An optional <tt>namespace</tt> can be
+        # passed to enclose the param names (see example below).
+        #
+        # ==== Example:
+        #   { :name => 'David', :nationality => 'Danish' }.to_query # => "name=David&nationality=Danish"
+        #
+        #   { :name => 'David', :nationality => 'Danish' }.to_query('user') # => "user%5Bname%5D=David&user%5Bnationality%5D=Danish"
         def to_query(namespace = nil)
           collect do |key, value|
             value.to_query(namespace ? "#{namespace}[#{key}]" : key)

activesupport/lib/active_support/core_ext/object.rb

@@ -1,3 +1,4 @@
+require 'active_support/core_ext/object/conversions'
 require 'active_support/core_ext/object/extending'
 require 'active_support/core_ext/object/instance_variables'
 require 'active_support/core_ext/object/misc'