Discussion :

[Fahelis]

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 :

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

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 :

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 :

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

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


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.

[-Rpass-]

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

[Fahelis]

Oui,

Voici le formulaire :

Code :

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

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 :

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

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 ).

[-Rpass-]

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

Code :

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

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 :

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

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

[Fahelis]

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
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 ?

[Nico_F]

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.

[Fahelis]

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

[Nico_F]

D'ou ma dernière phrase

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.

[Fahelis]

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

[gototog]

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)

[Fahelis]

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

[gototog]

Code :

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

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.

[Fahelis]

Bon cette fois c'est presque bon

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

J'ai bien mon message personnalisé :

Code :

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

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

Mais il vient me rajouter :

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 :

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

Et quand j'ajoute:

Code :

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

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

[gototog]

tu l'as bien mis en text?

Code :

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

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

[Fahelis]

Oui je l'ai repassé en text

Code :

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

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

[gototog]

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?

[Fahelis]

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 :

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

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 :

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

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 :

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
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"}

[gototog]

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 :

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

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

[Fahelis]

Effectivement il est un peu chatouilleux
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

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

Problème résolu