Validation.yml : Problème avec le type "integer"

Bonjour chers camarades Symfoniens, je viens quérir (une fois de plus) votre aide.

J'ai créé tout un tas de formulaires pour une interface d'administration (classique : ajout, modification, suppression de contenu) et maintenant j'en suis à m'assurer que les utilisateurs ne rentrent pas n'importe quoi en base de données.
Pour cela j'ai donc suivi la documentation officielle qui parle de validation de formulaire.

J'ai pu mettre en place diverses contraintes comme par exemple, ne pas laisser tel ou tel champ vide, ou encore vérifier si une date de fin est bien supérieure à une date de début... Mais là je me retrouve embêté avec une contrainte pourtant basique.
Je souhaite vérifier que mon champ contient un int et rien d'autre. Encore une fois j'ai donc suivi la documentation officielle, mais ça ne fonctionne pas.

Dans mon fichier "validation.yml" j'ai ceci :

Code:

---

1 2 3 4 5 6

AppBundle\Entity\AbstractDurationEntity: properties: minDuration: - Type: type: integer message: La valeur {{ value }} n'est pas un type {{ type }} valide

---

Et lorsque je valide mon formulaire, quelle que soit la valeur entrée, il ne valide pas et affiche le message d'erreur.

Quand je met "toto" dans la case minDuration et qu'il m'affiche :

Citation:

---

La valeur "toto" n'est pas un type integer valide

---

Là je suis entièrement d'accord et plutôt satisfait.

Mais quand je rentre "4" dans cette même case et qu'il m'affiche :

Citation:

---

La valeur "4" n'est pas un type integer valide

---

Là j'ai plutôt tendance à penser qu'il se fout de moi 8O


Quelqu'un a-t-il déjà connu ce problème, et encore mieux, quelqu'un a-t-il déjà eu ce problème et trouvé la solution ?

Merci de votre attention.

on dirait qu'il n'a pas réussi à convertir la valeur en entier, comment génère tu ton formulaire ? Avec un formtype ?

Oui,

Voici le formulaire :

Code:

---

1 2 3 4 5 6 7 8 9

public function buildForm(FormBuilderInterface $builder, array $options) { ... $builder->add('MinDuration','text', array( 'label' => 'Durée minimum', 'read_only' => $readOnly ));   ...

---

Mon controler :

Code:

---

1 2 3 4 5 6

$form = $this->createForm(new AlertType('add', $mode), $alert); $form->handleRequest($request);   if ($form->isValid()) { ...

---

Dis moi si tu as besoin de voir plus de code (J'essaye de ne pas surcharger la page inutilement :)).

d'accord, oui peux tu montrer la suite après le

Code:

---

if ($form->isValid()) {

---

pour voir le traitement du formulaire, en particulier quand tu récupères le minDuration et que tu crées l'entité. Car c'est la que ça pose problème.

Vu que c'est un champs de type integer je te conseillerais de mettre le type en integer ici même si ce n'est pas sur que cela règle ton problème :

Code:

---

1 2 3 4

$builder->add('MinDuration','integer', array( 'label' => 'Durée minimum', 'read_only' => $readOnly ));

---

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21

if ($form->isValid()) { if ($mode == 'simple') $alert->setTemplate('Alerts/metaAlert.html.twig'); else if ($mode == 'advanced') { $template = 'Alerts/'.time().$form->get('Template')->getData(); if (!strstr($form->get('Template')->getData(), '.html.twig')) $template = $template.'.html.twig';   $file = fopen('../src/AppBundle/Resources/views/'.$template, 'w'); fwrite($file, $form->get('Content')->getData()); fclose($file); $alert->setTemplate($template); $alert->setContent(null); } $em->persist($alert); $em->flush(); return $this->redirect($this->generateUrl('add_alert', array( 'succes' => true, 'mode' => $mode, ))); }

---

J'ai testé en mettant le type "integer" dans mon champ, ça ne valide pas le formulaire si ce n'est pas un int dans la case, mais par contre mon message d'erreur ne s'affiche plus du tout, du coup ça ne m'arrange pas trop, car l'erreur bien que bloquante pour l'utilisateur n'est pas visible directement.
En plus, même si je laisse en "text", je devrais quand même pouvoir faire un test sur ma valeur non ?

En mettant le type à 'text', même si tu tapes 4, il sera traité comme '4', donc comme une chaine de caractères.
Résultat : comme ton validateur vérifie qu'il a un entier et rien d'autre il te balance une violation de contrainte.

Vérifie dans la debug toolbar (section Form) que l'erreur est bien celle là, il est possible qu'il y ait une autre erreur et qu'elle ne soit pas rendue dans ta template.

Oui effectivement, présenté comme ça c'est d'ailleurs tout à fait logique.
Ce qui l'est moins en revanche c'est que, comme je le disais dans mon précédent message, une fois le type passé en integer, si je lui rentre une chaine de caractère je n'ai plus mon message d'erreur.
Il ne valide pas le formulaire pour autant, il me l'affiche à nouveau comme il le devrait et prend même soin de supprimer le contenu de mon champ, mais pas de message d'erreur :(

D'ou ma dernière phrase

Citation:

---

Vérifie dans la debug toolbar (section Form) que l'erreur est bien celle là, il est possible qu'il y ait une autre erreur et qu'elle ne soit pas rendue dans ta template.

---

Oui pardon, il est vrai que je n'ai pas détaillé, tu ne pouvais pas deviner :)

J'ai regardé et je n'ai pas d'autre erreur.
La seule erreur que j'ai correspond en fait à la case vide (puisqu'il supprime le contenu qui ne convient pas au type integer).

Pour plus de clarté :
- Si je rentre un int : Le formulaire est validé (ajout en BDD)
- Si je rentre un string : Le formulaire n'est pas validé, il se ré-affiche et j'obtiens une erreur dans ma barre qui est que le champ est vide

J'ai déjà eu ce problème

Un champ de type number, integer,etc... type la valeur rentrée.
Lorsque l'entrée est invalide (par exemple l"utilisateur se trompe et écrit "1378a") le formulaire est bien invalide mais comme la variable est typé l'input ne conserve pas la valeur invalide ce qui est horrible ergonomiquement...

Le plus simple c'est de laisser ton input texte est de faire une validation regex avec /^[\d]*$/ (0 ou N numéros)

Tu peux utiliser ça directement dans le fichier validation.yml ?
Ou bien tu crée une fonction dans ton entité et tu test sa valeur de retour ? (ça me paraitrait plus logique, mais sait-on jamais).

Je ne me suis jamais servi d'expression régulières, il va falloir que je potasse un peu le sujet.
Je reviendrais mettre mon résolu si j'y parviens, en attendant je laisse ouvert, peut-être y a t-il une autre solution détenue par quelqu'un :P

Code:

---

1 2 3 4 5 6

Acme\BlogBundle\Entity\acme: properties: age: - Regex: pattern: "/\d/" message: Votre age doit etre un nombre

---

issu de la doc de symfony2 : http://symfony.com/fr/doc/current/re...nts/Regex.html

Pour accélérer ton apprentissage et le test de tes regex je te conseil le site regex 101 tape ma regex dedans par exemple tu verras si ca match ou non. tu as en plus la description en haut a droite ou des rappels en bas a droite.

Bon cette fois c'est presque bon :P

J'ai raté ce point dans la doc (et pourtant je suis souvent dessus), merci beaucoup de me l'avoir pointé du doigt :)

Du coup ça marche bien... Mais (et oui il reste un "mais") maintenant il m'affiche deux messages d'erreurs :calim2:

J'ai bien mon message personnalisé :

Code:

---

1 2 3 4

maxDuration: - Regex: pattern: "/\d/" message: La durée maximum doit être un nombre

---

Mais il vient me rajouter :

Citation:

---

Cette valeur doit être un nombre.

---

Est-ce que je deviens fou ? Le message que je met ne devrait-il pas remplacer le message originel ?
En fait il agit comme si il y avait deux messages. Si je laisse vide j'obtiens :

Citation:

---

Cette valeur n'est pas valide.
Cette valeur doit être un nombre.

---

Et quand j'ajoute:

Code:

---

message: La durée maximum doit être un nombre

---

il ne remplace en fait que le premier. Sauf que je ne veux pas du deuxième :aie:

tu l'as bien mis en text?

Code:

---

$builder->add('MinDuration','text'

---

Oui je l'ai repassé en text :mrgreen:

Code:

---

1 2 3 4

$builder->add('MinDuration','text', array( 'label' => 'Durée minimum (en secondes)', 'read_only' => $readOnly ));

---

Les vérifications un peu basiques mais je demande quand même:
tu as bien vidé le cache?
tu n'as pas d'autres règles de validation sur maxDuration?
tu n'as pas d'annotation de validation dans ton entité?
Pas de confusion entre minDuration et maxDuration?

tu as bien vidé le cache?
Oui

tu n'as pas d'autres règles de validation sur maxDuration?
Une autre mais qui elle de son côté fonctionne bien, ne m'affichant rien quant elle est juste et une seule erreur si elle n'est pas respectée :

Code:

---

1 2 3 4

- Range: min: 0 minMessage: La durée ne peut pas être négative

---

tu n'as pas d'annotation de validation dans ton entité?
Non, j'ai fais toutes mes contraintes dans validation.yml

Pas de confusion entre minDuration et maxDuration?
Non, en fait j'ai trois champs

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

$builder->add('MinDuration','text', array( 'label' => 'Durée minimum (en secondes)', 'read_only' => $readOnly )); $builder->add('MaxDuration','text', array( 'label' => 'Durée maximum (en secondes)', 'read_only' => $readOnly )); $builder->add('FixedDuration','text', array( 'label' => 'Durée fixe (en secondes)', 'read_only' => $readOnly ));

---

Et j'ai (presque) les mêmes contraintes sur chaques

Code:

---

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

properties: minDuration: - Regex: pattern: "/\d/" message: La durée minimum doit être un nombre - Range: min: 0 minMessage: La durée ne peut pas être négative maxDuration: - Regex: pattern: "/\d/" message: La durée maximum doit être un nombre - Range: min: 0 minMessage: La durée ne peut pas être négative fixedDuration: - Regex: pattern: "/\d/" message: La durée fixe doit être un nombre - Range: min: 0 minMessage: La durée ne peut pas être négative getters: DurationValid: - "True": { message: "Vous devez spécifier une durée fixe ou une amplitude (pas les deux)"} AmplitudeValid: - "True": { message: "La durée maximum doit être supérieure à la durée minimum"}

---

Enfait les contraintes sont typées. Ca correspond aux catégories du lien ci dessous:
http://symfony.com/fr/doc/current/bo...tes-supportees
Range ne s'applique que sur une donnée numérique.

Perso c'est le seul truc que je n'aime pas sur symfony2 :p

La regex exclue déjà les nombre négatifs
Tu peux combiner les deux messages, ou laisse le message de la regex et te dire que si le mec marque -45 en durée, c'est qu'il a fait une faute de frappe ou qu'il sait retourner dans le temps.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

properties: minDuration: - Regex: pattern: "/\d/" message: La durée minimum doit être un nombre positif maxDuration: - Regex: pattern: "/\d/" message: La durée maximum doit être un nombre positif fixedDuration: - Regex: pattern: "/\d/" message: La durée fixe doit être un nombre positif

---

Effectivement il est un peu chatouilleux :P
D'un autre côté ça peut se comprendre.

Enfin pour en revenir à notre sujet, la regex telle que tu l'as faite laissait passer les nombres négatifs et aussi les trucs du genre "45toto".
Mais après avoir lu un tutoriel très bien fait sur les regex (maintenant je suis au point sur le sujet :)), j'ai pu facilement adapter en rajoutant les caractères de début et fin de chaine : ^$
J'ai donc quelque chose maintenant de parfaitement fonctionnel :D

Merci beaucoup pour ton aide et merci également à Nico_F et -Rpass- :)

Problème résolu