Discussion :

[floctc]

Bonjour, je dois ajouter la fonctionnalité "Supprimer un client" dans une application et j'ai plusieurs soucis :
les données d'une fiche client sont affichées sur une page ficheClients.php, et c'est ici que je dois ajouter la fonction supprimer. Je me pose deux questions :

- dois-je mettre cette fonction sous la forme d'un bouton supprimer qui envoie vers la page ficheClients2.php (là où s'effectuent les requêtes de suppression)

-dois-je mettre un lien avec pour titre supprimer qui renvoie vers la page ficheClient2.php?

De plus, comment faire pour passer les paramètres de la page ficheCilents.php (contenus dans $_GET) à la page ficheClients2.php ?

Merci de votre aide.

[nicoaix]

Tu peux faire un formulaire avec un bouton "supprimer" et tu passes tes données en post ($_post).

[floctc]

J'y ai pensé, mais le problème c'est que avec cette méthode, si je veux par exemple faire un echo de la variable $numCli (le numéro du client) j'ai un message qui m'indique que la variable n'est pas définie

Voila comment j'ai construit mon formulaire :

Code :

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

1
2
3
4
 
<form method="post" action="ficheClient2.php" name="formSupprimerTesteur" class="cssform">
	<input type="submit" value="Supprimer" class="bouton" />
</form>

[waldo2188]

Salut,

Et pourquoi pas simplement un lien qui redirige l'utilisateur vers ta page ficheClients.php en lui ajoutant un attribut.
par exemple : ficheClients.php?clidel=1

dans un fichier ficheClients.php tu test si la variable $_GET['clidel'] existe et dans ce cas la tu effectues les opérations pour supprimer le client.

il ne reste plus qu'a afficher un petit message à l'utilisateur pour l'informer de l'état de la suppression.


cela répond t'il à ta question?

[floctc]

Oui, ca me parait une bonne solution et ça m'évite de créer le fichier ficheClient2.php.
Seulement, si je fais un echo $numCli, rien ne s'affiche et j'ai cette erreur :

Notice: Undefined variable: numCli in C:\wamp\www\appli\html\ficheClient.php on line 20
suppression du client n°

Voilà comment je m'y suis pris, ce n'est peut être pas la bonne méthode ...

Code :

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

1
2
3
4
 
if (isset($_GET['suppClient']) && $_GET['suppClient'] == 1 ) {
	echo "suppression du client n° $numCli";
	}

et pour le lien :

Code :

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

1
2
 
<a href="ficheClient.php?suppClient=1">Supprimer</a>

Merci!

[nicoaix]

Ta variable $numCli est définie où?
Attention si tu supprimes le client avec le numéro x et qu'ensuite tu recherches le client qui a le numéro x tu auras forcément une absence de valeur en retour.

[floctc]

Je n'ai plus l'erreur, j'avais mis le bout de code avant la définition de $numCli, mais maintenant le echo n'affiche rien! Pourtant, je veux juste vérifier que je récupère le bon numéro de client pour pouvoir le supprimer, je n'ai pas encore fait le code pour le supprimer, donc il est toujours là.

Une idée de mon erreur?

Code :

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

1
2
3
4
5
6
 
if (isset($_GET['numSociete'])) {
	$numCli =  $_GET['numSociete'];	
	if (isset($_GET['suppClient']) && $_GET['suppClient'] == 1 ) {
		echo "suppression du client n° $numCli";
	}

[Invité]

ATTENTION DANGER !

Si le parametre est passé dans l'URL (suppClient), RIEN ne l'empeche de changer la valeur de suppClient dans l'URL, et supprimer la fiche d'UN AUTRE CLIENT !!

Pour ce genre de manip., tu as plus interet a mettre l'idclient dans une variable de session, ou a la rigueur en $_POST !
Mais surtout pas en $_GET !

[waldo2188]

Il est clair que dans la variable get c'est pas ce qu'il y a de plus sécurisant ^^.

En effet rien empêche l'utilisateur de changer l'id du client pour en tuer un autre.

Un formulaire est aussi tout à fait jouable en utilisant la même technique.
Le formulaire pointe sur la même page

Code :

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

1
2
3
4
5
if (isset($_POST['numSociete'])) {
	$numCli =  $_POST['numSociete'];	
	if (isset($_POST['suppClient']) && $_POST['suppClient'] == 1 ) {
		echo "suppression du client n° $numCli";
	}

[floctc]

C'est bon j'ai réussi à faire ce que je voulais. J'ai opté pour le lien qui renvoie sur la même page, en utilisant $_GET, car l'application étant en local et l'accès à la page de suppression sécurisée, je peux me permettre de passer les données par la méthode get.

Code :

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

1
2
3
4
5
 
if(isset($_GET['suppFiche']) && $_GET['suppFiche']!='') {
sql("DELETE FROM societe WHERE NumSociete=".$_GET['suppFiche']." ",3);
header("Location: ficheClient.php");
}

Merci de votre aide!