graphql-fragment_cache 1.13.0 → 1.13.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: af4f8899ace77b66aaabd9af9292de07a40d2384ef8db4a24394e75bdd4be78d
-  data.tar.gz: 21b141a16d460b4d1b8b6cf1146e54898a87c45b4f670aa0ab6cbaac78dc48d3
+  metadata.gz: 9fe4856af12987ff49d92ed9551c46cb64634d48a0112c323436e953ec55990a
+  data.tar.gz: 0d69c3d4096474724b0ec6b409bb5ecdbbcda5b81f20e5ae035e1ae1c77f18ce
 SHA512:
-  metadata.gz: 6af77b269f6712c5c39983a796c0f109d6098297de2ad2c2fd87b5f4b8c0e53f9b71d14a2d6e7d8eb0ec1f03c55f43f84f766cdc4663bcf0cd8c7c1148fef506
-  data.tar.gz: e396f56ee9182d1147662fb484264ead58ea34efb867a9c6be9d1d018b94f9fb69089b981cb510c693ecd142a9c1fc20280d34c26bab71d9dc40af45d471944b
+  metadata.gz: 19f8d2bb9bc21a2bb0b79244e230fbede1b19e60beeadee2d0e7197b4a531d044bc5ca85120300c70a7ef2892e125a2a89b820e86a6aad14c5bd7e4fc572ebbf
+  data.tar.gz: 136ba3362d1c7fddad10dac6c27f4f3b95b624d0c14dedef3cd28afa63a61330a5d4f95b92a2aeed47b70dfbfb5bf7690cc0f9695561d56a1e34d0fd2ebfd691

data/CHANGELOG.md

@@ -2,6 +2,10 @@
 
 ## master
 
+## 1.13.1 (2022-10-12)
+
+- [PR#84](https://github.com/DmitryTsepelev/graphql-ruby-fragment_cache/pull/84) Fix Renew Cache Read Multi Bug ([@KTSCode][])
+
 ## 1.13.0 (2022-09-12)
 
 - [PR#83](https://github.com/DmitryTsepelev/graphql-ruby-fragment_cache/pull/83) Update Lookahead usage to support graphql-2.0.14 ([@DmitryTsepelev][])
@@ -138,3 +142,4 @@
 [@mretzak]: https://github.com/mretzak
 [@daukadolt]: https://github.com/daukadolt
 [@frostmark]: https://github.com/frostmark
+[@KTSCode]: https://github.com/KTSCode

data/lib/graphql/fragment_cache/fragment.rb

@@ -19,15 +19,20 @@ module GraphQL
           fragments_to_cache_keys = fragments
             .map { |f| [f, f.cache_key] }.to_h
 
-          cache_keys = fragments_to_cache_keys.values
-
-          cache_keys_to_values = FragmentCache.cache_store.read_multi(*cache_keys)
-
-          fetched_fragments_to_values = cache_keys_to_values
-            .map { |key, val| [fragments_to_cache_keys.key(key), val] }
-            .to_h
+          # Filter out all the cache_keys for fragments with renew_cache: true in their context
+          cache_keys = fragments_to_cache_keys
+            .reject { |k, _v| k.context[:renew_cache] == true }.values
+
+          # If there are cache_keys look up values with read_multi otherwise return an empty hash
+          cache_keys_to_values = if cache_keys.empty?
+            {}
+          else
+            FragmentCache.cache_store.read_multi(*cache_keys)
+          end
 
-          fetched_fragments_to_values
+          # Fragmenst without values or with renew_cache: true in their context will have nil values like the read method
+          fragments_to_cache_keys
+            .map { |fragment, cache_key| [fragment, cache_keys_to_values[cache_key]] }.to_h
         end
       end
 

data/lib/graphql/fragment_cache/version.rb

@@ -2,6 +2,6 @@
 
 module GraphQL
   module FragmentCache
-    VERSION = "1.13.0"
+    VERSION = "1.13.1"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: graphql-fragment_cache
 version: !ruby/object:Gem::Version
-  version: 1.13.0
+  version: 1.13.1
 platform: ruby
 authors:
 - DmitryTsepelev
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-09-12 00:00:00.000000000 Z
+date: 2022-10-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: graphql