forest_liana 9.17.0 → 9.17.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 79bda2b54d1daacca266bd2f1b8197ef59a8964c46d895fda1c240bdbcc6b7d1
-  data.tar.gz: 1f60f2be8749c3d3e3aeb9208e4cae70cd3616030a6563980ca843743186073c
+  metadata.gz: 44ec79f4e42ec91b8c5306d87672aab03a6b30072e88e27fdcb311d3d7d2c8de
+  data.tar.gz: e5f37934ccdbf40a78499fd22fcafcd037b8eed1b12034c773bbb499d16b60c7
 SHA512:
-  metadata.gz: 9a3754ed9509e127884404b37c1afb9a059522379673b48514c511dc5f94bb97c0dfef68bc2d7284dae25dea580875a772bae8eb464d529ab79c1eb96315c459
-  data.tar.gz: 89c13361f05f461155aae07c8f63b2524513ff50342506d9e7139ea5c9dcbea7fcaf67166a27f6601199696861d4acc611d7d4461e0aa6c4f75998b2ea9bd0a4
+  metadata.gz: 93d789834be64fe90cdc9b58c8201df5b75396d627a1138ec1d46e437cfb60d7d63246ece332db3890a6a0cb5eb672683655abc5736dc5c85d38ab2aea47e41a
+  data.tar.gz: 91b3c9bf5856f4b628ca31b4140bab3f00b96bd3a7afe670cefaa8701499a86e2fca9e0c431b7a621c86283bae0eefa13afb5104f25ad89589105eb007fa34b9

data/app/services/forest_liana/base_getter.rb

@@ -1,5 +1,7 @@
 module ForestLiana
   class BaseGetter
+    include ForestLiana::RecordFindable
+
     def get_collection(collection_name)
       ForestLiana.apimap.find { |collection| collection.name.to_s == collection_name }
     end

data/app/services/forest_liana/belongs_to_updater.rb

@@ -1,5 +1,7 @@
 module ForestLiana
   class BelongsToUpdater
+    include ForestLiana::RecordFindable
+
     attr_accessor :errors
 
     def initialize(resource, association, params)
@@ -12,7 +14,7 @@ module ForestLiana
 
     def perform
       begin
-        @record = @resource.find(@params[:id])
+        @record = find_record(@resource, @resource, @params[:id])
         if (SchemaUtils.polymorphic?(@association))
           if @data.nil?
             new_value = nil

data/app/services/forest_liana/has_many_associator.rb

@@ -1,5 +1,7 @@
 module ForestLiana
   class HasManyAssociator
+    include ForestLiana::RecordFindable
+
     def initialize(resource, association, params)
       @resource = resource
       @association = association
@@ -8,8 +10,8 @@ module ForestLiana
     end
 
     def perform
-      @record = @resource.find(@params[:id])
-      associated_records = @resource.find(@params[:id]).send(@association.name)
+      @record = find_record(@resource, @resource, @params[:id])
+      associated_records = @record.send(@association.name)
 
       if @data.is_a?(Array)
         @data.each do |record_added|

data/app/services/forest_liana/has_many_dissociator.rb

@@ -1,5 +1,7 @@
 module ForestLiana
   class HasManyDissociator
+    include ForestLiana::RecordFindable
+
     def initialize(resource, association, params, forest_user)
       @resource = resource
       @association = association
@@ -10,8 +12,8 @@ module ForestLiana
     end
 
     def perform
-      @record = @resource.find(@params[:id])
-      associated_records = @resource.find(@params[:id]).send(@association.name)
+      @record = find_record(@resource, @resource, @params[:id])
+      associated_records = @record.send(@association.name)
 
       remove_association = !@with_deletion || @association.macro == :has_and_belongs_to_many
       if @data.is_a?(Array)

data/app/services/forest_liana/has_many_getter.rb

@@ -29,6 +29,9 @@ module ForestLiana
 
       if association_class.primary_key.is_a?(Array)
         adapter_name = association_class.connection.adapter_name.downcase
