forest_liana 9.12.0 → 9.12.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 41634384199110d9a502915f8257c2e3ae3bb42553138cc3d178420de4ad2dea
-  data.tar.gz: 2c8ddb09bf770fd5f0dc8a051a0403a7caca8077becadf49444d11eede44eaab
+  metadata.gz: a205f59494a1a151c8bb1a5be912905bf7f1c9242136ae63de22f0372f7268bf
+  data.tar.gz: ddba735dde964ffe46c0dc2907edd065e707a3d63d8283f25b2155c903d3eeb5
 SHA512:
-  metadata.gz: 5c25ddcb57957aa97d0e2fbb5a480a95821fb73d379ad3df207b6be05abc12e8181be943fca1845f499a14ba2aaa7df7e01df80d67eb272579cc490be652ba4b
-  data.tar.gz: 055f53038f537efca379093b048376ac044a1eb8e6c65a0305a7643fa95b2bd74313d7b9258d1daef6a21e62da40e48bb30db0498ee2bc85b8f284815237aa17
+  metadata.gz: eaaf6fbca9592c4853bcdfa2ec6c5983ee0f468fd62d36aab03cf55dc865a7ae7195c079b8d05a185c174a81cadb834f87e7ff85c69f489336a58ceb65c76559
+  data.tar.gz: 3f2dc59f88da2c969f8cf865809232d0b58988cf1f395d59837b2df0d3ef094854274e82eca82c3432d8672d9a1c760dc0204c86347cbdff3ae5304b5f555323

data/app/services/forest_liana/resources_getter.rb

@@ -227,10 +227,20 @@ module ForestLiana
     def compute_select_fields
       select = ['_forest_admin_eager_load']
       @params[:fields][@collection_name].split(',').each do |path|
+        association = get_one_association(path)
+        if association
+          while association.options[:through]
+            association = get_one_association(association.options[:through])
+          end
+
+          if SchemaUtils.polymorphic?(association)
+            select << "#{@resource.table_name}.#{association.foreign_type}"
+          end
+          select << "#{@resource.table_name}.#{association.foreign_key}"
+        end
+
         if @params[:fields].key?(path)
-          association = ForestLiana::QueryHelper.get_one_associations(@resource)
-                                                .select { |association| association.name == path.to_sym }
-                                                .first
+          association = get_one_association(path)
           table_name = association.table_name
 
           @params[:fields][path].split(',').each do |association_path|
@@ -245,7 +255,13 @@ module ForestLiana
         end
       end
 
-      select
+      select.uniq
+    end
+
+    def get_one_association(name)
+      ForestLiana::QueryHelper.get_one_associations(@resource)
+        .select { |association| association.name == name.to_sym }
+        .first
     end
   end
 end

data/lib/forest_liana/version.rb

@@ -1,3 +1,3 @@
 module ForestLiana
-  VERSION = "9.12.0"
+  VERSION = "9.12.1"
 end

data/spec/dummy/app/models/tree.rb

@@ -7,4 +7,6 @@ class Tree < ActiveRecord::Base
     class_name: 'Island',
     inverse_of: :eponymous_tree,
     optional: true
+
+  has_one :location, through: :island
 end

data/spec/helpers/forest_liana/query_helper_spec.rb

@@ -37,6 +37,7 @@ module ForestLiana
             { name: :cutter, klass: User },
             { name: :island, klass: Island },
             { name: :eponymous_island, klass: Island },
+            { name: :location, klass: Location },
           ]
         end
 
@@ -52,13 +53,17 @@ module ForestLiana
 
     describe 'get_one_association_names_symbol' do
       it 'should return the one-one associations names as symbols' do
-        expect(QueryHelper.get_one_association_names_symbol(Tree)).to eq([:owner, :cutter, :island, :eponymous_island])
+        expect(QueryHelper.get_one_association_names_symbol(Tree)).to eq(
+          [:owner, :cutter, :island, :eponymous_island, :location]
+        )
       end
     end
 
     describe 'get_one_association_names_string' do
       it 'should return the one-one associations names as strings' do
-        expect(QueryHelper.get_one_association_names_string(Tree)).to eq(['owner', 'cutter', 'island', 'eponymous_island'])
+        expect(QueryHelper.get_one_association_names_string(Tree)).to eq(
+          ['owner', 'cutter', 'island', 'eponymous_island', 'location']
+        )
       end
     end
 
@@ -71,12 +76,12 @@ module ForestLiana
         end
       end
 
