plain_old_model 0.1.2 → 0.1.3

data/lib/plain_old_model/attribute_assignment.rb

@@ -25,16 +25,17 @@ module PlainOldModel
 
     def assign_attributes(new_attributes, options = {})
       return unless new_attributes
+      attributes = new_attributes.dup
+
       associations.each do |association|
         attr_name = association.attr_name
-        if new_attributes.include?(attr_name)
-          value = association.create_value_from_attributes(new_attributes[attr_name])
-
+        if attributes.include?(attr_name)
+          value = association.create_value_from_attributes(attributes[attr_name])
           set_attribute(attr_name, value)
-          new_attributes  = new_attributes.delete_if { |key, value| key.to_s == attr_name.to_s }
+          attributes  = attributes.delete_if { |key, value| key.to_s == attr_name.to_s }
         end
       end
-      assign_simple_attributes(new_attributes, options)
+      assign_simple_attributes(attributes, options)
     end
 
     def associations

data/lib/plain_old_model/version.rb

@@ -1,3 +1,3 @@
 module PlainOldModel
-  VERSION = "0.1.2"
+  VERSION = "0.1.3"
 end

data/spec/lib/base_spec.rb

@@ -79,6 +79,15 @@ describe PlainOldModel::Base do
         @person.addresses[1].lname.should == "last name 2"
       end
 
+      it "should not alter the params passed in" do
+        passed_params = { addresses: [{ fname: "first name 1", lname: "last name 1"}, { fname: "first name 2", lname: "last name 2"}]}
+        @person = Person.new(passed_params)
+        @person.addresses.length.should == 2
+        @person.addresses.first.class.should == Address
+        @person.addresses.first.fname.should == "first name 1"
+        passed_params.should == { addresses: [{ fname: "first name 1", lname: "last name 1"}, { fname: "first name 2", lname: "last name 2"}]}
+      end
+
       it "should create each class via factory_method if one is specified" do
         @person = Person.new({ phones: [{ number: '423-5841'}, {number: '383-9139'}]})
         @person.phones.length.should == 2

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: plain_old_model
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.3
   prerelease: 
 platform: ruby
 authors:
@@ -10,11 +10,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-02-13 00:00:00.000000000 Z
+date: 2013-03-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
-  requirement: &70107737128780 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -22,10 +22,15 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70107737128780
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: activemodel
-  requirement: &70107737128360 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -33,10 +38,15 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70107737128360
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &70107737127940 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -44,10 +54,15 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70107737127940
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70107737127440 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -55,10 +70,15 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70107737127440
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec-core
-  requirement: &70107737126940 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -66,7 +86,12 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70107737126940
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 description: ! "This gem is created to cater the projects which do not require a backend/database,\n
   \ but still need some of the niceties offered by the ActiveRecord"
 email:
@@ -103,15 +128,21 @@ required_ruby_version: !ruby/object:Gem::Requirement
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
+      segments:
+      - 0
+      hash: -3603961560709073915
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
+      segments:
+      - 0
+      hash: -3603961560709073915
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.17
+rubygems_version: 1.8.24
 signing_key: 
 specification_version: 3
 summary: This gem is created to cater the projects which do not require a backend/database,