[Config] Fix YamlReferenceDumper for nested prototypes

Author: ogizanagi

Definition/Dumper/XmlReferenceDumper.php

@@ -96,7 +96,10 @@ private function writeNode(NodeInterface $node, $depth = 0, $root = false, $name
                     $rootAttributes[$key] = str_replace('-', ' ', $rootName).' '.$key;
                 }
 
-                if ($prototype instanceof ArrayNode) {
+                if ($prototype instanceof PrototypedArrayNode) {
+                    $prototype->setName($key);
+                    $children = array($key => $prototype);
+                } elseif ($prototype instanceof ArrayNode) {
                     $children = $prototype->getChildren();
                 } else {
                     if ($prototype->hasDefaultValue()) {

Definition/Dumper/YamlReferenceDumper.php

@@ -199,8 +199,14 @@ private function getPrototypeChildren(PrototypedArrayNode $node)
 
         if ($prototype instanceof ArrayNode) {
             $keyNode = new ArrayNode($key, $node);
+            $children = $prototype->getChildren();
+
+            if ($prototype instanceof PrototypedArrayNode) {
+                $children = $this->getPrototypeChildren($prototype);
+            }
+
             // add children
-            foreach ($prototype->getChildren() as $childNode) {
+            foreach ($children as $childNode) {
                 $keyNode->addChild($childNode);
             }
         } else {

Tests/Definition/Dumper/XmlReferenceDumperTest.php

@@ -78,6 +78,20 @@ enum=""
         pass=""
     />
 
+    <!-- prototype -->
+    <cms-page page="cms page page">
+
+        <!-- prototype -->
+        <!-- title: Required -->
+        <!-- path: Required -->
+        <page
+            locale="page locale"
+            title=""
+            path=""
+        />
+
+    </cms-page>
+
 </config>
 
 EOL

Tests/Definition/Dumper/YamlReferenceDumperTest.php

@@ -65,6 +65,15 @@ enum:                 ~ # One of "this"; "that"
         -
             user:                 ~
             pass:                 ~
+    cms_pages:
+
+        # Prototype
+        page:
+
+            # Prototype
+            locale:
+                title:                ~ # Required
+                path:                 ~ # Required
 
 EOL;
     }

Tests/Fixtures/Configuration/ExampleConfiguration.php

@@ -24,6 +24,7 @@ public function getConfigTreeBuilder()
         $rootNode
             ->fixXmlConfig('parameter')
             ->fixXmlConfig('connection')
+            ->fixXmlConfig('cms_page')
             ->children()
                 ->booleanNode('boolean')->defaultTrue()->end()
                 ->scalarNode('scalar_empty')->end()
@@ -67,6 +68,18 @@ public function getConfigTreeBuilder()
                         ->end()
                     ->end()
                 ->end()
+                ->arrayNode('cms_pages')
+                    ->useAttributeAsKey('page')
+                    ->prototype('array')
+                        ->useAttributeAsKey('locale')
+                        ->prototype('array')
+                            ->children()
+                                ->scalarNode('title')->isRequired()->end()
+                                ->scalarNode('path')->isRequired()->end()
+                            ->end()
+                        ->end()
+                    ->end()
+                ->end()
             ->end()
         ;