-      context 'on a model having 2 belongsTo associations' do
+      context 'on a model having 3 belongsTo/hasOne associations' do
         tables_associated_to_relations_name =
           QueryHelper.get_tables_associated_to_relations_name(Tree)
 
         it 'should return the one-one associations' do
-          expect(tables_associated_to_relations_name.keys.length).to eq(2)
+          expect(tables_associated_to_relations_name.keys.length).to eq(3)
         end
 
         it 'should return relationships having a name different than the targeted model' do

data/spec/requests/resources_spec.rb

@@ -4,7 +4,9 @@ describe 'Requesting Tree resources', :type => :request  do
   let(:scope_filters) { {'scopes' => {}, 'team' => {'id' => '1', 'name' => 'Operations'}} }
   before do
     user = User.create(name: 'Michel')
-    Tree.create(name: 'Lemon Tree', owner: user, cutter: user)
+    tree = Tree.create(name: 'Lemon Tree', owner: user, cutter: user)
+    island = Island.create(name: 'Lemon Island', trees: [tree])
+    Location.create(coordinates: '1,2', island: island)
 
     Rails.cache.write('forest.users', {'1' => { 'id' => 1, 'roleId' => 1, 'rendering_id' => '1' }})
     Rails.cache.write('forest.has_permission', true)
@@ -32,6 +34,8 @@ describe 'Requesting Tree resources', :type => :request  do
   after do
     User.destroy_all
     Tree.destroy_all
+    Island.destroy_all
+    Location.destroy_all
   end
 
   token = JWT.encode({
@@ -54,7 +58,7 @@ describe 'Requesting Tree resources', :type => :request  do
   describe 'index' do
     describe 'without any filter' do
       params = {
-        fields: { 'Tree' => 'id,name' },
+        fields: { 'Tree' => 'id,name,location' },
         page: { 'number' => '1', 'size' => '10' },
         searchExtended: '0',
         sort: '-id',
@@ -98,7 +102,8 @@ describe 'Requesting Tree resources', :type => :request  do
 
       it 'should respond the tree data' do
         get '/forest/Tree', params: params, headers: headers
-        expect(JSON.parse(response.body)).to eq({
+        
+        expect(JSON.parse(response.body)).to include({
           "data" => [{
             "type" => "Tree",
             "id" => "1",
@@ -108,9 +113,30 @@ describe 'Requesting Tree resources', :type => :request  do
             },
             "links" => {
               "self" => "/forest/tree/1"
+            },
+            "relationships" => {
+              "location" => {
+                "data" => { "id" => "1", "type" => "Location" },
+                "links" => { "related" => {} }
+              }
             }
           }],
-          "included" => []
+          "included" => [{
+            "type" => "Location",
+            "id" => "1",
+            "attributes" => include(
+              "id" => 1,
+              "created_at" => nil,
+              "updated_at" => nil,
+              "coordinates" => nil
+            ),
+            "links" => { "self" => "/forest/location/1" },
+            "relationships" => {
+              "island" => {
+                "links" => { "related" => {} }
+              }
+            }
+          }]
         })
       end
     end

data/spec/services/forest_liana/resources_getter_spec.rb

@@ -290,6 +290,22 @@ module ForestLiana
       end
     end
 
+    describe 'when getting a has_one through association' do
+      let(:resource) { Tree }
+      let(:fields) { { 'Tree' => 'id,location' } }
+
+      it 'should get the expected records, including the foreign_key for the direct association' do
+        getter.perform
+        records = getter.records
+        count = getter.count
+
+        expect(records.count).to eq Tree.count
+        expect(count).to eq Tree.count
+        expect(records.map(&:id)).to match_array(Tree.pluck(:id))
+        expect(records.map(&:island_id)).to match_array(Tree.pluck(:island_id))
+      end
+    end
+
     describe 'when filtering on an ambiguous field' do
       let(:resource) { Tree }
       let(:pageSize) { 5 }

data/spec/services/forest_liana/schema_adapter_spec.rb

@@ -61,7 +61,18 @@ module ForestLiana
           end
 
           expect(collection.fields.map { |field| field[:field].to_s}).to eq(
-            ["age", "created_at", "cutter", "eponymous_island", "id", "island", "name", "owner", "updated_at"]
+            [
+              "age",
+              "created_at",
+              "cutter",
+              "eponymous_island",
+              "id",
+              "island",
+              "location",
+              "name",
+              "owner",
+              "updated_at"
+            ]
           )
         end
       end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: forest_liana
 version: !ruby/object:Gem::Version
-  version: 9.12.0
+  version: 9.12.1
 platform: ruby
 authors:
 - Sandro Munda
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-05-05 00:00:00.000000000 Z
+date: 2025-05-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails