setOptions on TwigField will remove the template

[mamazu]

The problem

The TwigField is basically just a normal field with a default template. The problem is if you want to change the options of this field it will either have to specify the template again or you will lose it.

Example

<?php

$field = TwigField::create('some_field', '@Sylius/some_template.html.twig')
    ->setOptions(['vars' => ['labels' => 'path/to/label'])
;

Will result in the options being:

[
    'vars' => ['labels' => 'path/to/label']
]

and not:

[
    'template' => '@Sylius/some_template.html.twig',
    'vars' => ['labels' => 'path/to/label']
]

Possible solution

The quick and easy answer would be to have the TwigField extend the Field and change the behavior of the setOptions function to merge with the template. This would be a big change though.

[loic425]

@mamazu You're right. FYI, there is a "addOption". @lchrusciel merged too quickly the notice about this issue in the doc.
But I'm in favour of the merge options with the setOptions, that was my initial idea when implementing the grid builder. I think it'd be better for DX as we already have users who have that issue.

[mamazu]

So extending it would be the best idea? I can make a merge request for that if you want.