fixed prototype block prefixes hierarchy of the CollectionType

Author: yceruto

Extension/Core/Type/CollectionType.php

@@ -93,7 +93,7 @@ public function finishView(FormView $view, FormInterface $form, array $options)
                 $view->vars['multipart'] = true;
             }
 
-            if ($prefixOffset > -2 && $prototype->getConfig()->getOption('block_prefix')) {
+            if ($prefixOffset > -3 && $prototype->getConfig()->getOption('block_prefix')) {
                 --$prefixOffset;
             }
 

Tests/Extension/Core/Type/CollectionTypeTest.php

@@ -428,37 +428,66 @@ public function testEntriesBlockPrefixes()
     public function testEntriesBlockPrefixesWithCustomBlockPrefix()
     {
         $collectionView = $this->factory->createNamed('fields', static::TESTED_TYPE, [''], [
+            'allow_add' => true,
             'entry_options' => ['block_prefix' => 'field'],
         ])
             ->createView()
         ;
 
-        $this->assertCount(1, $collectionView);
-        $this->assertSame([
+        $expectedBlockPrefixes = [
             'form',
             'collection_entry',
             'text',
             'field',
             '_fields_entry',
-        ], $collectionView[0]->vars['block_prefixes']);
+        ];
+
+        $this->assertCount(1, $collectionView);
+        $this->assertSame($expectedBlockPrefixes, $collectionView[0]->vars['block_prefixes']);
+        $this->assertSame($expectedBlockPrefixes, $collectionView->vars['prototype']->vars['block_prefixes']);
     }
 
     public function testEntriesBlockPrefixesWithCustomBlockPrefixedType()
     {
         $collectionView = $this->factory->createNamed('fields', static::TESTED_TYPE, [''], [
+            'allow_add' => true,
             'entry_type' => BlockPrefixedFooTextType::class,
         ])
             ->createView()
         ;
 
-        $this->assertCount(1, $collectionView);
-        $this->assertSame([
+        $expectedBlockPrefixes = [
             'form',
             'collection_entry',
             'block_prefixed_foo_text',
             'foo',
             '_fields_entry',
-        ], $collectionView[0]->vars['block_prefixes']);
+        ];
+
+        $this->assertCount(1, $collectionView);
+        $this->assertSame($expectedBlockPrefixes, $collectionView[0]->vars['block_prefixes']);
+        $this->assertSame($expectedBlockPrefixes, $collectionView->vars['prototype']->vars['block_prefixes']);
+    }
+
+    public function testPrototypeBlockPrefixesWithCustomBlockPrefix()
+    {
+        $collectionView = $this->factory->createNamed('fields', static::TESTED_TYPE, [], [
+            'allow_add' => true,
+            'entry_options' => ['block_prefix' => 'field'],
+        ])
+            ->createView()
+        ;
+
+        $expectedBlockPrefixes = [
+            'form',
+            'collection_entry',
+            'text',
+            'field',
+            '_fields_entry',
+        ];
+
+        $this->assertCount(0, $collectionView);
+        $this->assertSame($expectedBlockPrefixes, $collectionView->vars['prototype']->vars['block_prefixes']);
     }
 
     public function testSubmitNull($expected = null, $norm = null, $view = null)