suppression d'un avatar

Version imprimable

bonjour à tous, sur le site que je suis entrain de développer en projet pour mon iut, j'ai donné la possibilité aux utilisateurs de mettre un avatar.
l'upload et le référencement de celui-ci dans la base de donnée fonctionne parfaitement. J'ai fait une requete qui supprime l'url de l'avatar dans la base de donnée en cas de suppression de l'utilisateur. jusqu'ici tout fonctionne mais si je laisse comme ça, mon serveur va se retrouver blinder d'avatar obsolete. j'ai donc essayé de supprimer les fichiers avec la fonction unlink() mais apparement je n'ai pas les droits. je travaille sous windows en local avec wamp et voilà le bout de code que j'ai mis :
Code:

1 2 3 4 5 6 $quer=mysql_query("select id_avatar from internaute where id_internaute='".$element[$i]['id_internaute']."';"); $query=mysql_fetch_array($quer); $quer2=mysql_query("select url from avatar where id_avatar='".$query['id_avatar']."';"); $query2=mysql_fetch_array($quer2); chmod('http://localhost/boby/image/avatar',0777); unlink($query2['url']);
l'url récupérée dans $query2['url'] est bonne je pense puisque je l'utilise pour afficher l'avatar dans plusieurs pages de mon site. ( dans url, il y a aussi le fichier . jpeg mais j'ai essayé en le retirant sans aucun changements ... )
si quelqu'un a une idée pour ce problème de droits, j'ai deja essayé tout ce que je savais :S

25/05/2010, 18h49
sabotage
Déjà tu peux faire une seule requête avec jointure.
Ensuite chmod() sur du windows, ce n'est pas vraiment bien.
unlink() fonctionne avec des chemins complets :
Code:

1 2 3 4 $quer=mysql_query("select url from internaute JOIN avatar USING (id_avatar) WHERE id_internaute=".$element[$i]['id_internaute']); unlink($_SERVER['DOCUMENT_ROOT'] . mysql_result($quer,0));
Je suppose par contre que ton champ "url" contient seulement le nom du fichier et non une url.

En prime, c'est toujours une bonne chose de nettoyer une chaine qu'on utilise avec unlink de tout caractère suspect comme \.
25/05/2010, 20h30
Kuchiki Byakuya

pour mon champs url, non c'est réellement une url du style : http://localhost/boby/image/avatar/11.jpg
je vais regarder ce que tu m'as donné, j'imagine que $_SERVEUR['DOCUMENT_ROOT'] est sensé contenir le chemin complet non ?
en tout cas en testant ce que tu m'as donné j'ai toujours la meme erreur :

Warning: unlink(localhost/boby/image/avatar/73.jpg) [<a href='function.unlink'>function.unlink</a>]: No such file or directory in C:\wamp\www\boby\script_modification.php on line 52
25/05/2010, 21h23
sabotage

C'est une erreur de stocker ton url complète :
- tu repetes la meme information sur toute les lignes
- tu ne peux plus désigner le fichier sur le disque
25/05/2010, 22h15
Kuchiki Byakuya

d'accord, je changerai ça. mais meme si je change ça ton code ne suffit pas si ? parce quand je le met pr le moment ça me met l'erreur que j'ai écrite au dessus. ( désolé si je ne suis pas très rapide, je ne suis pas tres doué en php vu qu'on viens de le commencer en cours :roll: )
EDIT : le tableau $_SERVER['DOCUMENT_ROOT'] est créé automatiquement lors de l'upload du fichier ou il faut que je le remplisse moi même avec l'adresse complète du dossier où sont stoquées les images ?
25/05/2010, 23h52
sabotage

Ca ne peut pas fonctionner puisque ta reference de fichier est mauvaise.

Il y aura par contre peut etre besoin d'ajouter un "/" ; selon les systemes DOCUMENT_ROOT contient ou non le / final.

Code:

unlink($_SERVER['DOCUMENT_ROOT'] . "/" . mysql_result($quer,0));
25/05/2010, 23h55
Madfrix

Citation:

Envoyé par Kuchiki Byakuya

J'ai fait une requete qui supprime l'url de l'avatar dans la base de donnée en cas de suppression de l'utilisateur