+        pk_columns = association_class.primary_key.map do |pk|
+          "#{association_class.table_name}.#{pk}"
+        end.join(', ')
 
         if adapter_name.include?('sqlite')
           # For SQLite: concatenate columns for DISTINCT count
@@ -37,12 +40,9 @@ module ForestLiana
           end.join(" || '|' || ")
 
           @records_count = @records.distinct.count(Arel.sql(pk_concat))
+        elsif adapter_name.include?('postgresql')
+          @records_count = @records.distinct.count(Arel.sql("ROW(#{pk_columns})"))
         else
-          # For PostgreSQL/MySQL: use DISTINCT with multiple columns
-          pk_columns = association_class.primary_key.map do |pk|
-            "#{association_class.table_name}.#{pk}"
-          end.join(', ')
-
           @records_count = @records.distinct.count(Arel.sql(pk_columns))
         end
       else
@@ -94,7 +94,7 @@ module ForestLiana
     end
 
     def prepare_query
-      parent_record = ForestLiana::Utils::CompositePrimaryKeyHelper.find_record(get_resource(), @resource, @params[:id])
+      parent_record = find_record(get_resource(), @resource, @params[:id])
       association = parent_record.send(@params[:association_name])
       @records = optimize_record_loading(association, @search_query_builder.perform(association))
     end

data/app/services/forest_liana/record_findable.rb

@@ -0,0 +1,27 @@
+module ForestLiana
+  module RecordFindable
+    private
+
+    def find_record(scope, resource, id)
+      primary_key = resource.primary_key
+
+      if primary_key.is_a?(Array)
+        id_values = parse_composite_id(id)
+        conditions = primary_key.zip(id_values).to_h
+        scope.find_by(conditions)
+      else
+        scope.find(id)
+      end
+    end
+
+    def parse_composite_id(id)
+      return id if id.is_a?(Array)
+
+      if id.to_s.start_with?('[') && id.to_s.end_with?(']')
+        JSON.parse(id.to_s)
+      else
+        raise ForestLiana::Errors::HTTP422Error.new("Composite primary key ID must be in format [value1,value2], received: #{id}")
+      end
+    end
+  end
+end

data/app/services/forest_liana/resource_getter.rb

@@ -14,7 +14,7 @@ module ForestLiana
     def perform
       records = optimize_record_loading(@resource, get_resource())
       scoped_records = ForestLiana::ScopeManager.apply_scopes_on_records(records, @user, @collection_name, @params[:timezone])
-      @record = ForestLiana::Utils::CompositePrimaryKeyHelper.find_record(scoped_records, @resource, @params[:id])
+      @record = find_record(scoped_records, @resource, @params[:id])
     end
   end
 end

data/app/services/forest_liana/resource_updater.rb

@@ -1,5 +1,7 @@
 module ForestLiana
   class ResourceUpdater
+    include ForestLiana::RecordFindable
+
     attr_accessor :record
     attr_accessor :errors
 
@@ -14,7 +16,7 @@ module ForestLiana
       begin
         collection_name = ForestLiana.name_for(@resource)
         scoped_records = ForestLiana::ScopeManager.apply_scopes_on_records(@resource, @user, collection_name, @params[:timezone])
-        @record = scoped_records.find(@params[:id])
+        @record = find_record(scoped_records, @resource, @params[:id])
 
         if has_strong_parameter
           @record.update(resource_params)

data/app/services/forest_liana/schema_adapter.rb

@@ -454,7 +454,31 @@ module ForestLiana
     def add_default_value(column_schema, column)
       # TODO: detect/introspect the attribute default value with Rails 5
       #       ex: attribute :email, :string, default: 'arnaud@forestadmin.com'
-      column_schema[:default_value] = column.default if column.default
+
+      unless column.default.nil?
+        column_schema[:default_value] = normalize_default_value(column)
+      end
+    end
+
+    def normalize_default_value(column)
+      # In Ruby 4.0+ and Rails 8+, ActiveRecord returns default values as strings
+      case column.type
+      when :boolean
+        case column.default.to_s
+        when '0', 'f', 'false'
+          false
+        when '1', 't', 'true'
+          true
+        else
+          column.default
+        end
+      when :integer
+        column.default.to_i
+      when :float, :decimal
+        column.default.to_f
+      else
+        column.default
+      end
     end
 
     def add_validations(column_schema, column)

data/lib/forest_liana/version.rb

@@ -1,3 +1,3 @@
 module ForestLiana
-  VERSION = "9.17.0"
+  VERSION = "9.17.2"
 end

data/spec/services/forest_liana/composite_primary_keys_spec.rb

@@ -0,0 +1,223 @@
+require 'rails_helper'
+
+module ForestLiana
+  describe 'Composite Primary Keys Support' do
+    let(:rendering_id) { 13 }
+    let(:user) { { 'id' => '1', 'rendering_id' => rendering_id } }
+    let(:scopes) { { 'scopes' => {}, 'team' => { 'id' => '1', 'name' => 'Operations' } } }
+
+    before do
+      ForestLiana::ScopeManager.invalidate_scope_cache(rendering_id)
+      allow(ForestLiana::ScopeManager).to receive(:fetch_scopes).and_return(scopes)
+    end
+
+    describe RecordFindable do
+      # Create a test class that includes the module to test private methods
+      let(:test_class) do
+        Class.new do
+          include ForestLiana::RecordFindable
+          # Expose private methods for testing
+          public :find_record, :parse_composite_id
+        end
+      end
+
+      let(:helper) { test_class.new }
+
+      describe '#parse_composite_id' do
+        it 'correctly parses composite ID in JSON format' do
+          expect(helper.parse_composite_id('[1,2]')).to eq([1, 2])
+          expect(helper.parse_composite_id('[10,20]')).to eq([10, 20])
+          expect(helper.parse_composite_id('["a","b"]')).to eq(['a', 'b'])
+        end
+
+        it 'returns array as-is when already an array' do
+          expect(helper.parse_composite_id([1, 2])).to eq([1, 2])
+        end
+
+        it 'raises error for invalid composite ID format' do
+          expect {
+            helper.parse_composite_id('invalid')
+          }.to raise_error(ForestLiana::Errors::HTTP422Error)
+        end
+      end
+
+      describe '#find_record' do
+        it 'finds record using composite key conditions' do
+          mock_resource = double('Resource', primary_key: [:user_id, :island_id])
+          mock_scoped_records = double('ScopedRecords')
+          mock_record = double('Record')
+
+          allow(mock_scoped_records).to receive(:find_by)
+            .with({ user_id: 1, island_id: 2 })
+            .and_return(mock_record)
+
+          result = helper.find_record(mock_scoped_records, mock_resource, '[1,2]')
+
+          expect(result).to eq(mock_record)
+          expect(mock_scoped_records).to have_received(:find_by).with({ user_id: 1, island_id: 2 })
+        end
+
+        it 'falls back to standard find for simple primary key' do
+          mock_resource = double('Resource', primary_key: 'id')
+          mock_scoped_records = double('ScopedRecords')
+          mock_record = double('Record')
+
+          allow(mock_scoped_records).to receive(:find).with(123).and_return(mock_record)
+
+          result = helper.find_record(mock_scoped_records, mock_resource, 123)
+
+          expect(result).to eq(mock_record)
+          expect(mock_scoped_records).to have_received(:find).with(123)
+        end
+      end
+    end
+
+    describe HasManyGetter do
+      describe '#count with composite primary key' do
+        let(:composite_association_class) do
+          mock_class = double('CompositeModel')
+          allow(mock_class).to receive(:primary_key).and_return([:user_id, :island_id])
+          allow(mock_class).to receive(:table_name).and_return('user_islands')
+          allow(mock_class).to receive(:connection).and_return(double('Connection', adapter_name: adapter_name))
+          mock_class
+        end
+
+        let(:mock_records) { double('Records') }
+
+        before do
+          allow(mock_records).to receive(:distinct).and_return(mock_records)
+          allow(mock_records).to receive(:count).and_return(5)
+        end
+
+        context 'with PostgreSQL adapter' do
+          let(:adapter_name) { 'PostgreSQL' }
+
+          it 'uses ROW() syntax for COUNT DISTINCT' do
+            getter = HasManyGetter.allocate
+            getter.instance_variable_set(:@records, mock_records)
+
+            allow(getter).to receive(:model_association).and_return(composite_association_class)
+
+            getter.count
+
+            expect(mock_records).to have_received(:count) do |arg|
+              expect(arg.to_s).to include('ROW(')
+              expect(arg.to_s).to include('user_islands.user_id')
+              expect(arg.to_s).to include('user_islands.island_id')
+            end
+          end
+        end
+
+        context 'with MySQL adapter' do
+          let(:adapter_name) { 'Mysql2' }
+
+          it 'uses standard multi-column syntax for COUNT DISTINCT' do
+            getter = HasManyGetter.allocate
+            getter.instance_variable_set(:@records, mock_records)
+
+            allow(getter).to receive(:model_association).and_return(composite_association_class)
+
+            getter.count
+
+            expect(mock_records).to have_received(:count) do |arg|
+              expect(arg.to_s).not_to include('ROW(')
+              expect(arg.to_s).to include('user_islands.user_id, user_islands.island_id')
+            end
+          end
+        end
+
+        context 'with SQLite adapter' do
+          let(:adapter_name) { 'SQLite' }
+
+          it 'uses concatenation syntax for COUNT DISTINCT' do
+            getter = HasManyGetter.allocate
+            getter.instance_variable_set(:@records, mock_records)
+
+            allow(getter).to receive(:model_association).and_return(composite_association_class)
+
+            getter.count
+
+            expect(mock_records).to have_received(:count) do |arg|
+              expect(arg.to_s).to include("||")
+              expect(arg.to_s).to include("'|'")
+            end
+          end
+        end
+      end
+    end
+
+    describe BelongsToUpdater do
+      describe 'with composite primary key parent' do
+        it 'uses find_record to find the parent record' do
+          composite_model = double('CompositeModel')
+          association = double('Association', name: :user, klass: User)
+          mock_record = double('Record', save: true)
+          allow(mock_record).to receive(:user=)
+
+          params = { id: '[1,2]', 'data' => { id: '5', type: 'User' } }
+
+          updater = described_class.new(composite_model, association, params)
+          allow(updater).to receive(:find_record).and_return(mock_record)
+          allow(User).to receive(:find).and_return(double('User'))
+          allow(SchemaUtils).to receive(:polymorphic?).and_return(false)
+
+          updater.perform
+
+          expect(updater).to have_received(:find_record)
+            .with(composite_model, composite_model, '[1,2]')
+        end
+      end
+    end
+
+    describe HasManyAssociator do
+      describe 'with composite primary key parent' do
+        it 'uses find_record to find the parent record' do
+          composite_model = double('CompositeModel')
+          association = double('Association', name: :trees, klass: Tree)
+          mock_record = double('Record')
+          mock_associated_records = double('AssociatedRecords')
+
+          allow(mock_record).to receive(:send).with(:trees).and_return(mock_associated_records)
+          allow(mock_associated_records).to receive(:<<)
+
+          params = { id: '[1,2]', 'data' => [{ id: '5' }] }
+
+          associator = described_class.new(composite_model, association, params)
+          allow(associator).to receive(:find_record).and_return(mock_record)
+          allow(Tree).to receive(:find).and_return(double('Tree'))
+
+          associator.perform
+
+          expect(associator).to have_received(:find_record)
+            .with(composite_model, composite_model, '[1,2]')
+        end
+      end
+    end
+
+    describe HasManyDissociator do
+      describe 'with composite primary key parent' do
+        it 'uses find_record to find the parent record' do
+          composite_model = double('CompositeModel')
+          association = double('Association', name: :trees, klass: Tree, macro: :has_many)
+          mock_record = double('Record')
+          mock_associated_records = double('AssociatedRecords')
+
+          allow(mock_record).to receive(:send).with(:trees).and_return(mock_associated_records)
+          allow(mock_associated_records).to receive(:delete)
+
+          params = { id: '[1,2]', 'data' => [{ id: '5' }], delete: 'false' }
+          forest_user = user
+
+          dissociator = described_class.new(composite_model, association, params, forest_user)
+          allow(dissociator).to receive(:find_record).and_return(mock_record)
+          allow(Tree).to receive(:find).and_return(double('Tree'))
+
+          dissociator.perform
+
+          expect(dissociator).to have_received(:find_record)
+            .with(composite_model, composite_model, '[1,2]')
+        end
+      end
+    end
+  end
+end

