suppression d'enregistrement et message de confirmation

**RunCodePhp** · 26/04/2012, 12h52

Code :

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

1
2
3
4
5
6
7
8
9
10
		<table>
			<th>Confirmation de suppression de l'enregistrement</th>
 
			<form method="post" action="suppr.php">
			<input type="hidden" name="action" value="<?php echo $action;?>" />
			<input type="hidden" name="cdec" value="<?php echo $cdec; ?>" />
			<input type="submit" name="action" value="Supprimer"> &nbsp;&nbsp;
			<a href="../interfaces/tab.php">Annuler</a>
			</form>
		</table>

Si ton code est exactement comme celui c-dessus, alors il y a une erreur de HTML, peut être que le navigateur n'apprécie pas et renvoie rien concernant le formulaire.
(pure supposition).

Tu as 2 éléments du même nom (action), ce n'est pas correcte :

Code :

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

1
2
			<input type="hidden" name="action" value="<?php echo $action;?>" />
			<input type="submit" name="action" value="Supprimer">

Change le nom du submit par exemple.

Le tableau n'a pas de ligne de définie (pas de tr).
Le formulaire se trouve dans un tableau mais dans aucune cellule (un td th).
Tout élément doit se trouver dans une cellule, et le ou les cellules doivent être dans une ligne.

Par ailleurs, la norme W3C dit que tous les éléments dans un formulaire (input, select, textarea, etc ...) doivent être placés dans un élément de type bloc.
L'élément le plus adapté est : <fieldset>, par moment accompagné d'une légende <legend>
On voit cependant une <div>, des <p>, etc ... Pourquoi pas.

Quoi qu'on fasse faut respecter ces règles de bases sinon on peu faire fasse à des comportements plus ou moins hasardeux ou capricieux des navigateurs.

Exemple :

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
 
<!-- FORMULAIRE -->
<form>
    <fieldset>
        <legend>Veuillez confirmer la suppression</legend>
        <input type="submit" name="action" value="supprimer_conf" />
    </fieldset>
</form>
 
<!-- TABLEAU -->
<table><!-- déclaration du tableau -->
<tr><!-- ligne 1 -->
    <th>Cellule 1</th>
    <th>Cellule 2</th>
</tr>
<tr><!-- ligne 2 -->
    <td>Cellule 1</td>
    <td>Cellule 2</td>
</tr>
</table>

Je ne vois pas vraiment l'utilité d'un tableau dans ce fichier suppr.php.

Au passage, théoriquement on n'utilise pas (ou plus) le HTML pour de la mise en forme, mais essentiellement le Css.
(on peu rajouter des styles sur une balise form, un fielset, input, etc ... pour obtenir une présentation correcte).

Attention tout de même à ne pas utiliser les tableaux comme élément HTML quelque peu fourre-tout.
Si c'est une partie Admin, pourquoi pas (faut au moins faire un code valide), mais pour une partie publique ou le référencement est nécessaire ce n'est plus pertinent, ça peu être très néfaste même.
C'est juste une remarque.

**redoran** · 26/04/2012, 14h09

RunCodePhp:

Le tableau n'a pas de ligne de définie (pas de tr).... ect....

j'étais juste dans le fonctionnel bien sur il reste la mise en forme avec du css merci pour les orientations
actuellement je suis absorbé par la variable $_POST['cdec'] qui prend la valeur null après la validation de suppression.

**RunCodePhp** · 26/04/2012, 15h17

j'étais juste dans le fonctionnel bien sur

Effectivement, il vaut mieux toujours faire simple au départ.
Mais encore faut il que le code simple HTML ne soit pas erroné.
Des balises manquantes peu provoquer des dysfonctionnent comme celui que tu as.
Comme 2 fois le même nom (2 fois action), etc ...
C'est largement suffisant pour que cela ne fonctionne mas normalement.
Comme je ne vois pas d'autres erreurs que celles là, je me dis que ...

