forest_liana 9.5.2 → 9.5.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 93e5e9bbc17d64a32b2d38ce4effa876d682c356dbf1d180eb5e0117db644604
-  data.tar.gz: 52b8528c2d513cd0b906d8d69f7a163053976cdfc09fdb8484f83702354c2b23
+  metadata.gz: '01219141321710917e3b67dcd37db280425c4cf1620c491172c4bc7ab5616047'
+  data.tar.gz: 12f0c3ae8d97f28a82e70f38c93545f0bed346607c7301a63436077a3e95e858
 SHA512:
-  metadata.gz: 40fd658783f4362c1ec6fd94224a160d31ee29277f87eee3bfc53223d91f4e7854be9ba6c0eecd645c39723c5e4338962fc70b785c53e0e527a9e687b8fc3660
-  data.tar.gz: 2620e9af72ecf6c7cd596ac4d596c36b10477b3ee0091cacb1758c233e890b85719c5acdfe678fffdc2e41e6c07c501585ddb6f81cd65b597a8864cd58187d06
+  metadata.gz: 775dbe23c00ecd54bd64cde35a6efc3c40d72d1e873fa9e11d7077c830a1778001d6a90f4c4e9373dcd782e39d371b6673dab6b8f8e7a539a44b0bda49d14f78
+  data.tar.gz: 42441b9c0e1df5f8d760b0216982865ce1c6193452c9fd2677708b1c3171cd162dee9ae61365013a8bb4790998ff2f3cac2e3fe96dc80bd64ecfc6bd003b7ae5

data/app/services/forest_liana/scope_manager.rb

@@ -49,20 +49,15 @@ module ForestLiana
     end
 
     def self.fetch_scopes(rendering_id)
-      response = ForestLiana::ForestApiRequester.get("/liana/v4/permissions/renderings/#{rendering_id}")
-
-      if response.is_a?(Net::HTTPOK)
-        Rails.cache.fetch('forest.scopes.' + rendering_id.to_s, expires_in: @@scope_cache_expiration_delta) do
-          data = {}
-          parse_response = JSON.parse(response.body)
-
-          data['scopes'] = decode_scope(parse_response['collections'])
-          data['team'] = parse_response['team']
-
-          data
-        end
-      else
-        raise 'Unable to fetch scopes'
+      Rails.cache.fetch("forest.scopes.#{rendering_id}", expires_in: @@scope_cache_expiration_delta) do
+        response = ForestLiana::ForestApiRequester.get("/liana/v4/permissions/renderings/#{rendering_id}")
+        raise 'Unable to fetch scopes' unless response.is_a?(Net::HTTPOK)
+        parsed_response = JSON.parse(response.body)
+
+        {
+          'scopes' => decode_scope(parsed_response['collections']),
+          'team' => parsed_response['team']
+        }
       end
     end
 

data/lib/forest_liana/version.rb

@@ -1,3 +1,3 @@
 module ForestLiana
-  VERSION = "9.5.2"
+  VERSION = "9.5.4"
 end

data/spec/services/forest_liana/scope_manager_spec.rb

@@ -73,6 +73,14 @@ module ForestLiana
               'team' => {'id' => '1', 'name' => 'Operations'},
             })
         end
+
+        it 'calls the API only once, even with multiple fetch_scopes calls' do
+          described_class.fetch_scopes(rendering_id)
+          described_class.fetch_scopes(rendering_id)
+
+          expect(ForestLiana::ForestApiRequester).to have_received(:get).once
+        end
+
       end
 
       describe 'when scope contains dynamic values' do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: forest_liana
 version: !ruby/object:Gem::Version
-  version: 9.5.2
+  version: 9.5.4
 platform: ruby
 authors:
 - Sandro Munda
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-11-18 00:00:00.000000000 Z
+date: 2024-12-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails