cleanup

Author: michael-rubel

src/Traits/HelpsLoopFunctions.php

@@ -40,19 +40,6 @@ private function ignoreKeys(): array
             : $defaults;
     }
 
-    /**
-     * Check if the function can walk recursively over an array.
-     *
-     * @param mixed      $value
-     * @param int|string $key
-     *
-     * @return bool
-     */
-    private function canWalkRecursively(mixed $value, int|string $key): bool
-    {
-        return is_array($value) && ! empty($this->{$key});
-    }
-
     /**
      * @param int|string $key
      *

src/Traits/LoopFunctions.php

@@ -61,12 +61,6 @@ public function arrayToProperties(?array $data, mixed $rescue = null): void
     {
         collect($data ?? [])
             ->except($this->ignoreKeys())
-            ->each(function ($value, $key) use ($rescue) {
-                if ($this->canWalkRecursively($value, $key)) {
-                    $this->arrayToProperties($value, $rescue);
-                }
-
-                $this->assignValue($key, $value, $rescue);
-            });
+            ->each(fn ($value, $key) => $this->assignValue($key, $value, $rescue));
     }
 }

tests/ArrayMappingTest.php

@@ -35,7 +35,7 @@ public function testCanMapAnArrayToProperties()
     }
 
     /** @test */
-    public function testCanMapAnArrayRecursively()
+    public function testCanMapAnArray()
     {
         $array = [
             'test'     => true,
@@ -64,6 +64,7 @@ public function testAlreadyInitializedPropertiesArentOverridenByNestedArrays()
             ],
         ];
 
+
         $this->arrayToProperties($array);
 
         $this->assertStringContainsString('Michael', $this->name);
@@ -108,7 +109,9 @@ public function testCanMapUsingPropertiesFrom()
         $array = [
             'test'            => true,
             'additional_data' => [
-                'next' => 'test',
+                'next' => [
+                    'test' => true,
+                ],
             ],
         ];