Ticket #10537: add_new_record_method_to_ares.diff

test/base_test.rb

@@ -301 +301 @@
-    assert_equal true, rick.save
-    assert_equal '5', rick.id
-  end
+  
+  def test_new_record
+    rick = Person.new
+    
+    # new_record? is an alias for new? to improve compatibility with ActionRecord
+    assert rick.new? && (rick.new? == rick.new_record?)
+    
+    assert_equal true, rick.save
+    assert !rick.new? && (rick.new? == rick.new_record?)
+  end  
-
-  def test_id_from_response
-    p = Person.new
@@ -350 +360 @@
-    end    
-    assert_raises(ActiveResource::ResourceConflict) { Person.create(:name => 'Rick') }
-  end
-
+
-  def test_update
-    matz = Person.find(:first)
-    matz.name = "David"

lib/active_resource/base.rb

@@ -553 +553 @@
-    def new?
-      id.nil?
-    end
+    
+    # Alias for +new?+
+    #
+    # This method is added for compatibility with ActiveRecord.
+    def new_record?
+      new?
+    end
-
-    # Get the +id+ attribute of the resource.
-    def id

README

@@ -123 +123 @@
-  ryan.new?  #=> false
-  ryan.id    #=> 2
-
+In the previous example, +new?+ is used to verify the state of the record before and after saving it.
+The same could be accomplished by using the +new_record?+ method instead.
++new_record?+ is an alias for +new?+ included for compatibility with ActiveRecord.
+
+  ryan = Person.new(:first => 'Ryan')
+  ryan.new_record?  #=> true
+  ryan.save         #=> true
+  ryan.new_record?  #=> false
+  ryan.id           #=> 2
+
-==== Update
-
-'save' is also used to update an existing resource - and follows the same protocol as creating a resource