activerecord_to_poro 0.0.4 → 0.0.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 24e632b82acbcdf3589327c6d5af701573ba97cf
-  data.tar.gz: 0e7f98e49af88b6644f2f1d5da2a91d02fbf8529
+  metadata.gz: 69aba8f2a10463279cad3508895914d15e273f0a
+  data.tar.gz: 23c28ef06203deef5f487f83da6d0bca03e6db85
 SHA512:
-  metadata.gz: 73cdfc500b19c1a94cc1875db048748dc952fc286e37605ea1a8ce83e76bb4cfeb18e3c1364a7c72e6004003a0c55eadc6db785f6b8dbdacb7f7444fd2e01139
-  data.tar.gz: 84437b91e7943cd95e8c98e90f8aa7901f0345b25e92917a9d4349f01a2464724a3e6935492e27de2bfba201ae54d803d7d1192a8db967785d36da8af7eb98e9
+  metadata.gz: 69e06f848fc228e3500275833eb28519140b34568a6994673798909ceba8140de3aa585f398a3f0ddf48f22c8d1daf8ffc45fca537e5d973bd4c4257ec91e2b9
+  data.tar.gz: de404d5cac77aa930610899ed0d6acdefe52cd1ffd73a26c9da85bd27730baa66ff91c4d37a656ea783a4fbaea10b7a69dc248929026227670b642d5c56701ae

data/Gemfile

@@ -1,5 +1,5 @@
 source 'https://rubygems.org'
-ruby "2.1.0"
+#ruby "2.1"
 
 # Specify your gem's dependencies in yaoc.gemspec
 gemspec

data/activerecord_to_poro.gemspec

@@ -26,7 +26,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "bundler", "~> 1.5"
   spec.add_development_dependency "rake"
   spec.add_development_dependency 'sqlite3'
-  spec.add_development_dependency 'activerecord'
+  spec.add_development_dependency 'activerecord', '4.0.2'
   spec.add_development_dependency 'database_cleaner'
 
 
@@ -45,8 +45,9 @@ Gem::Specification.new do |spec|
 
   # https://github.com/pry/pry-stack_explorer
   spec.add_development_dependency 'pry-stack_explorer'
-  # https://github.com/nixme/pry-debugger
-  spec.add_development_dependency 'pry-debugger'
+  # https://github.com/deivid-rodriguez/pry-byebug
+  # pre-debugger / debugger does not work with ruby 2.x
+  spec.add_development_dependency 'pry-byebug'
 
   # https://github.com/dkubb/equalizer
   spec.add_development_dependency 'equalizer'

data/lib/activerecord_to_poro/metadata_enabled_ar.rb

@@ -8,9 +8,38 @@ module ActiverecordToPoro
       record = pre_created_object || record_by_primary_key || new
 
       record.tap do |new_obj|
+
         new_obj.attributes = attrs
+
+        _patch_has_many_members(attrs, new_obj) unless new_obj.new_record?
+      end
+
+    end
+
+    def _patch_has_many_members(attrs, new_obj)
+      has_many_attrs = attrs.slice(* _has_many_attr_names(new_obj))
+
+      has_many_attrs.each_pair do |name, updated_records|
+
+        new_obj.public_send(name).each do |attached_record|
+          record_with_updated_values = updated_records.find {|r| r == attached_record}
+          next unless record_with_updated_values
+
+          _apply_change_set_to_record(attached_record, record_with_updated_values.changes)
+        end
+
       end
+    end
 
+    def _has_many_attr_names(obj_or_class)
+      class_to_check = obj_or_class.respond_to?(:reflect_on_all_associations) ? obj_or_class : obj_or_class.class
+      class_to_check.reflect_on_all_associations(:has_many).map(&:name)
+    end
+
+    def _apply_change_set_to_record(attached_record, changes)
+      changes.each_pair do |attr_name, (_, new_value)|
+        attached_record.public_send("#{attr_name}=", new_value)
+      end
     end
 
     def _as_scope(attr)

data/lib/activerecord_to_poro/object_mapper.rb

@@ -81,11 +81,11 @@ module ActiverecordToPoro
     end
 
     def dump_result_source=(new_dump_result)
-      @dump_result_source = new_dump_result.tap do |source|
-        unless source.respond_to? :_from_attrs_with_metadata
-          source.send(:extend, MetadataEnabledAr)
-        end
+      unless new_dump_result.respond_to? :_from_attrs_with_metadata
+        new_dump_result.send(:extend, MetadataEnabledAr)
       end
+
+      @dump_result_source = new_dump_result
     end
 
     def add_default_mapping_for_current_class

data/lib/activerecord_to_poro/version.rb

@@ -1,3 +1,3 @@
 module ActiverecordToPoro
-  VERSION = "0.0.4"
+  VERSION = "0.0.5"
 end

data/spec/acceptance/map_ar_associations_spec.rb

@@ -84,6 +84,17 @@ feature 'Map active record associations', %q{
     expect(mapper.dump(poro).roles.last.permissions.size).to eq 1
   end
 
+  scenario "updates an associated ActiveRecord object from a modified poro object" do
+    poro = mapper.load(a_active_record_object)
+    expect(mapper.dump(poro).roles.size).to eq 2
+
+    role_to_change = poro.roles.first
+    new_name = "#{role_to_change.name}_new"
+    role_to_change.name = new_name
+
+    expect(mapper.dump(poro).roles.first.name).to eq new_name
+  end
+
   scenario "lazy loads associated objects" do
     expect(a_active_record_object).not_to receive :salutation
     expect(a_active_record_object).not_to receive :roles

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: activerecord_to_poro
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.0.5
 platform: ruby
 authors:
 - Dieter Späth
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-06-27 00:00:00.000000000 Z
+date: 2014-07-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: abstract_type
@@ -84,16 +84,16 @@ dependencies:
   name: activerecord
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '='
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 4.0.2
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '='
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 4.0.2
 - !ruby/object:Gem::Dependency
   name: database_cleaner
   requirement: !ruby/object:Gem::Requirement
@@ -235,7 +235,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: pry-debugger
+  name: pry-byebug
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="