En supprimer justement ce qui ne sert à rien (<table>, <th>, etc ...) juste conserver le <form> avec un <fieldset> (comme l'exemple), avoir un seul input avec comme nom "action", obtiens tu le même problème de valeurs null en POST ?
(remets ton code avec ces correction qu'on voit)

Aussi, avant de valider la confirmation de la suppression, et en faisant un clic droit dans la page pour afficher le code source HTML généré, vois tu les divers éléments du formulaire avec les valeurs attendues/prévues (action et cdec) ?

Par ailleurs, comme la suite de cette dernière action (après validation) est assez particulière (contrôle, requête SQL et surtout une redirection), pour juste vérifier on peu faire comme ceci :
Exemple :

Code :

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

1
2
3
4
5
6
7
8
elseif (isset($_POST['action'], $_POST['cdec']) && $_POST['action'] === 'supprimer_conf') {
    print_r($_POST);
    // Vérification de la valeur de cdec (filter_input)
    echo 'bien dans la boucle else if';
 
    exit(); // point d'arrêt
    // ... etc ...
}

Je ne vois toujours pas de point d'arrêt dans ton code après le header('Location: ...'), normalement il faut le mettre sinon tu risque d'avoir des problèmes.

**redoran** · 26/04/2012, 15h40

effectivement ta raison , le temps que tu rédigé ton poste j'étais en plein chantier de ma page suppr.php.

Comme 2 fois le même nom (2 fois action), etc ...

bien vue +1

Code :

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

elseif (isset($_POST['sub'], $_POST['cdec']) && $_POST['action'] === 'supprimer_conf')

donc c'est isset($_POST['sub'] au lieu de isset($_POST['action'],
ensuite j'ai affecté a la $cdec= $_POST['cdec'] après le elseif parce que elle prenais la valeur null une fois dans la boucle elseif.

code complet de la page suppr.php

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
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
 
<?php
........
// Action par défaut
$action = $_POST['action'];
$cdec='';
 
if (isset($_POST['action'], $_POST['cdec']) && $_POST['action'] === 'supprimer') {
    $cdec = $_POST['cdec'];
    $action = 'supprimer_conf';
	echo 'bien dans la boucle if ';
	echo $_POST['cdec'];echo $_POST['action'] ;
}
elseif (isset($_POST['sub'], $_POST['cdec']) && $_POST['action'] === 'supprimer_conf') {
 
echo 'bien dans la boucle else if';	    
     // affectation de la valeur $_POST['cdec']
	$cdec = $_POST['cdec'];
 
 
// Vérification de la valeur de cdec (filter_input)
		$int_options = array("options"=> array("min_range"=>0, "max_range"=>1000000000));
		$verif=filter_var($cdec, FILTER_VALIDATE_INT, $int_options);
    // Si la valeur est correcte
    if ($verif !== FALSE) {
        // Suppression / Requête SQL
					$supenreg=$cbd->prepare('DELETE  FROM tab where cdec=:cdec ');
					$supenreg->bindParam(':cdec', $cdec , PDO::PARAM_INT); 
 
					try {
						// On envois la requête
							$supenreg->execute();
						// libèré la connexion du serveur
							$supenreg->closeCursor();	
						header ('location: ../interfaces/tab.php'); 		
 
						}
						catch( Exception $e )
						{
						  echo 'Erreur d\'affichage : ', $e->getMessage();
						}  
        // Redirection
        header('Location: ../interfaces/tab.php');
       exit;
    }
}
 
?>
<!DOCTYPE html>
<HTML>
<head>
....ect...
</head>
<body>
<?php    ?>
	<div>
 
			<p>Confirmation de suppression de l'enregistrement</p>
 
			<form method="post" action="suppr.php">
			<input type="hidden" name="action" value="<?php echo $action;?>" />
			<input type="hidden" name="cdec" value="<?php echo $cdec; ?>" />
			// changement name du submit action par sub
                        <input type="submit" name="sub" value="Supprimer"> &nbsp;&nbsp;
			<a href="../interfaces/tab.php">Annuler</a>
 
	</div>
 
</body>
</html>

**RunCodePhp** · 26/04/2012, 15h57

elseif (isset($_POST['sub'], $_POST['cdec']) && $_POST['action'] === 'supprimer_conf')

Si tu veux vérifier la présence du bouton submit "sub", alors il serait mieux de faire ainsi :

Code :

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

elseif (isset($_POST['action'], $_POST['sub'], $_POST['cdec']) && $_POST['action'] === 'supprimer_conf')

En somme on conserve la vérification sur "action" en rajoutant "sub" (et non remplacer l'un par l'autre).
Il faut que ces 3 éléments existent.

Je ne vois pas la balise fermante </form> ?
Mise à part ça, obtiens tu toujours des erreurs ?
Entre tu dans cette alternative ou pas ?
Que donne le code source HTML généré ? (est il bon ou pas) ?

Je ne vois pas pourquoi tu mets 2 fois une redirection pointant vers la même page.
Théoriquement il faudrait conserver juste celle dans le try, (en oubliant pas le exit car ici il n'y en a pas).

Car s'il y a une erreur, elle sera "catchée" (attrapée), donc c'est le code dans la catch qui sera exécuté, et c'est un echo (un affichage) qui est fait.
Faire un header() (même en mettant un exit après) alors que juste avant on aura fait un revoie (echo) va provoquer l'erreur très connue :
Warning ... header allready sent by ... (une entête a déjà été renvoyé ...)

**redoran** · 26/04/2012, 16h11

Re ;

Mise à part ça, obtiens tu toujours des erreurs ?

non aucune erreur sa fonction très bien avec suppression de l'enregistrement voulu.

Je ne vois pas pourquoi tu mets 2 fois une redirection pointant vers la même page.

c'est du au copie coller

**RunCodePhp** · 26/04/2012, 16h20

non aucune erreur sa fonction très bien avec suppression de l'enregistrement voulu.

Ok.

N’oublie pas le </form> (à moins que ce soit juste un oubli ici seulement).
de même le rajout de $_POST['action'] dans le isset(), car si tu ne le rajoute pas et s'il n'existe pas, c'est le :
$_POST['action'] === 'supprimer_conf'
qui va provoquer une erreur.