update deprecations to include alternative methods (where available)

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

activerecord/lib/active_record/associations/has_and_belongs_to_many_association.rb

@@ -81,7 +81,7 @@ module ActiveRecord
 
         self
       end
-      deprecate :push_with_attributes
+      deprecate :push_with_attributes => "consider using has_many :through instead"
 
       alias :concat_with_attributes :push_with_attributes
 

activerecord/lib/active_record/associations/has_many_association.rb

@@ -31,13 +31,13 @@ module ActiveRecord
           @reflection.klass.find_all(conditions, orderings, limit, joins)
         end
       end
-      deprecate :find_all
+      deprecate :find_all => "use find(:all, ...) instead"
 
       # DEPRECATED. Find the first associated record.  All arguments are optional.
       def find_first(conditions = nil, orderings = nil)
         find_all(conditions, orderings, 1).first
       end
-      deprecate :find_first
+      deprecate :find_first => "use find(:first, ...) instead"
 
       # Count the number of associated records. All arguments are optional.
       def count(*args)

activerecord/lib/active_record/base.rb

@@ -826,7 +826,7 @@ module ActiveRecord #:nodoc:
       def quote(value, column = nil) #:nodoc:
         connection.quote(value, column)
       end
-      deprecate :quote
+      deprecate :quote => :quote_value
 
       # Used to sanitize objects before they're used in an SELECT SQL-statement. Delegates to <tt>connection.quote</tt>.
       def sanitize(object) #:nodoc:
@@ -2012,7 +2012,7 @@ module ActiveRecord #:nodoc:
       def quote(value, column = nil)
         self.class.connection.quote(value, column)
       end
-      deprecate :quote
+      deprecate :quote => :quote_value
       
       
       # Interpolate custom sql string in instance context.

activerecord/lib/active_record/deprecated_associations.rb

@@ -18,7 +18,7 @@ module ActiveRecord
           def add_#{association_name}(*items)
             #{association_name}.concat(items)
           end
-          deprecate :add_#{association_name}
+          deprecate :add_#{association_name} => "use #{association_name}.concat instead"
         end_eval
       end
 
@@ -27,7 +27,7 @@ module ActiveRecord
           def remove_#{association_name}(*items)
             #{association_name}.delete(items)
           end
-          deprecate :remove_#{association_name}
+          deprecate :remove_#{association_name} => "use #{association_name}.delete instead"
         end_eval
       end
 
@@ -36,7 +36,7 @@ module ActiveRecord
           def has_#{collection_name}?(force_reload = false)
             !#{collection_name}(force_reload).empty?
           end
-          deprecate :has_#{collection_name}?
+          deprecate :has_#{collection_name}? => "use !#{collection_name}.empty? instead"
         end_eval
       end
 
@@ -45,7 +45,7 @@ module ActiveRecord
           def find_in_#{collection_name}(association_id)
             #{collection_name}.find(association_id)
           end
-          deprecate :find_in_#{collection_name}
+          deprecate :find_in_#{collection_name} => "use #{collection_name}.find instead"
         end_eval
       end
 
@@ -56,7 +56,7 @@ module ActiveRecord
               #{collection_name}.find_all(runtime_conditions, orderings, limit, joins)
             end
           end
-          deprecate :find_all_in_#{collection_name}
+          deprecate :find_all_in_#{collection_name} => "use #{collection_name}.find(:all, ...) instead"
         end_eval
       end
 
@@ -65,7 +65,7 @@ module ActiveRecord
           def create_in_#{collection_name}(attributes = {})
             #{collection_name}.create(attributes)
           end
-          deprecate :create_in_#{collection_name}
+          deprecate :create_in_#{collection_name} => "use #{collection_name}.create instead"
         end_eval
       end
 
@@ -74,7 +74,7 @@ module ActiveRecord
           def build_to_#{collection_name}(attributes = {})
             #{collection_name}.build(attributes)
           end
-          deprecate :build_to_#{collection_name}
+          deprecate :build_to_#{collection_name} => "use #{collection_name}.build instead"
         end_eval
       end
 
@@ -87,7 +87,7 @@ module ActiveRecord
               raise "Comparison object is a #{association_class_name}, should have been \#{comparison_object.class.name}"
             end
           end
-          deprecate :#{association_name}?
+          deprecate :#{association_name}? => :==
         end_eval
       end
 
@@ -96,7 +96,7 @@ module ActiveRecord
           def has_#{association_name}?(force_reload = false)
             !#{association_name}(force_reload).nil?
           end
-          deprecate :has_#{association_name}?
+          deprecate :has_#{association_name}? => "use !#{association_name} insead"
         end_eval
       end
     end

activerecord/lib/active_record/deprecated_finders.rb

@@ -10,7 +10,7 @@ module ActiveRecord
       def find_on_conditions(ids, conditions) # :nodoc:
         find(ids, :conditions => conditions)
       end
-      deprecate :find_on_conditions
+      deprecate :find_on_conditions => "use find(ids, :conditions => conditions)"
 
       # This method is deprecated in favor of find(:first, options).
       #
@@ -22,7 +22,7 @@ module ActiveRecord
       def find_first(conditions = nil, orderings = nil, joins = nil) # :nodoc:
         find(:first, :conditions => conditions, :order => orderings, :joins => joins)
       end
-      deprecate :find_first
+      deprecate :find_first => "use find(:first, ...)"
 
       # This method is deprecated in favor of find(:all, options).
       #
@@ -38,7 +38,7 @@ module ActiveRecord
         limit, offset = limit.is_a?(Array) ? limit : [ limit, nil ]
         find(:all, :conditions => conditions, :order => orderings, :joins => joins, :limit => limit, :offset => offset)
       end
-      deprecate :find_all
+      deprecate :find_all => "use find(:all, ...)"
     end
   end
-end
+end