graphiti 1.2.6 → 1.2.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f71f80174c5c0982ea1f0d3c63ebcbccb76a9f6bac5b632e6a7a8b39460b2b8a
-  data.tar.gz: 48d23a31698373be0953c8a77e24e2f6cf4f0711e0bdee25ddcf4c74125f04cf
+  metadata.gz: 1d04305bd43a58f1a835af8e80a03f599430303cc967a99f4f662679eb5c0968
+  data.tar.gz: 869acd442f3b00895c72023a20ed9080f06559e0ba619b05bc2366169b776b72
 SHA512:
-  metadata.gz: 02ae503dfeedd57ccd8a25e8f14432003dd11c2e20591fc21b26f1e8ea0afdd75c8be551ea6520e2da29f95a0d589b70fdad2d37b9be45c10ff813004fa1b417
-  data.tar.gz: 3f5e8ed4262a7e16b45068102ace2afadc43e5e8863f91d87db93573bc66e3cf85cce1889d4afd2fdabc691f2301e3b9499a4c4264bbe6cf7801ba333d932caa
+  metadata.gz: 82c3cc4fe7cfcc01deca4594460894c00c367f267a6d080bdc98b104e5d29b247ca858a80afc513faeeafc6e5d9fc12fb28dc36765a25d2dc2eab453bb2d5a08
+  data.tar.gz: 5351083894448c44d55cc88de12c2ad81c6319c8e2e7c7afc1a3a3db40b4abff30b51d90ed210cc2c940c013bf367d4f9f06c66b93954836febe2ba316ba6643

data/lib/graphiti/sideload/polymorphic_belongs_to.rb

@@ -114,8 +114,8 @@ class Graphiti::Sideload::PolymorphicBelongsTo < Graphiti::Sideload::BelongsTo
 
       match = ->(c) { c.group_name == group_name.to_sym }
       if (sideload = children.values.find(&match))
-        query = remove_invalid_sideloads(sideload.resource, query)
-        sideload.resolve(group, query, graph_parent)
+        duped = remove_invalid_sideloads(sideload.resource, query)
+        sideload.resolve(group, duped, graph_parent)
       else
         err = ::Graphiti::Errors::PolymorphicSideloadChildNotFound
         raise err.new(self, group_name)
@@ -127,14 +127,16 @@ class Graphiti::Sideload::PolymorphicBelongsTo < Graphiti::Sideload::BelongsTo
 
   # We may be requesting a relationship that some subclasses support,
   # but not others. Remove anything we don't support.
+  # TODO: spec to ensure this dupe logic doesn't mutate the original
   def remove_invalid_sideloads(resource, query)
-    query = query.dup
-    query.instance_variable_set(:@hash, nil)
-    query.sideloads.each_pair do |key, value|
+    duped = query.dup
+    duped.instance_variable_set(:@hash, nil)
+    duped.instance_variable_set(:@sideloads, ::Graphiti::Util::Hash.deep_dup(query.sideloads))
+    duped.sideloads.each_pair do |key, value|
       unless resource.class.sideload(key)
-        query.sideloads.delete(key)
+        duped.sideloads.delete(key)
       end
     end
-    query
+    duped
   end
 end

data/lib/graphiti/version.rb

@@ -1,3 +1,3 @@
 module Graphiti
-  VERSION = "1.2.6"
+  VERSION = "1.2.7"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: graphiti
 version: !ruby/object:Gem::Version
-  version: 1.2.6
+  version: 1.2.7
 platform: ruby
 authors:
 - Lee Richmond
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2019-10-08 00:00:00.000000000 Z
+date: 2019-10-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: jsonapi-serializable
@@ -344,8 +344,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.6
+rubygems_version: 3.0.6
 signing_key: 
 specification_version: 4
 summary: Easily build jsonapi.org-compatible APIs