fix: bug with sub-items

lib/paramoid/base.rb

@@ -8,6 +8,10 @@ def sanitize(params)
       ensure_required_params!(params)
     end
 
+    def permitted_params
+      scalar_params.to_params
+    end
+
     protected
 
     # @param [Symbol] name
@@ -82,10 +86,6 @@ def transformers
       @transformers ||= {}.with_indifferent_access
     end
 
-    def permitted_params
-      scalar_params.to_params
-    end
-
     def transformed_keys
       @transformed_keys ||= scalar_params
     end

lib/paramoid/object.rb

@@ -66,12 +66,18 @@ def ensure_required_params!(params, path: [])
 
       _raise_on_missing_parameter!(params, output_key, current_path) if @required
 
-      if nested?
-        if params.is_a?(Array)
-          params.each { |param| @nested.ensure_required_params!(param[output_key], path: current_path) }
-        else
-          @nested.ensure_required_params!(params ? params[output_key] : nil, path: current_path)
+      return params unless nested?
+
+      ensure_required_nested_params!(params, current_path)
+    end
+
+    def ensure_required_nested_params!(params, current_path)
+      if params.is_a?(Array)
+        params.flatten.each do |param|
+          @nested.ensure_required_params!(param[output_key], path: current_path)
         end
+      else
+        @nested.ensure_required_params!(params ? params[output_key] : nil, path: current_path)
       end
 
       params
@@ -103,17 +109,23 @@ def apply_nested_defaults!(params)
       return params unless params
 
       params ||= @nesting_type == :list ? [] : {}
-      if params.is_a?(Array)
-        params.map! { |param| @nested.apply_defaults!(param[output_key]) }
-      else
-        return params unless params[output_key]
 
-        result = @nested.apply_defaults!(params[output_key])
-        params[output_key] = result if result
+      return apply_nested_param_default_when_available!(params, output_key) unless params.is_a?(Array)
+
+      params.map! do |param|
+        apply_nested_param_default_when_available!(param, output_key)
       end
       params
     end
 
+    def apply_nested_param_default_when_available!(param, key)
+      return param unless param[key]
+
+      result = @nested.apply_defaults!(param[key])
+      param[key] = result if result
+      param
+    end
+
     def to_defaults
       if nested?
         nested_defaults = @nested.to_defaults

spec/paramoid/complex_spec.rb

@@ -53,7 +53,7 @@
             }
           },
           person: {},
-          items: [{}, {}]
+          items: [{}, { sub_items: [{ id: 5 }] }]
         }
       end
       it 'has the default value for those' do
@@ -62,7 +62,10 @@
             'buyer' => { 'payment_method' => { 'uuid' => 1 } },
             'total' => 0,
             'name' => 'some_name',
-            'items' => [{ 'price' => 0 }, { 'price' => 0 }],
+            'items' => [
+              { 'price' => 0 },
+              { 'price' => 0, 'sub_items' => [{ 'price' => 0, 'id' => 5 }] }
+            ],
             'person_attributes' => { 'role' => :user }
           }
         )
@@ -91,11 +94,24 @@
       ActionController::Parameters.new(params_hash)
     end
 
+    it 'returns permitted params' do
+      expect(subject.permitted_params).to eq(
+        [
+          :total, :name,
+          { buyer: [{ payment_method: [:id] }],
+            items: [:id, :name, :price, :discount, { sub_items: [:id, :price] }],
+            person: %i[id full_name role] }
+        ]
+      )
+    end
+
     it 'has the default values correctly nested' do
       expect(sanitized.to_unsafe_h).to eq(
         {
           'buyer' => { 'payment_method' => { 'uuid' => 1 } },
-          'items' => [{ 'price' => 0, 'name' => 'some_name', 'discount' => 0.1, 'id' => 5 }],
+          'items' => [
+            { 'price' => 0, 'name' => 'some_name', 'discount' => 0.1, 'id' => 5 }
+          ],
           'total' => 100,
           'name' => 'some_name',
           'person_attributes' => { 'role' => :admin, 'id' => 1, 'full_name' => 'some_name' }

spec/support/params.rb

@@ -45,6 +45,11 @@ def initialize(user = nil)
 
       default :price, 0
       param :discount, transformer: ->(data) { data&.to_f / 100 } if user.admin?
+
+      array :sub_items do
+        param :id
+        default :price, 0
+      end
     end
 
     default :total, 0