Discussion :

[zonas]

hello !

J'ai une liste de dossiers/fichiers générée par php.

Ils sont classés dans un tableau avec pour chaque ligne un formulaire (seul le bouton supprimer est visible) et le nom de l’élément.

Lorsque que je clique sur supprimer, logiquement l’élément devrais être effacé.

Seulement cela ne fonctionne pas. :



Je vous serais gré si vous pouviez me déceler l'erreur qui se glisse sous mon nez.



Voila la fonction :

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
function list_fich($dir, $chemin)
{
	foreach($dir as $lien)
	{
		echo '<table><td><form method="post" action="liste_archive_admin.php">';
		echo "<input type=\"hidden\" name =\"lien\" value=\"$lien\">";
		echo "<input type=\"hidden\" name=\"chemin\" value=\"$chemin\" >";
		echo "<input type=\"submit\" name=\"bouton\" value=\"supprimer\" onclick=\"return confirmer()\"></td>";
		if(is_file($lien) == true)
		{
				echo "<td><a name=\"lien\" href=\"liste_archive_admin.php?path=$path/$lien\" onclick=\"javascript:ouvre('$_GET[path]/$lien'); return false\" >$lien</a></td></table>";
		}
		else
		{
				echo "<td><a name=\"lien\" href=\"#\" onclick=\"javascript:ouvre('$_GET[path]/$lien'); return false\" >$lien</a></td></table>";
		}
		echo "</form>";
 
 
		if( $_POST["lien"]== "$lien")
		{
			unlink("$_POST[chemin]/$_POST[lien]");
 
			echo "<script>window.location.replace(\"liste_archive_admin.php\")</script>";
		}
	}
}

A savoir que quand je fait :

Code :

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

echo"$_POST[chemin]/$_POST[lien]";

Cela affiche bien le chemin ET le nom de l'élément après click sur le bouton supprimer.


Pour info supplémentaire, j'appelle la fonctionn avec ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
if(!empty($_GET["path"]))
{
	$dossier = scandir($_GET["path"]) or die("Le dossier est vide.");
	echo "<h2>Liste des documents accessibles dans ce dossier :</h2>";
	unset($dossier[0], $dossier[1]);
	list_fich($dossier, $_GET["path"]);
}
 
 
$_GET["path"] //contenant le chemin vers $dossier.

[sabotage]

1 - tu as un problème de sécurité : tu laisses l'utilisateur fournir n'importe quel chemin sur le serveur, lister les fichier et éventuellement les supprimer
2 - tu as un problème de logique : tu listes les fichiers et commence à les afficher, si dans le lot, il y en a un a supprimer, tu le supprimes et tu changes de page, donc tout le début n'a servi à rien.
3 - soit plus précis que "ça ne fonctionne pas" ; par exemple est-ce que la redirection se fait mais pas la suppression ? Tu as essayé de retirer la redirection pour voir s'il y avait une erreur ? Tu as affiché toutes les erreurs PHP ?

Sinon il faut indiquer le chemin entier :

Code :

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

unlink($_SERVER['DOCUMENT_ROOT'] . DIRECTORY_SEPARATOR . $chemin . DIRECTORY_SEPARATOR . $fichier);

Je mets bien $chemin et $fichier car il n'est pas question de supprimer des éléments sans qu'il y ait un contrôle de sécurité qu'il te reste à écrire.

[zonas]

1 - tu as un problème de sécurité : tu laisses l'utilisateur fournir n'importe quel chemin sur le serveur, lister les fichier et éventuellement les supprimer

Seul certaines personne agréés auront accès a cette fonctionnalité, pas de soucis la dessus.

2 - tu as un problème de logique : tu listes les fichiers et commence à les afficher, si dans le lot, il y en a un a supprimer, tu le supprimes et tu changes de page, donc tout le début n'a servi à rien.

Mon but est de supprimer un dossier/fichier et de recharger la page: elle s’appelle elle même.

3 - soit plus précis que "ça ne fonctionne pas" ;

ok : Ca fonctionne pas tout a fait.

je dec'.

par exemple est-ce que la redirection se fait mais pas la suppression ? Tu as essayé de retirer la redirection pour voir s'il y avait une erreur ?

Lorsque je clique sur le bouton supprimer php est sensé supprimer le dossier/fichier lié puis recharger la page.

En pratique, php recharge bien la page mais ne supprime pas l’élément. :/



J'ai déjà réussi à supprimer des fichiers avec cette fonction :
Mon fichier .php était alors dans un dossier et ne listait que les éléments du même dossier. Je n'utilisait pas de paramètre url.

C'est dès lors que j'ai voulu utiliser le paramètre path et $_GET que j'ai du préciser le chemin du dossier. (puisque évidement le fichier .php n'est plus dans le dossier mais dans un dossier parent).

Tu as affiché toutes les erreurs PHP ?

Dans mon idiotie j'ai oublié que j'avais mit une ligne pour retirer les erreurs :/
J’essaie de réparer les 2 petites erreurs que j'ai déjà.

Merci de ta réponse.

[sabotage]

2 - oui donc la suppression n'a rien à faire dans la boucle qui liste les fichiers

Tu as essayé en mettant le chemin complet ?

[zonas]

2 - oui donc la suppression n'a rien à faire dans la boucle qui liste les fichiers

Pas faux.


J'ai utilisé la fonction avec le chemin complet :


Rien a faire, il me recharge la page mais ne fait rien d'autre.
Il n'affiche pas d'erreur non plus.

[sabotage]

Il faut définir $chemin et $lien comme étant les données venant de $_POST dans le unlink.

Et comme je t'ai dis, pour debuguer, retire la redirection et affiche toutes les erreurs PHP : unlink() renvoit des warning.