data/spec/services/forest_liana/schema_adapter_default_values_spec.rb

@@ -0,0 +1,157 @@
+module ForestLiana
+  describe SchemaAdapter do
+    describe 'normalize_default_value' do
+      context 'with boolean fields' do
+        before(:each) do
+          @original_apimap = ForestLiana.apimap.dup
+          @original_models = ForestLiana.models.dup
+
+          Object.const_set(:BooleanFieldWithDefault, Class.new(ActiveRecord::Base) do
+            self.table_name = 'boolean_fields_with_defaults'
+          end)
+
+          ActiveRecord::Migration.suppress_messages do
+            if ActiveRecord::Base.connection.table_exists?('boolean_fields_with_defaults')
+              ActiveRecord::Base.connection.drop_table('boolean_fields_with_defaults')
+            end
+            ActiveRecord::Base.connection.create_table('boolean_fields_with_defaults') do |t|
+              t.boolean :active, default: false
+              t.boolean :verified, default: true
+              t.boolean :nullable
+            end
+          end
+        end
+
+        after(:each) do
+          ForestLiana.apimap = @original_apimap
+          ForestLiana.models = @original_models
+
+          ActiveRecord::Migration.suppress_messages do
+            ActiveRecord::Base.connection.drop_table('boolean_fields_with_defaults') if ActiveRecord::Base.connection.table_exists?('boolean_fields_with_defaults')
+          end
+          Object.send(:remove_const, :BooleanFieldWithDefault) if Object.const_defined?(:BooleanFieldWithDefault)
+        end
+
+        it 'should convert boolean default values to proper booleans' do
+          ForestLiana.models = [BooleanFieldWithDefault]
+          ForestLiana.apimap = []
+
+          adapter = SchemaAdapter.new(BooleanFieldWithDefault)
+          collection = adapter.perform
+
+          active_field = collection.fields.find { |f| f[:field] == 'active' }
+          verified_field = collection.fields.find { |f| f[:field] == 'verified' }
+          nullable_field = collection.fields.find { |f| f[:field] == 'nullable' }
+
+          expect(active_field[:default_value]).to eq(false)
+          expect(active_field[:default_value].class).to eq(FalseClass)
+
+          expect(verified_field[:default_value]).to eq(true)
+          expect(verified_field[:default_value].class).to eq(TrueClass)
+
+          expect(nullable_field[:default_value]).to be_nil
+        end
+      end
+
+      context 'with enum fields' do
+        before(:each) do
+          @original_apimap = ForestLiana.apimap.dup
+          @original_models = ForestLiana.models.dup
+
+          Object.const_set(:EnumFieldModel, Class.new(ActiveRecord::Base) do
+            self.table_name = 'enum_field_models'
+            enum status: { inactive: 0, active: 1, archived: 2 }
+            enum role: { user: "0", admin: "1", superadmin: "2" }
+          end)
+
+          ActiveRecord::Migration.suppress_messages do
+            if ActiveRecord::Base.connection.table_exists?('enum_field_models')
+              ActiveRecord::Base.connection.drop_table('enum_field_models')
+            end
+            ActiveRecord::Base.connection.create_table('enum_field_models') do |t|
+              t.integer :status, default: 0
+              t.integer :role, default: 1
+            end
+          end
+        end
+
+        after(:each) do
+          ForestLiana.apimap = @original_apimap
+          ForestLiana.models = @original_models
+
+          ActiveRecord::Migration.suppress_messages do
+            ActiveRecord::Base.connection.drop_table('enum_field_models') if ActiveRecord::Base.connection.table_exists?('enum_field_models')
+          end
+          Object.send(:remove_const, :EnumFieldModel) if Object.const_defined?(:EnumFieldModel)
+        end
+
+        it 'should convert enum default values to integers' do
+          ForestLiana.models = [EnumFieldModel]
+          ForestLiana.apimap = []
+
+          adapter = SchemaAdapter.new(EnumFieldModel)
+          collection = adapter.perform
+
+          status_field = collection.fields.find { |f| f[:field] == 'status' }
+          role_field = collection.fields.find { |f| f[:field] == 'role' }
+
+          expect(status_field[:default_value]).to eq(0)
+          expect(status_field[:default_value].class).to eq(Integer)
+
+          expect(role_field[:default_value]).to eq(1)
+          expect(role_field[:default_value].class).to eq(Integer)
+        end
+      end
+
+      context 'with numeric fields' do
+        before(:each) do
+          @original_apimap = ForestLiana.apimap.dup
+          @original_models = ForestLiana.models.dup
+
+          Object.const_set(:NumericFieldModel, Class.new(ActiveRecord::Base) do
+            self.table_name = 'numeric_field_models'
+          end)
+
+          ActiveRecord::Migration.suppress_messages do
+            if ActiveRecord::Base.connection.table_exists?('numeric_field_models')
+              ActiveRecord::Base.connection.drop_table('numeric_field_models')
+            end
+            ActiveRecord::Base.connection.create_table('numeric_field_models') do |t|
+              t.integer :count, default: 0
+              t.float :rate, default: 0.5
+              t.decimal :price, default: 9.99
+            end
+          end
+        end
+
+        after(:each) do
+          ForestLiana.apimap = @original_apimap
+          ForestLiana.models = @original_models
+
+          ActiveRecord::Migration.suppress_messages do
+            ActiveRecord::Base.connection.drop_table('numeric_field_models') if ActiveRecord::Base.connection.table_exists?('numeric_field_models')
+          end
+          Object.send(:remove_const, :NumericFieldModel) if Object.const_defined?(:NumericFieldModel)
+        end
+
+        it 'should convert numeric default values to proper types' do
+          ForestLiana.models = [NumericFieldModel]
+          ForestLiana.apimap = []
+
+          adapter = SchemaAdapter.new(NumericFieldModel)
+          collection = adapter.perform
+
+          count_field = collection.fields.find { |f| f[:field] == 'count' }
+          rate_field = collection.fields.find { |f| f[:field] == 'rate' }
+          price_field = collection.fields.find { |f| f[:field] == 'price' }
+
+          expect(count_field[:default_value]).to eq(0)
+          expect(count_field[:default_value].class).to eq(Integer)
+
+          expect(rate_field[:default_value]).to be_a(Float)
+          expect(price_field[:default_value]).to be_a(Float)
+        end
+      end
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: forest_liana
 version: !ruby/object:Gem::Version
