Petit soucis de validation [Résolu]

[philippef]

Bonjour,

C'est surement simple !! mais je vois pas
Quand je valide le formulaire et que je n'ai pas de message $id_proprio ne s'affiche pas

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
if (isset($_SESSION['id_proprio'])){
	$id_proprio = $_SESSION['id_proprio'];
}
$message = array();
 
if (isset($_POST['valider'])){
	// Affichage des messages/infos
	if (!empty($message)) {
    		foreach ($message as $msg) {
        		echo '<p align="center"><span style="color:red">'.$msg.'</span></p>';
    		}
	}
	//si pas de message, enregistrement de l'annonce
	if (empty($message)) {
		echo "$id_proprio";
	}
}

Je fais un echo de $id_proprio juste pour voir si mes valeurs sont correctes

[sabotage]

Assure toi d'avoir bien démarré la session et controle son contenu.

Code :

var_dump($_SESSION);

Au passage simplement

Code :

echo $id_proprio;

[philippef]

Var_dump($_SESSSION) me renvoie bien tous les paramètres passés en session dont id_proprio.

Si je fais un echo $id_proprio juste après if (isset($_POST['valider'])) l'id_proprio apparait bien.

Si je provoque une erreur j'entre bien dans la boucle foreach

Est ce que ce n'est pas dû à $message=array(); ?

[ABCIWEB]

Envoyé par philippef

Est ce que ce n'est pas dû à $message=array(); ?

Logiquement il faudrait définir le contenu de $message à l'intérieur de la condition if (isset($_POST['valider']))

[sabotage]

Est ce que ce n'est pas dû à $message=array(); ?

Je pensais que cette ligne n'était pas que pour le test.
Si tu définis manuellement $message, a quoi cela sert-il de tester sa valeur juste en dessous ?

[philippef]

Envoyé par sabotage

Je pensais que cette ligne n'était pas que pour le test.
Si tu définis manuellement $message, a quoi cela sert-il de tester sa valeur juste en dessous ?

Dans une autre page je controle que le formulaire est correctement rempli et j'utilise $message[] = 'mon texte'; J'ai environ 15 controles

Avant d'enregistrer le formulaire je controle $message . Si il y a des messages je les affiche, s'il n'y a pas de message alors j'enregistre.

[sabotage]

Pourquoi dans le code que tu nous montres définis tu $message comme un tableau vide et juste en dessous controles-tu s'il est vide ou non ?

[philippef]

Pardon si je ne suis pas clair dans mon explication.
Je redonnes le code complet de ma page

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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
<?php
require('proprietaires/annonces/fonctionsphp.php');
require ('connect.php');
if (isset($_SESSION['id_proprio'])){
	$id_proprio = $_SESSION['id_proprio'];
}
$message = array();
 
$departement = "";
//
$displayDiv0 = 'none';
$displayDiv1 = 'block';
$displayDiv2 = 'none';
$displayDiv3 = 'none';
$displayDivSaison = 'none';
$semaine = array();
$Disponibilite = array();
 
$year = date('Y');
$date = date("Y-m-d");
//récupération du n° de la dernière annonce
$sql = mysql_query("SELECT ref_annonce FROM logements ORDER BY ref_annonce DESC Limit 1");
$nombreligne = mysql_num_rows($sql);
if ($nombreligne > 0){
	$res = mysql_fetch_array($sql);
	$ref_annonce = $res['ref_annonce'];
	$ref_annonce = $ref_annonce + 1;
}
else {
	$ref_annonce = "000001";
}
include ('b_control_formulare.php');
 
//*************************************************************
//   Enregistrements dans la table logements
//*************************************************************
if (isset($_POST['valider'])){
	// Affichage des messages/infos
	if (!empty($message)) {
    		foreach ($message as $msg) {
        		echo '<p align="center"><span style="color:red">'.$msg.'</span></p>';
    		}
	}
	//si pas de message, enregistrement de l'annonce
	if (empty($message)) {
		echo $id_proprio $type_logement $codep $ville;
	}
}
//******** FIN *************
if (empty($_SESSION['id_proprio'])){
	$displayDiv1='none'; 
	echo "<div style='position:relative;width:470px;margin-left:10px;background-color:#ffffff;'>";
	echo "<p align='center'><span style='color:red;'><br /><br />Merci de vous enregistrer ou de vous identifier pour déposer une annonce</span></p></div>";
}
else {
	include ('c_insert_update.php');
	include ('d_property_formulare.php');
	include ('e_calendar_formulare.php');
}
?>

j'espère que c'est plus clair comme ça

[sabotage]

tu as un problème de syntaxe :

Code :

1
2
3
if (empty($message)) {
	echo $id_proprio . ' ' . $type_logement . ' ' . $codep $ville;
}

[philippef]

Même si je ne mets que echo $id_proprio; rien ne s'affiche.

C'est bien ce code qui pose problème

Code :

1
2
3
if (empty($message)) {
           echo $id_propio;
}

Puisque ce code fonctionne et il affiche les messages

Code :

1
2
3
4
5
	if (!empty($message)) {
    		foreach ($message as $msg) {
        		echo '<p align="center"><span style="color:red">'.$msg.'</span></p>';
    		}
	}

Est ce qu'il y a une autre solution ?

[micetf]

Je ne serais pas étonné si $message n'était pas vide.
Deux manières de le savoir :
- afficher le code source généré et voir si cette ligne html apparait (le contenu du tableau $message a été parcouru, mais il contient un espace) :

Code :

<p align="center"><span style="color:red"> </span></p>

- ajouter cette ligne avant le test sur le $_POST['valider'] (test explicite) :

Code :

echo (empty($message)) ? '<p>Message vide</p>' : '<p>Message non vide</p>';

Si mon hypothèse se confirme, il faudra aller faire un tour dans b_control_formulare.php et voir comment tu gères $message dans ce script.
Fred

[philippef]

Je viens de regarder le code source et je n'ai rien vu d'anormal (pas d'espace)

par contre le 2è test affiche 'Message non vide'.

Pour les zones obligatoires mes messages sont construits de la même façon

Code :

$message[] = "La zone XX n'est pas renseignée.";

Est ce que ' peut créer un problème?

[micetf]

Je persiste.
Quelque part dans b_control_formulare.php,
tu dois faire ceci :

Code :

$message[]='';

et, dans ce cas, $message n'est pas vide :
il contient un élément vide (ce qui n'est pas pareil).

D'où la ligne html dans le code source :

Code :

<p align="center"><span style="color:red"></span></p>

sans espace (je fais une concession) entre les balises <span>.
Si $message était réellement vide, cette ligne n'existerait pas.

Me trompe-je ?

[philippef]

Tu veux dire que je dois traiter ce code autrement ?

Code :

1
2
3
4
5
6
7
if (!empty($_POST['nbre_pieces'])) {
	$chaine = $_POST['nbre_pieces'];
	if (tout_numerique($chaine))
		$message[] = "";
	else
		$message[] = "Le nombre de pièces n'est pas valide.";
}

Je n'ai pas d'espace entre <span> et </span>

[sabotage]

Tout simplement :

Code :

1
2
3
if (isset($_POST['nbre_pieces']) && !tout_numerique($_POST['nbre_pieces'])) {
	$message[] = "Le nombre de pièces n'est pas valide.";
}

[micetf]

Effectivement,
en remplaçant :

Code :

1
2
3
4
	if (tout_numerique($chaine))
		$message[] = "";
	else
		$message[] = "Le nombre de pièces n'est pas valide.";

par

Code :

1
2
	if (!tout_numerique($chaine))
		$message[] = "Le nombre de pièces n'est pas valide.";

Lorsqu'il n'y a pas de message, tu devrais voir ton id_proprio s'afficher.
Fred

[philippef]

Merci à vous Sabotage et Micetf, c'est bien $message= "" qui m'ennuie.

Je viens de corriger et ça marche