caprese 0.3.3 → 0.3.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 1064cb8d4344aa314d5cf4a3a851a4e52fad8ce7
-  data.tar.gz: 12be18352ab0c879d4c1bb138faa785b0ae5a28a
+  metadata.gz: 0ca4c84e5919ce00359d805fbbbfd4da7e26c769
+  data.tar.gz: bd3df0c2fc3a9f29f91c9577c18ebf9a48b9a9df
 SHA512:
-  metadata.gz: f02cc3064dbc79f29464be3222073bbfd2da45c85c63f3a96ce566ae5a45fd48b835ee54260b942ff70fc25bb9f41a6383302f9e0fd45fd5f86d54fea3cad240
-  data.tar.gz: 0df51ffba9329b200042af7521230213763ea0ada30799a50786160dcfea0500af1f02926bdf945ba058203e470d6ced4c36e21fba3785a9b6d304a28cf05f08
+  metadata.gz: 46b96f45c77864c3ae31ec75291a0ae9e1e4c3ce4480c952896c5a07751cd799678fe9d525237abb16ba8df485d76340232819cb390ab374916352fadbb32471
+  data.tar.gz: b8fad22479e137399a200458e706bf632c18dceda7be71b8e61fa3295381a773100fd232852d48639ca695df6449fbcb697c4589d775454adb878c10035ecdf0

data/CHANGELOG.md

@@ -18,3 +18,8 @@
 ## 0.3.3
 
 * Depend on all route primary key URL params to be named `:id`, rather than `Caprese.config.resource_primary_key`
+
+## 0.3.4
+
+* Fix bug in processing of collection relationship data (5e21582)
+* Fix bug in persisting of collection relationship data on create (1eb4720)

data/lib/caprese/controller/concerns/persistence.rb

@@ -50,6 +50,8 @@ module Caprese
 
       record.save!
 
+      persist_collection_relationships(record)
+
       execute_after_create_callbacks(record)
       execute_after_save_callbacks(record)
 
@@ -242,12 +244,12 @@ module Caprese
     # @param [Hash,Array<Hash>] relationship_data the resource identifier data to use to find/build records
     # @return [ActiveRecord::Base,Array<ActiveRecord::Base>] the record(s) for the relationship
     def records_for_relationship(owner, permitted_params, relationship_name, relationship_data)
-      if relationship_data.is_a?(Array)
-        relationship_data.map do |relationship_data_item|
-          ref = record_for_relationship(owner, relationship_name, relationship_data_item[:data])
+      if relationship_data[:data].is_a?(Array)
+        relationship_data[:data].map do |relationship_data_item|
+          ref = record_for_relationship(owner, relationship_name, relationship_data_item)
 
-          if ref && contains_constructable_data?(relationship_data_item[:data])
-            assign_record_attributes(ref, permitted_params, relationship_data_item[:data])
+          if ref && contains_constructable_data?(relationship_data_item)
+            assign_record_attributes(ref, permitted_params, relationship_data_item)
           end
 
           ref
@@ -316,5 +318,25 @@ module Caprese
     def contains_constructable_data?(resource_identifier)
       [:attributes, :relationships].any? { |k| resource_identifier.key?(k) }
     end
+
+    # Called in create, after the record is saved. When creating a new record, and assigning to it
+    # existing has_many association relation, the records in the relation will be pushed onto the
+    # appropriate target, but the relationship will not be persisted in their attributes until their
+    # owner is saved.
+    #
+    # This methods persists the collection relation(s) pushed onto the record's association target(s)
+    def persist_collection_relationships(record)
+      record.class.reflect_on_all_associations
+      .select { |ref| ref.collection? && record.association(ref.name).any? }
+      .map do |ref|
+        [
+          ref.has_inverse? ? ref.inverse_of.name : ref.options[:as],
+          record.association(ref.name).target
+        ]
+      end
+      .to_h.each do |name, targets|
+        targets.each { |t| t.update name => record }
+      end
+    end
   end
 end

data/lib/caprese/version.rb

@@ -1,3 +1,3 @@
 module Caprese
-  VERSION = '0.3.3'
+  VERSION = '0.3.4'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: caprese
 version: !ruby/object:Gem::Version
-  version: 0.3.3
+  version: 0.3.4
 platform: ruby
 authors:
 - Nick Landgrebe
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2017-02-07 00:00:00.000000000 Z
+date: 2017-02-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: active_model_serializers