[Form] Combiner un formulaire imbriqué (collection) avec un événement de formulaire paramétré

**Sokaru** · 06/03/2012, 15h34

Bonjour,

je suis actuellement face à un problème et j'avoue que j'ai du mal à voir si une solution peut exister. J'ai bon espoir mais personnellement j'ai beau tourner le problème dans tous les sens je trouve pas.

Mon objectif est d'offrir plusieurs mode de saisie pour une même entité, elle même incluse dans une collection d'une autre entité. J'ai bien compris la notion des forms embed, j'ai bien compris la notion des event forms, mais je n'arrive pas à fusionner les deux. J'explique.

Prenons l'exemple de la tâche avec les tags de la doc officielle. Mon objectif serait ici de contrôler les champs tags du formulaire.

http://symfony.com/doc/current/cookb...llections.html

En objets on a donc :

Entity Task
Entity Tag
Form TaskType
Form TagType

Pour ce qui est de la notion de form embed, la Task contient une liste de Tags, mappé dans le formulaire par la collection :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$builder->add('tags', 'collection', array('type' => new TagType()));

Pour ce qui est des event forms, j'aurais quelque chose dans ce style :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
class TagFieldCreator implements EventSubscriberInterface
{
    private $factory;
    private $display;
 
    public function __construct(FormFactoryInterface $factory, $display_type)
    {
        $this->factory = $factory;
        $this->display = $display_type;
    }
 
    public static function getSubscribedEvents()
    {
        return array(FormEvents::PRE_SET_DATA => 'preSetData');
    }
 
    public function preSetData(DataEvent $event)
    {
        $data = $event->getData();
        $form = $event->getForm();
 
        switch ($this->display)
        {
            case 'text':
                $form->add($this->factory->createNamed('text', 'name'));
                break;
            case 'select':
                $form->add($this->factory->createdNamed('choice', 'name', null, array(
                    'choices'   => array(
                        'tag1' => 'Tag 1',
                        'tag2' => 'Tag 2',
                        'tag3' => 'Tag 3',
                    )
                )));
        }
    }
}

Avec ça, mon TagType devrait ressembler à ça :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
class TagType extends AbstractType
{
    private $display;
 
    public function __construct($display)
    {
        $this->display = $display;
    }
 
    public function buildForm(FormBuilder $builder, array $options)
    {
        $subscriber = new TagFieldCreator($builder->getFormFactory(), $display);
        $builder->addEventSubscriber($subscriber);
    }
 
    public function getName()
    {
        return 'tag';
    }
}

Et mon problème dans tout ça, c'est que si je créais un formulaire simple pour un Tag directement depuis le controleur, j'aurais moyen de savoir quel $display je veux envoyer à mon TagType. Ce qui donnerait quelque chose du style :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$form = $this->createForm(new TagType($display), $tag);

Or, mon TagType est appelé dans la création de la collection, dans la classe TaskType. Rappel :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$builder->add('tags', 'collection', array('type' => new TagType()));

Donc comment à partir d'ici je pourrait différencier mes TagType et en avoir dans ma collection par exemple un en mode 'text' et l'autre en mode 'select' comme décrit dans l'event form?

Merci d'avance à ceux qui auront le courage de réfléchir au problème, après avoir eu le courage de lire et d'essayer de comprendre mon exposé

**arnooo999** · 06/03/2012, 20h08

Bonjour,
J'ai tout lu mais j'ai du mal à comprendre la finalité de la chose.
s'agit t'il:

d'éviter de la duplication de code
d'avoir un select ou un input[text] selon la valeur de l'entité Tag
de pouvoir saisir la valeur d'un tag à la fois par select et/ou par input dans le même formulaire

D'aprés le code de TagType et de TagFieldCreator je n'ai pas l'impression que tu ais besoin d'utiliser les listeners puisque tu n'utilises pas la valeur de $data.
Ce code donnerait le même résultat:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
 
class TagType extends AbstractType
{
    private $display;
 
    public function __construct($display)
    {
        $this->display = $display;
    }
 
    public function buildForm(FormBuilder $builder, array $options)
    {
        switch($this->display){
           case 'text':
                $form->add( 'name','text'));
                break;
           case 'select':
                $form->add( 'name', 'choice', array(
                    'choices'   => array(
                        'tag1' => 'Tag 1',
                        'tag2' => 'Tag 2',
                        'tag3' => 'Tag 3',
                    )
                ));
        }
    } 
    public function getName()
    {
        return 'tag';
    }
}