Le mieux à faire dans ce cas là (le plus optimisé et la méthode permettant à 100% de ne pas garder des bribes d'informations dans ta base) est de construire ta base avec le moteur MySQL InnoDB te permettant automatiquement de supprimer toutes les informations de l'utilisateur de tes tables adjacentes via les foreigns keys et les instructions ON DELETE CASCADE notamment
26/05/2010, 01h20
Kuchiki Byakuya
Citation:

Envoyé par Madfrix

Le mieux à faire dans ce cas là (le plus optimisé et la méthode permettant à 100% de ne pas garder des bribes d'informations dans ta base) est de construire ta base avec le moteur MySQL InnoDB te permettant automatiquement de supprimer toutes les informations de l'utilisateur de tes tables adjacentes via les foreigns keys et les instructions ON DELETE CASCADE notamment

c'est ce que je voulais faire au début mais les profs nous ont demandé d'essayer de faire autrement :S

@sabotage : je vais essayer ça demain matin, pour information, j'upload en utilisant la fonction copie, est ce approprié à ta méthode ou pas ?
EDIT : finalement j'ai testé ce soir. dans url j'ai 75.jpg ce qui est le nom du fichier à supprimer. j'ai mis ça :
Code:

1 2 $quer=mysql_query("select url from internaute JOIN avatar USING (id_avatar) WHERE id_internaute=".$element[$i]['id_internaute']); unlink($_SERVER['DOCUMENT_ROOT'] ."/". mysql_result($quer,0));
mais ça ne marche pas et si je veux afficher la variable serveur, il me dit qu'elle n'est pas definie.
26/05/2010, 11h15
sabotage

Citation:

j'upload en utilisant la fonction copie

On utilise plutot move_uploaded_file() qui est faite pour ça.
Attention sur la notion par contre, que tu utilises copy() ou move_uploaded_file() ces fonctions ne font que déplacer un fichier qui est déjà arrivé dans un repertoire temporaire sur son ton serveur lors de l'envoi du formulaire.

Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 if(is_uploaded_file($_FILES['fichier']['tmp_name']))//test si le fichier est réellement uploader sur le dossier temp { $nom = $_FILES['fichier']['name'];//met le nom du fichier dans la variable $nom $type = str_replace('.','',strstr($nom, '.'));//met dans $type les caractères du nom qui sont APRES le . afin d'obtenir l'extension du fichier. $login=$_SESSION['login']; $quer=mysql_query("select id_avatar from internaute where login='$login';")or die(mysql_error());// $query=mysql_fetch_array($quer); mysql_query("DELETE FROM avatar where id_avatar='".$query['id_avatar']."';")or die(mysql_error());// suppression de l'ancien avatar mysql_query("INSERT INTO avatar values('','aucun');")or die(mysql_error());// création du tuple pour lui donner un id $requete=mysql_query("select MAX(id_avatar) from avatar;")or die(mysql_error()); // on selectionne le dernier id inséré $req=mysql_fetch_array($requete); $url=SITE.'image/avatar/'.$req['MAX(id_avatar)'].'.'.$type; // création de l'url de l'image pour la mettre dans la base $id_avatar=$req['MAX(id_avatar)']; mysql_query("UPDATE internaute SET id_avatar='$id_avatar' where login='$login';")or die(mysql_error()); // on update la table internaute pour donner au nouvel inscrit son avatar. mysql_query("UPDATE avatar SET url='$url' where id_avatar='$id_avatar';")or die(mysql_error()); //update la table avatar pour mettre l'url if(move_uploaded_file($_FILES['fichier']['tmp_name'], './image/avatar/'.$req['MAX(id_avatar)'].'.'.$type)) //copie l'avatar sur l'espace de stockage, et test si celà a fonctionné. { echo 'avatar modifié'; }
voilà comment j'upload mon image, ici $_FILES['fichier']['tmp_name'] est il l'équivalent de ton $_SERVEUR['DOCUMENT_ROOT'] ??
meme ainsi je n'arrive pas a faire fonctionner ton code car il ne reconnait pas la variable $_SERVEUR
EDIT :bon apparement ta requete avec jointure il ne l'aime pas beaucoup, car elle ne reçoit aucun résultat, j'ai donc remis mes deux requetes et cela renvoie quelque chose.
voilà le bout de code que j'ai pour le moment :
Code:

1 2 3 4 5 $quer=mysql_query("select id_avatar from internaute where id_internaute='".$element[$i]['id_internaute']."';"); $query=mysql_fetch_array($quer); $quer2=mysql_query("select url from avatar where id_avatar='".$query['id_avatar']."';"); $query2=mysql_fetch_array($quer2); unlink(SITE.'./image/affiche/'.$query2['url']);
la constante SITE contient : http://localhost/boby/
( j'ai remis un chemin au lieu du $_SERVEUR['DOCUMENT_ROOT'] car il me dit que cette variable n'existe pas ).
mon erreur a changé :
Warning: unlink() [<a href='function.unlink'>function.unlink</a>]: http does not allow unlinking in C:\wamp\www\boby\script_modification.php on line 52 
il s'agit donc d'un problème de droits et non d'adresse non ?
En attente de votre aide :mrgreen:

Désolé pour le double post mais si je continue, ça va juste être illisible :mrgreen:
bon finalement j'ai trouvé la solution à mon problème. Comme je l'ai dit plus haut, la requete avec jointure n'a pas marché donc j'ai refait en deux requetes :
Code:

1 2 3 4 5 $quer=mysql_query("select id_avatar from internaute where id_internaute='".$element[$i]['id_internaute']."';"); $query=mysql_fetch_array($quer); $quer2=mysql_query("select url from avatar where id_avatar='".$query['id_avatar']."';"); $query2=mysql_fetch_array($quer2); unlink("./image/avatar/".$query2['url']);
j'ai mis une adresse en relatif au lieu d'absolu et ça marche nikel ( merci pour la remarque sur l'inneficacité du http://localhost/boby/image/avatar/ devant le nom du fichier ( dans la basse de donnée )
merci à tous

J'aurais plutôt ajoute if(fil_exists()) par sécurité. Sais-on jamais, si tu as une variable vide et que ça supprime tout le rep aii :cry:

Code:

1
2
3
4
5
6
7
8
9
10
$quer=mysql_query("select id_avatar from internaute where id_internaute='".$element[$i]['id_internaute']."';");
$query=mysql_fetch_array($quer);
$quer2=mysql_query("select url from avatar where id_avatar='".$query['id_avatar']."';");
$query2=mysql_fetch_array($quer2);
if (file_exists("./image/avatar/".$query2['url']) {
unlink("./image/avatar/".$query2['url'])
}
else {
//ton message d'erreur ou autre
}