-  version: 9.17.0
+  version: 9.17.2
 platform: ruby
 authors:
 - Sandro Munda
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-01-06 00:00:00.000000000 Z
+date: 2026-01-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -311,6 +311,7 @@ files:
 - app/services/forest_liana/operator_date_interval_parser.rb
 - app/services/forest_liana/pie_stat_getter.rb
 - app/services/forest_liana/query_stat_getter.rb
+- app/services/forest_liana/record_findable.rb
 - app/services/forest_liana/resource_creator.rb
 - app/services/forest_liana/resource_getter.rb
 - app/services/forest_liana/resource_updater.rb
@@ -334,7 +335,6 @@ files:
 - app/services/forest_liana/stripe_subscriptions_getter.rb
 - app/services/forest_liana/token.rb
 - app/services/forest_liana/utils/beta_schema_utils.rb
-- app/services/forest_liana/utils/composite_primary_key_helper.rb
 - app/services/forest_liana/utils/context_variables.rb
 - app/services/forest_liana/utils/context_variables_injector.rb
 - app/services/forest_liana/value_stat_getter.rb
@@ -452,6 +452,7 @@ files:
 - spec/services/forest_liana/ability/permission/smart_action_checker_spec.rb
 - spec/services/forest_liana/ability/permission_spec.rb
 - spec/services/forest_liana/apimap_sorter_spec.rb
