Почему форма не проходит валидацию при использовании типа «collection» с текстовыми значениями?

Question

[Александр Евгеньевич]

public function buildForm(FormBuilderInterface $builder, array $options)
    {
        $builder
            ->add('description', 'text')
            ->add('coordinates', 'collection', array(
                'type' => 'text'
            ));

    }

    public function setDefaultOptions(OptionsResolverInterface $resolver){

        $resolver->setDefaults(array(
            'allow_extra_fields' => true,
            'csrf_protection' => false,
            'data_class' => '\AppBundle\Entity\Marker'
        ));

    }

Передаю :

$dataMarker = array(
                'description' => "asdasdada",
                'coordinates' => array(123123, 234234)
            );

            $form = $this->get('form.factory')->create(new MarkerType(), new Marker());
            $form->submit($dataMarker);

Выдает "This form should not contain extra fields".

А если вместо colleciton использовать тип text, и передавать в поле coordinates тоже текст, то нормально проходит без ошибок.

Answer 1

[Александр Евгеньевич]

Оказалось, что нужно указывать "allow_extra_fields" непосредственно в массиве инициализации поля с типом collection:

->add('coordinates', 'collection', array(
                'type' => 'text',
                'allow_add' => true,
                'allow_extra_fields' => true
            ));

Только схерали там вообще эта ошибка.

Comments

[voronkovich]

Вы можете убрать "allow_extra_fields", эта опция автоматически выставляется в true, если вы укажете "allow_add" => true. Ошибка и была в том, что allow_add не был указан.

[Александр Евгеньевич]

voronkovich: мне уже было лень копаться в исходниках, просто я так и не понял, тогда при чем allow_add к ошибке "This form should not contain extra fields" ?

[voronkovich]

Александр Евгеньевич: Я посмотрел исходники CollectionType, оказалось что allow_add и allow_extra_fields никак не связаны. Извините, что ввел в заблуждение.
Тогда, это на самом деле странная ситуация, т.к. в документации не сказано про то что нужно использовать allow_extra_fields (хотя, это и логично, если используется валидация) для коллекций.

[Александр Евгеньевич]

voronkovich: ну вы ж видели доки, там все, что касается таких параметров, как allow_extra_fields, всегда указывалось в резолвере, или в опциях при создании формы, через фабрику. А тут, вон, оказывается, нужно было положить в параметры типа, хотя в доках не указано. Ну короче странные вещи.

[voronkovich]

Александр Евгеньевич: может дело в cascade_validation? По умолчанию оно стоит false, т.е. не проверяет наличие доп, полей в дочерних формах. Поэтому в доках и не указывают allow_extra_fields