#27 fixed missing sub-key in template data for nested arrays

qossmic · Apr 23, 2024 · d6e9fdf · d6e9fdf
1 parent 251bb0a
commit d6e9fdf
Show file tree

Hide file tree

Showing 2 changed files with 37 additions and 1 deletion.
diff --git a/src/Component/ComponentItemFactory.php b/src/Component/ComponentItemFactory.php
@@ -127,7 +127,7 @@ private function createVariationParameters(array $parameters, array $variation):
 
         foreach ($parameters as $name => $type) {
             if (\is_array($type)) {
-                $paramValue = $this->createVariationParameters($type, $variation[$name] ?? []);
+                $paramValue[$name] = $this->createVariationParameters($type, $variation[$name] ?? []);
             } else {
                 $paramValue = $this->faker->getFakeData([$name => $type], $variation[$name] ?? null);
             }

diff --git a/tests/Functional/Service/ComponentItemFactoryTest.php b/tests/Functional/Service/ComponentItemFactoryTest.php
@@ -242,6 +242,42 @@ public function testCreateForParamWithOptionalVariationValue(): void
         self::assertNull($variations['variation1']['optionalEmpty']);
     }
 
+    public function testCreateForArrayParameter(): void
+    {
+        $data = [
+            'name' => 'TestComponent',
+            'title' => 'Test title',
+            'description' => 'description',
+            'category' => 'MainCategory',
+            'path' => 'path/to/component',
+            'renderPath' => 'path/to/component',
+            'parameters' => [
+                'arrayParam' => [
+                    'param1' => 'String',
+                    'param2' => 'Boolean',
+                ],
+            ],
+            'variations' => [
+                'variation1' => [
+                    'arrayParam' => [
+                        'param1' => 'Some cool hipster text',
+                    ],
+                ],
+            ],
+        ];
+
+        /** @var ComponentItemFactory $factory */
+        $factory = self::getContainer()->get('twig_doc.service.component_factory');
+
+        $component = $factory->create($data);
+        $variations = $component->getVariations();
+
+        self::assertIsArray($variations);
+        self::assertArrayHasKey('variation1', $variations);
+        self::assertEquals('Some cool hipster text', $variations['variation1']['arrayParam']['param1']);
+        self::assertIsBool($variations['variation1']['arrayParam']['param2']);
+    }
+
     public static function getInvalidComponentConfigurationTestCases(): iterable
     {
         yield [