Discussion :

[Pictavia]

Bonjour,

Pour que mon entité User soit valide, j'ai besoin que soit validé un certain nombre de critères (stocké dans l'entité Critere).

Le formulaire que j'ai creer directement dans ma vue affiche la liste de critères avec checkbox oui/non :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
{% for critere in criteres %}
                <tr><td>            
                    {{ critere.nom }}
                </td><td>               
                    <label class="radio">
                        <input type="radio" name="{{critere.id}}" id="optionsRadios1" value="non" checked>
                        non
                    </label>
                    <label class="radio">
                        <input type="radio" name="{{critere.id}}" id="optionsRadios2" value="oui">
                        oui
                    </label>
                </td></tr>
{% endfor %}

Pour que l'User puisse être créer, il faut que toutes les checkbox soient à 'oui'. Pour cela, j'ai créé dans ma classe User, un attribut $allCritere qui doit être à 'true' pour être valide.

Je veux donc transmettre à cet attribut la valeur 'true' si l'ensemble des checkbox ont la valeur 'oui'.

Dans le controleur ca me donne ça :

Code :

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

1
2
3
4
5
6
7
8
 
$validateur = 'true';
foreach ($criteres as $critere) {
               if ($request->request->get($critere->getId()) == "non"){
                   $validateur = false;
               }
}
$user->setAllCritere($validateur); // si le validateur = false, mon entité User ne sera pas valide

Je souhaiterais que cette vérification ne se fasse pas dans le controleur. Mais en amont : y'a t'il une méthode idéale pour transmettre les valeurs des checkbox à mon attribut $allCritere, sans passer par le controleur ? Javascript ? Formbuilder (creer un champ comprenant l'ensemble des sous-champs checkbox : j'ai essayé, mais je crois que ça me dépasse...) ?

J'espère que mes explications ne sont pas trop confuses.
Merci d'avance pour vos conseils.

[Invité]

Pourquoi ne pas tenter cela:

• mettre des champs hidden à raison d'un par checkbox
• Remplir chaque champs à la sélection
• et grâce à une validation inline ne valider le formulaire que si ces champs ont la bonne valeur

[Nico_F]

Que l'on soit bien clair, quand tu dis que tu ne veux pas que la validation se fasse dans le contrôleur, tu veux dire "pas dans le fichier du controller". Parce que sinon ça revient à dire : "pas coté serveur" et là il y a un gros problème de sécurité.
La validation coté client sans validation serveur derrière viole à elle seule toutes les bonnes pratiques existantes en matière de sécurité, séparation des couches etc. N'ayons pas peur des mots : c'est de la merde en boite.

Partons du principe que tu ne veux juste pas encombrer ton contrôleur d'une vérification lourde métier.

Utilise les méthodes de Callback qui sont faites pour définir tes propres méthodes de validation. Les callbacks sont appelés en même temps que les autres méthodes de validation (lorsque tu fais appel à la méthode isValid()).

Tu peux alors retourner toi même des erreurs dans ton formulaire en fonction des attributs de ton objet sans être obligé de rajouter quoique ce soit. Qui plus est, tu n'auras plus besoin de ton attribut $allCritere.

Jète un oeil à ça je pense que toutes les réponses que tu attends se trouvent ici : http://symfony.com/doc/2.0/reference.../Callback.html (attention à la version certaines choses changent si tu te trouves en 2.0 ou 2.1)

++

[Pictavia]

@Nico_F
Oui, je parlais de mon fichier controller, plus précisément dans ma méthode RegisterAction.
Ce n'est pas uniquement pour ne pas l'encombrer mais pour pouvoir passer cette validation comme première étape d'une validation multistep à l'aide du CraueFormFlowBundle :
la validation de cette première étape ressemble à celle du "cochez la case si vous avez bien lu les conditions d'utilisation", sauf que dans mon cas, il y a x cases à cocher.

Tu peux alors retourner toi même des erreurs dans ton formulaire en fonction des attributs de ton objet sans être obligé de rajouter quoique ce soit. Qui plus est, tu n'auras plus besoin de ton attribut $allCritere.

Justement, je n'ai pas d'attribut en relation avec les checkbox et je voudrais m'en passer d'autant que mon nombre de checkbox peut varier.
Concraitement, l'idéal serait que je construise un formulaire avec un "champ spécial" pouvant contenir toutes mes checkbox quel que soit leur nombre. La valeur renvoyé par ce champ serait un tableau $allCritere, attribut de ma classe User, sur lequel je puisse faire un Callback.
Mais il y a probablement plus simple...

[Nico_F]

Code :

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

Justement, je n'ai pas d'attribut en relation avec les checkbox et je voudrais m'en passer d'autant que mon nombre de checkbox peut varier.

Tu veux dire que ces checkboxes ne sont reliées à aucun champs de la base : ce sont des attributs de ta classe mais non mappés ?
Ou ce ne sont même pas des attributs de ta classe ?

Dans le premier cas ce n'est pas un souci dans la mesure ou tu peux gérer des champs non mappés : tu récupères leur valeur sans problème et tu en fais ce que tu veux comme s'il s'agissait de n'importe quel autre champ.

Dans le second tu essayes de faire quelque chose d'assez bizarre : une sorte de formulaire mappé sur une classe, en y ajoutant des éléments qui ne correspondent pas à des attributs de cette classe. Je ne vois pas bien comment tu peux gérer ça. Il est possible de faire des formulaires avec une classe, des formulaires sans classe, mais un mélange des deux me paraît plus compliqué car dans un cas ou a une classe et dans l'autre un tableau.

Tu devrais soit avoir un champs pour chaque cas de checkbox, que tu ne mapperais pas mais que tu pourrais contrôler aisément (si tu n'en as que 5 ou 6 différents ce n'est pas excessivement gênant : ça l'est d'avantage si tu en as 40), ou bien alors embarquer une collection d'un formulaire (ce serait ça ton "champ spécial") qui n'aurait qu'une checkbox et qui se baserait sur une classe custom: j'aime moins cette solution mais c'est toujours mieux qu'une validation en javascript. Et puis ça t'évite d'avoir à surcharger ta classe avec des attributs que tu n'utiliseras pas.

Pour la validation, je reste sur les méthodes de callback.

++

[Pictavia]

ce ne sont même pas des attributs de ta classe ?

Effectivement, je me retrouve dans ce cas bizarre.
Ce formulaire sert simplement à s'assurer que le recruteur s'est bien assuré que tous les critères d'inclusion dans l'étude sont respectés. Aujourd'hui, je ne connais pas tous ces critères. Comme leur nombre nombre va varier avec l'avancée de l'étude, je ne veux pas en faire des attributs de ma classe.
Je m'apperçois en effet que la difficulté est trop grande pour mon niveau. Cette solution que tu proposes :

embarquer une collection d'un formulaire (ce serait ça ton "champ spécial") qui n'aurait qu'une checkbox et qui se baserait sur une classe custom

est une bonne idée malgré le fait qu'elle te plaise moins. Je récupère cette collection dans mon attribut $allCritere, et je fais un callBack dessus qui validerait l'objet si l'ensemble des cases est coché.
Je trouve que c'est un bon compromis.
Un grand merci pour ton aide, Nico_F.
++