Ruby on Rails | Screencasts | Download | Documentation | Weblog | Community | Source

Changeset 5006

Show
Ignore:
Timestamp:
09/04/06 23:36:13 (2 years ago)
Author:
bitsweat
Message:

load nil is noop. use new? instead of new_resource?

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • trunk/activeresource/lib/active_resource/base.rb

    r4985 r5006  
    103103    end 
    104104 
    105     def new_resource
     105    def new
    106106      id.nil? 
    107107    end 
     
    116116     
    117117    def save 
    118       new_resource? ? create : update 
     118      new? ? create : update 
    119119    end 
    120120 
     
    135135    # resources. 
    136136    def load(attributes) 
     137      return self if attributes.nil? 
    137138      attributes.each do |key, value| 
    138139        @attributes[key.to_s] = 
  • trunk/activeresource/test/base/load_test.rb

    r4985 r5006  
    1313 
    1414    @person = Person.new 
     15  end 
     16 
     17  def test_load_nil 
     18    assert_nothing_raised do 
     19      assert_equal @person, @person.load(nil) 
     20    end 
    1521  end 
    1622