+- spec/services/forest_liana/composite_primary_keys_spec.rb
 - spec/services/forest_liana/filters_parser_spec.rb
 - spec/services/forest_liana/forest_api_requester_spec.rb
 - spec/services/forest_liana/has_many_getter_spec.rb
@@ -461,6 +462,7 @@ files:
 - spec/services/forest_liana/resource_updater_spec.rb
 - spec/services/forest_liana/resources_getter_composite_keys_spec.rb
 - spec/services/forest_liana/resources_getter_spec.rb
+- spec/services/forest_liana/schema_adapter_default_values_spec.rb
 - spec/services/forest_liana/schema_adapter_spec.rb
 - spec/services/forest_liana/scope_manager_spec.rb
 - spec/services/forest_liana/serializer_factory_spec.rb
@@ -760,6 +762,7 @@ test_files:
 - spec/services/forest_liana/ability/permission/smart_action_checker_spec.rb
 - spec/services/forest_liana/ability/permission_spec.rb
 - spec/services/forest_liana/apimap_sorter_spec.rb
+- spec/services/forest_liana/composite_primary_keys_spec.rb
 - spec/services/forest_liana/filters_parser_spec.rb
 - spec/services/forest_liana/forest_api_requester_spec.rb
 - spec/services/forest_liana/has_many_getter_spec.rb