Et mon problème dans tout ça, c'est que si je créais un formulaire simple pour un Tag directement depuis le controleur, j'aurais moyen de savoir quel $display je veux envoyer à mon TagType. Ce qui donnerait quelque chose du style :
Code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$form = $this->createForm(new TagType($display), $tag);

Qu'est ce qui t'empeche d'avoir un attribut display dans TaskType (transmis par le constructeur également) afin de le transmettre dans TagType?

**Sokaru** · 06/03/2012, 20h44

Tout d'abord, merci de tenter de m'aider

La finalité que je veux atteindre, c'est d'avoir, pour une même classe (Entity liée à la bdd), différentes méthodes de saisie.

L’intérêt serait surtout d'avoir une base propre car au final tout va dans la même table car tous les Tags sont des Tags, seul la méthode de saisie pour l'utilisateur change.

De plus je veux pouvoir, à terme, pouvoir ajouter des modes de saisie, et ça m’embêterait d'ajouter des tables à chaque fois juste pour ça.

Enfin pour répondre à ta dernière question, ce qui m'empêche d'ajouter le paramètre display au TagType, c'est qu'il est créé via une fonction dans le builder de la classe TaskType :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$builder->add('tags', 'collection', array('type' => new TagType()));

Si je trasmets un paramètre ici, puisque tous les TagType sont créés via ce constructeur, il n'y a plus de moyen de différencier les différents TagType dans mon formulaire. Or mon objectif est, je le rappelle, d'afficher plusieurs TagType différemment dans un même formulaire.

J'espère que mon problème est plus clair...

**arnooo999** · 07/03/2012, 11h10

Qu'est ce qui détermine qu'un Tag doit s'afficher dans un formulaire de type Text ou de type Select?
A priori le choix d'utiliser tel ou tel type de saisie dépend d'un ou de plusieurs attribut de l'entité Tag.
Il faut donc analyser le tag qui est passé au formulaire dans le listener pour savoir quel type de saisie tu veux adopter

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
class TagFieldCreator implements EventSubscriberInterface
{
/....
    public function preSetData(DataEvent $event)
    {
        $data = $event->getData();/*@var $data Tag*/
        if(!$data){
           return;
        }
        $form = $event->getForm();
 
        switch ($data->getType())
        {
            case 'text':
                $form->add($this->factory->createNamed('text', 'name'));
                break;
            case 'select':
                $form->add($this->factory->createdNamed('choice', 'name', null, array(
                    'choices'   => array(
                        'tag1' => 'Tag 1',
                        'tag2' => 'Tag 2',
                        'tag3' => 'Tag 3',
                    )
                )));
        }
    }
}

**Sokaru** · 07/03/2012, 11h22

A priori le choix d'utiliser tel ou tel type de saisie dépend d'un ou de plusieurs attribut de l'entité Tag.

Oui c'est exactement ça.

Du coup je comprends le principe de ton exemple, je vais essayer ça ce soir.

Il me semble que j'avais essayé de voir le contenu du resultat de $event->getData(), ça me retournait toujours NULL, mais c'est peut-être moi qui avait mal fait un truc alors.

Merci pour ta réponse.

**Sokaru** · 07/03/2012, 18h39

Malheureusement, c'est bien ce qu'il me semblait. Avec ce code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
$task = new Task();
 
$configs = $configuration->getConfs();
foreach ($configs as $config) {
    $tag = new Tag();
    $tag->setConf($config);
    $task->addTag($tag);
}
 
$form = $this->createForm(new TaskType(), $task);

J'ai tout de même le

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$data = $event->getData();

qui est à NULL dans l'EventSubscriber.

Est-ce moi qui ai mal fait ou oublié quelque chose?

**Sokaru** · 07/03/2012, 18h50

Finalement j'ai trouvé la solution, c'était dû au fait que j'utilisais l'event pre_set_data. Avec l'event set_data, j'arrive bien à récupérer ma valeur.

Par contre si quelqu'un pouvait m'expliquer les traitements qui sont effectués entre le pre_set_data, le set_data et le post_set_data, histoire que je sois sûr que ça va faire ce que je veux, ça serait sympa.

Sinon merci de ton aide arnooo