Backed out of new_record? to new? transformation as it would screw up existing models that did boolean calls on "new" attributes [DHH]

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

1 files changed, 4 insertions, 4 deletions

diff --git a/activerecord/lib/active_record/associations/has_and_belongs_to_many_association.rb b/activerecord/lib/active_record/associations/has_and_belongs_to_many_association.rb
index f10c5957f8..8c0e50e38e 100644
--- a/activerecord/lib/active_record/associations/has_and_belongs_to_many_association.rb
+++ b/activerecord/lib/active_record/associations/has_and_belongs_to_many_association.rb
@@ -19,7 +19,7 @@ module ActiveRecord
           attributes.collect { |attr| create(attr) }
         else
           record = build(attributes)
-          insert_record(record) unless @owner.new?
+          insert_record(record) unless @owner.new_record?
           record
         end
       end
@@ -75,7 +75,7 @@ module ActiveRecord
         join_attributes.each { |key, value| record[key.to_s] = value }
 
         callback(:before_add, record)
-        insert_record(record) unless @owner.new?
+        insert_record(record) unless @owner.new_record?
         @target << record
         callback(:after_add, record)
 
@@ -101,7 +101,7 @@ module ActiveRecord
         end
 
         def insert_record(record)
-          if record.new?
+          if record.new_record?
             return false unless record.save
           end
 
@@ -118,7 +118,7 @@ module ActiveRecord
                   attributes[column.name] = record.quoted_id
                 else
                   if record.attributes.has_key?(column.name)
-                    value = @owner.send(:quote, record[column.name], column)
+                    value = @owner.send(:quote_value, record[column.name], column)
                     attributes[column.name] = value unless value.nil?
                   end
               end