@@ -769,6 +772,7 @@ test_files:
 - spec/services/forest_liana/resource_updater_spec.rb
 - spec/services/forest_liana/resources_getter_composite_keys_spec.rb
 - spec/services/forest_liana/resources_getter_spec.rb
+- spec/services/forest_liana/schema_adapter_default_values_spec.rb
 - spec/services/forest_liana/schema_adapter_spec.rb
 - spec/services/forest_liana/scope_manager_spec.rb
 - spec/services/forest_liana/serializer_factory_spec.rb

data/app/services/forest_liana/utils/composite_primary_key_helper.rb

@@ -1,27 +0,0 @@
-module ForestLiana
-  module Utils
-    module CompositePrimaryKeyHelper
-      def self.find_record(scoped_records, resource, id)
-        primary_key = resource.primary_key
-
-        if primary_key.is_a?(Array)
-          id_values = parse_composite_id(id)
-          conditions = primary_key.zip(id_values).to_h
-          scoped_records.find_by(conditions)
-        else
-          scoped_records.find(id)
-        end
-      end
-
-      def self.parse_composite_id(id)
-        return id if id.is_a?(Array)
-
-        if id.to_s.start_with?('[') && id.to_s.end_with?(']')
-          JSON.parse(id.to_s)
-        else
-          raise ForestLiana::Errors::HTTP422Error.new("Composite primary key ID must be in format [value1,value2], received: #{id}")
-        end
-      end
-    end
-  end
-end