Discussion :

[Kagami]

bonjour,
est ce que quelqu'un a une idée sur laquelle mon fichier m'affiche cette erreur ? :

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
38
39
40
41
42
43
44
 
		<main>
 
				<h1>User</h1>
				<?php 
					regex($_POST);
					// $users = getUser();
					if ($_POST["button"] == "modifier"):?>
 
						<?php  
						$reponse = $db->query('SELECT * FROM user WHERE id='.$_GET["id"]);
 
						//while ($donnees = $reponse->fetch()):  
						foreach ($reponse as $key => $value): ?>
							<form method="POST" action="confirm_update?id=<?=$value["id"];?>">
								<label for="prenom">Prénom</label>
								<input class="<?=isset($ERRORS["prenom"])?'error':'';?>" id="prenom" type="text" name="prenom" 
									value="<?php echo ($value['prenom']) ?>" required>
								<br>
								<br>
								<label for="email">Adresse Mail</label>
								<input class="<?=isset($ERRORS["email"])?'error':'';?>" id="email" type="email" name="email" 
									value="<?php echo ($value['email']) ?>" required>
								<br>
								<br>
								<label for="phone">Numéro de téléphone</label>
								<input class="<?=isset($ERRORS["mobile"])?'error':'';?>" id="mobile" type="tel" name="mobile" 
									value="<?php echo ($value['mobile']) ?>" required>
								<br>
								<br>
								<p>* Tous les champs sont requis.</p>
								<input type="submit" value="Mettre à jour">
							</form>	
 
 
 
						<?php endforeach; ?> 
			<?php else:
						// $db->exec("DELETE FROM user WHERE id=".$_GET["id"]);
						echo "<p>L'utilisateur est supprimé<br>vous allez être redirigé</p>";
						// header("Refresh:3; url=".$_SERVER["HTTP_REFERER"]);
 
			endif;?>
			</main>

mon fichier pour confirmer mon update :

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
 
	<section>
					<?php   if(!empty($ERRORS)):?>
 						<?php 
 						 	//prenom: 
							$prenom     = $_POST["prenom"] ;
							//email:
							$email = $_POST["email"] ;
							//mobile:
							$mobile = $_POST["mobile"] ;
 
						  	$db->exec("UPDATE user set prenom=".$prenom ." email=".$email." mobile=".$mobile . "WHERE id=".$_GET["id"]);
							echo "<p>L'utilisateur est bien modifié </p>" 
						 ?>
 
						 <?php  else:
							$db->exec("DELETE FROM user WHERE id=".$_GET["id"]);
							echo "<p>L'utilisateur est supprimé<br>vous allez être redirigé</p>";
							// header("Refresh:3; url=".$_SERVER["HTTP_REFERER"]);
						?>	
					<?php endif; ?>
		</section>

merci

[sabotage]

Tu lances la requête UPDATE sans vérifier si tu as reçu les données $_POST.
Utilise isset().

[Kagami]

j'ai modifié mon code tu peux regarder

[Invité]

Bonjour,

Code php :

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

 					regex($_POST);

Kezako ??

Code php :

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

1
2
3
						$reponse = $db->query('SELECT * FROM user WHERE id= "'.$_GET["id"].'"');
...
						  $db->exec("UPDATE user set prenom=".$prenom ." email=".$email." mobile=".$mobile . "WHERE id=".$_GET["id"]);

Des données "externes" directement dans les requête : eh bêh, tu n'as peur de rien...
As-tu entendu parlé d'"injection SQL" ? de "requête préparée" ?

Code php :

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

1
2
3
4
5
						$prenom     = $_POST["prenom"] ;
						//prenom:
						$email = $_POST["email"] ;
						//adresse:
						$mobile = $_POST["mobile"] ;

A l'ouverture de page, le formulaire n'est pas encore soumis : $_POST est vide.
D'où les erreurs...

Code php :

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

1
2
3
4
5
						$prenom     = ( isset($_POST["prenom"]) )? $_POST["prenom"] : '';
						//prenom:
						$email = ( isset($_POST["email"]) )? $_POST["email"] : '';
						//adresse:
						$mobile = ( isset($_POST["mobile"]) )? $_POST["mobile"] : '';

Ensuite, il est bon de faire une GESTION d'ERREURS (champs obligatoires, validité email,...) avant d'enregistrer en Bdd.

Enfin (?), la partie "TRAITEMENT" est à mettre AVANT le FORMULAIRE (dans le code), pour pouvoir ré-afficher dedans les données reçues.