comment supprimer un album photo dans une galerie

**VIRGINIE87** · 09/04/2010, 10h21

Coucou,
Je réalise une galerie photo, je suis en train de développer la gestion des albums photos, et tout particulièrement la suppression d'un album.

Ce que je souhaite faire est que lorsque je sélectionne un album, cela le supprime à la fois dans la base de donné et physiquement (le dossier correspondant à l'album + les images).
Je suis donc à la recherche de conseils dans ma démarche.
Merci

Invité · 09/04/2010, 10h36

Bonjour,
Bien sur c'est en PHP. Toutes tes images sont dans un ou plusieur répertoire,
de ce fait je préféres parcourir en premier ma base donnée dans un while {}
limité au nom de l'album.
Pour chaque image, je la supprime puisque j'ais son chemin/nom
puis je supprimes le data de ma base de donnée
,

**VIRGINIE87** · 09/04/2010, 11h28

toutes mes images sont dans le répertoire photos

Invité · 09/04/2010, 12h09

Envoyé par VIRGINIE87

toutes mes images sont dans le répertoire photos

Trés bien donc tu lis ta base de données, et si tu n'as que le nom de photos
tu concatenes $laphoto="photo/".$nomphoto;

**VIRGINIE87** · 10/04/2010, 09h12

voila ce que j'ai commencé à faire, supprimer l'enregistrement dans la base de donné mais cela ne fonctionnement et je n'ai aucun message d'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
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
 
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"
     "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html lang="fr" xml:lang="fr" xmlns="http://www.w3.org/1999/xhtml">
<head>
 
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
</head>
 
<body>
<?php
 $serveur='localhost';
 $user='root';
 $motdepasse=''; 
 $bdd='BDDPhotos';
 $connect= mysql_connect($serveur,$user,$motdepasse) or die ("Impossible de se connecter: "); 
 mysql_select_db($bdd);
 ?>
<?php 
 if (isset($_GET['supprimer_news'])) // Si on demande de supprimer une news
{
    // Alors on supprime la news correspondante
    // On protège la variable "id_news" pour éviter une faille SQL
    $_GET['supprimer_news'] = addslashes($_GET['supprimer_news']);
    mysql_query('DELETE FROM rubrique WHERE id=\'' . $_GET['supprimer_news'] . '\' WHERE RubriqueID='$RubriqueID'');
}
?>
 
  <table border="1"><tr>
<th>Id Album</th>
<th>Nom Album</th>
<th>Description</th>
<th>Date</th>
<th>Heure</th>
<th>Action</th>
</tr>
 
 
<?php
	$SQL = mysql_query("SELECT * FROM rubrique WHERE RubriqueID='RubriqueID'");
 
   while($donnees=mysql_fetch_array($SQL)) 
     { 
 
 
?>
<tr>
<td><?php echo $donnees['RubriqueID']  ?></td>
<td><?php echo $donnees['RubriqueName'] ?></td>
<td><?php echo $donnees['RubriqueDesc'] ?></td>
<td><?php echo $donnees['Date'] ?></td>
<td><?php echo $donnees['Time'] ?></td>
<td><?php echo' <a href="afficher contenu champ sous forme de tableaux.php?supprimer_news='.$donnees['RubriqueID'].'">';?> Supprimer</a></td>
</tr>
<p></p>
 
<?php
} // Fin de la boucle
?>
</table>      
 
</body>
</html>

que me conseillez vous ?

**RunCodePhp** · 10/04/2010, 09h27

Code :

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

mysql_query('DELETE FROM rubrique WHERE id=\'' . $_GET['supprimer_news'] . '\' WHERE RubriqueID='$RubriqueID'\');

Ca va pas du tout, tu as 2 fois WHERE ? Il en faut faut qu'un.
Puis tu exploite $RubriqueID, celle ci n'est pas connue, et c'est uniquement $_GET['supprimer_news'] qui contient l'ID de la rubrique.

Puis utiliser addslashes ça ne va pas nom plus, au mieux on utilise mysql_real_escape_string().
Cependant, si cette donnée DOIT être un identifiant (un nombre entier), se qui me semble le cas, "typer" cette donnée est bien plus radical, plus efficace à mon sens que n'importe quelle autre fonction.

Code :

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

1
2
 
mysql_query('DELETE FROM rubrique WHERE id = \'' . (int)$_GET['supprimer_news'].'\');

Un conseil en passant.
Vu que cette fonctionnalitée a pour but de supprimer, des données, il me semble plus raisonnable d'utiliser un formulaire en POST pour transmettre cette info, l'ID de la rubrique.
Aussi, une fois la suppression faite, il serait bon de rajouter une redirection (un header), même sur la même page, ceci purge les données (si on peu dire) en cas d'actualisation de la page.

**VIRGINIE87** · 12/04/2010, 09h11

merci pour ces conseils.
voila les rectifications :

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
 
 
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"
     "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html lang="fr" xml:lang="fr" xmlns="http://www.w3.org/1999/xhtml">
<head>
 
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
</head>
 
<body>
<?php
 $serveur='localhost';
 $user='root';
 $motdepasse=''; 
 $bdd='BDDPhotos';
 $connect= mysql_connect($serveur,$user,$motdepasse) or die ("Impossible de se connecter: "); 
 mysql_select_db($bdd);
 ?>
<?php 
 if (isset($_POST['supprimer_news'])) // Si on demande de supprimer une news
{
    // Alors on supprime la news correspondante
    // On protège la variable "id_news" pour éviter une faille SQL
    //$_POST['supprimer_news'] = addslashes($_POST['supprimer_news']);
    mysql_query('DELETE FROM rubrique WHERE id=\'' . $_POST['supprimer_news'] . '\');
    header('Location: afficher contenu champ sous forme de tableaux.php');
	}
?>
 
  <table border="1"><tr>
<th>Id Album</th>
<th>Nom Album</th>
<th>Description</th>
<th>Date</th>
<th>Heure</th>
<th>Action</th>
</tr>
 
 
<?php
	$SQL = mysql_query("SELECT * FROM rubrique WHERE RubriqueID='RubriqueID'");
 
   while($donnees=mysql_fetch_array($SQL)) 
     { 
 
 
?>
<tr>
<td><?php echo $donnees['RubriqueID']  ?></td>
<td><?php echo $donnees['RubriqueName'] ?></td>
<td><?php echo $donnees['RubriqueDesc'] ?></td>
<td><?php echo $donnees['Date'] ?></td>
<td><?php echo $donnees['Time'] ?></td>
<td><?php echo' <a href="afficher contenu champ sous forme de tableaux.php?supprimer_news='.$donnees['RubriqueID'].'">';?> Supprimer</a></td>
</tr>
<p></p>
 
<?php
} // Fin de la boucle
?>
</table>      
 
</body>
</html>

mais j'obtient l'erreur suivante :

Parse error: parse error in H:\Wamp\wamp\wamp\www\afficher contenu champ sous forme de tableaux.php on line 25

ligne 25

mysql_query('DELETE FROM rubrique WHERE id=\'' . $_POST['supprimer_news'] . '\');

**RunCodePhp** · 12/04/2010, 11h23

Cette donnée "supprimer_news" n'est pas transmise en POST, mais en GET, car tu conserve toujours ton lien :

Code :

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

1
2
 
<a href="afficher contenu champ sous forme de tableaux.php?supprimer_news='.$donnees['RubriqueID'].'">';?> Supprimer</a>

Faut savoir, (ou choisir) :
- Soit tu préfère le transmettre via un lien, donc GET
- Soit tu opte pour POST, et là il faut un formulaire : <form></form> et cette donnée dans un <input ... />, suivie d'un bouton pour soumettre le formulaire.

Je te conseillais d'utiliser POST (donc un formulaire) pour X raison, mais tu est libre de faire comme tu veux.
Faut juste utiliser les bonnes variables (ou tableaux) selon le cas

Quand tu ne sais pas trop ce qui est est transmit, ou reçu, etc ..., fait des echo sur certaines variables, ou un print_r ou autre var_dump() ...
Voir IN VISU les données ça aide à comprendre ce qui va, et ce qui ne va pas.

**VIRGINIE87** · 12/04/2010, 11h40

je ne sais pas vraimement pas comment faire.
Tu me conseille un POST, mais dans ce que je veu faire c'est sous forme de listing et pas un formulaire donc plutot un GET si j'ai bien comprit ton explication.

**RunCodePhp** · 12/04/2010, 15h39

mais dans ce que je veu faire c'est sous forme de listing et pas un formulaire donc plutot un GET

Ce n'est pas du tout un problème.
Dis toi que, si techniquement il est mieux de le faire en POST, ça sera toujours possible, mais vraiment toujours.
Faut pas bloquer sur un truc pareil

Entre un <a><a/> ou <form></form>, c'est pareil, non ?

Après, tout est une question de présentation.
Un bouton submit, voir la balise form provoque peut être une présentation particulière, mais ça reste lié à des spécificités du navigateur, de la norme W3C aussi.
Mais tu est totalement libre de faire le rendu que tu veux, c'est du pure CSS dont il s'agit, et rien que du CSS ici.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
<?php
	$i = 0;
	while($donnees=mysql_fetch_array($SQL)) {
?>
<td>
	<form name="sup_<?php echo $i; ?>" action="page.php" method="post">
		<input type="hidden" name="supprimer_news" value="<?php echo (int)$donnees['RubriqueID']; ?>" />
		<input type="submit" value="Supprimer" />
	</form>
</td>
<?php
	$i++;
?>

La donnée se trouve dans un champ caché (hidden).
Le seul truc un peu particulier ici, c'est que j'ai rajouté un compteur ($i) pour donner un nom unique au formulaire (question d'habitude).

**VIRGINIE87** · 12/04/2010, 19h28

voila ce que j'ai fait mais aucune suppression dans la base est visible :

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
 
 
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"
     "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html lang="fr" xml:lang="fr" xmlns="http://www.w3.org/1999/xhtml">
<head>
 
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
</head>
 
<body>
<?php
 $serveur='localhost';
 $user='root';
 $motdepasse=''; 
 $bdd='BDDPhotos';
 $connect= mysql_connect($serveur,$user,$motdepasse) or die ("Impossible de se connecter: "); 
 mysql_select_db($bdd);
 
	$SQL = mysql_query("SELECT * FROM rubrique");
 
 
 
?>
 
 
  <table  ><tr>
<th>Id Album</th>
<th>Nom Album</th>
<th>Description</th>
<th>Date</th>
<th>Heure</th>
<th>Action</th>
</tr>
<?php
   while($donnees=mysql_fetch_array($SQL)) 
     { 
?>
<tr>
<td><?php echo $donnees['RubriqueID']  ?></td>
<td><?php echo $donnees['RubriqueName'] ?></td>
<td><?php echo $donnees['RubriqueDesc'] ?></td>
<td><?php echo $donnees['Date'] ?></td>
<td><?php echo $donnees['Time'] ?></td>
<td><?php echo' <a href="afficher contenu champ sous forme de tableaux.php?supprimer_news='.$donnees['RubriqueID'].'">';?> Supprimer</a></td>
</tr>
<p></p>
 
</table>      
 
<?php
 
}
if (isset($RubriqueID)){
$RubriqueID = $_GET['RubriqueID'];
$delete_stage = 'DELETE FROM rubrique WHERE RubriqueID='.$RubriqueID; 
$result = mysql_query($delete_stage, $connect) or die(mysql_error());
}
?>
</body>
</html>

l'url est transmisse correctement mais pas de suppression. d'ou vient le probléme ?

**RunCodePhp** · 12/04/2010, 20h08

Code :

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

1
2
3
4
5
if (isset($RubriqueID)){
$RubriqueID = $_GET['RubriqueID'];
$delete_stage = 'DELETE FROM rubrique WHERE RubriqueID='.$RubriqueID; 
$result = mysql_query($delete_stage, $connect) or die(mysql_error());
}

$RubriqueID n'existe pas, c'est $_GET['RubriqueID']

Code :

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

1
2
3
4
5
6
 
if (isset($_GET['RubriqueID']) && is_numeric($_GET['RubriqueID']) && (int)$_GET['RubriqueID'] > 0){
	$RubriqueID = (int)$_GET['RubriqueID'];
	$delete_stage = 'DELETE FROM rubrique WHERE RubriqueID='.(int)$RubriqueID; 
	$result = mysql_query($delete_stage, $connect) or die(mysql_error());
}

Puis "type" un minimum tes données, de même de faire un minimum de vérification.
Un de ces 4 tu va te faire hacker, ça va faire mal ... ça arrive jamais au bon moment, car il y a jamais de bon moment.
Sérieux, au dire de certains sites, c'est des 10ènes, des 100ènes (même plus) de tentatives par jours.

Puis au niveau du déroulement, ça ne va pas non plus.
En 1er tu sélectionne toutes les rubriques, ensuite les affichent, et en dernier tu supprime celle sélectionnée.
Ca veux dire que tu affichera celle qui sera supprimée juste après.
Ca va en dérouter plus d'un.

Faut faire l'inverse.
En 1er supprimer la rubrique.
Après, récupérer toutes les rubriques.
Ici, on ne récupèrera et affichera que les rubriques existantes, celle supprimée n'y sera pas, vu quelle a été supprimée juste avant.
Vois tu

**VIRGINIE87** · 13/04/2010, 09h36

voila les modifications que j'ai apporté mais toujours aucune suppression dans ma base.

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
 
<head>
</head>
<body>
<?php
 $serveur='localhost';
 $user='root';
 $motdepasse=''; 
 $bdd='BDDPhotos';
 $connect= mysql_connect($serveur,$user,$motdepasse) or die ("Impossible de se connecter: "); 
 mysql_select_db($bdd);
 
 
 if (isset($_GET['RubriqueID']) && is_numeric($_GET['RubriqueID']) && (int)$_GET['RubriqueID'] > 0){	
	$RubriqueID = (int) $_GET['RubriqueID'];
	$delete_stage = 'DELETE FROM rubrique WHERE RubriqueID='.$RubriqueID; 
	$result = mysql_query($delete_stage, $connect) or die(mysql_error());
	header('Location: supprimera.php');  	
 
}
 
	$req = 'SELECT * FROM rubrique'; 
	$resultat = mysql_query($req, $connect) or die(mysql_error());
 
?>
 
    <table  ><tr>
	<th>Id Album</th>
	<th>Nom Album</th>
	<th>Description</th>
	<th>Date</th>
	<th>Heure</th>
	<th>Action</th>
	</tr>
<?php
 while ($donnee = mysql_fetch_array($resultat)) {
 ?>
	<tr>
	<td><?php echo $donnee['RubriqueID']  ?></td>
	<td><?php echo $donnee['RubriqueName'] ?></td>
	<td><?php echo $donnee['RubriqueDesc'] ?></td>
	<td><?php echo $donnee['Date'] ?></td>
	<td><?php echo $donnee['Time'] ?></td>
	<td><?php echo' <a href="supprimera.php?supprimer_news='.$donnee['RubriqueID'].'">';?> Supprimer</a></td>
	</tr>
<p></p>
 
</table>
 
<?php
 
 	}
?>
</body>

ai-je bien modifié le déroulement ?

**RunCodePhp** · 13/04/2010, 09h58

Arfff ... c'est pas RubriqueID mais supprimer_news

Mais vu qu'on est 2 à s'emmêler les pinceaux entre les noms, le plus simple serait d'en utiliser qu'un seul du début à la fin.
(Va falloir que j'aille voir Afflelou aussi ...
Parait il que pour 1 euros de plus on a 2 paires de lunettes pour le prix d'une.

Je pourrais t'en offrir une du coup

)

Comme RubriqueID est le nom du champ dans la Bdd, c'est plus simple d'utiliser celui ci.
De plus, c'est celui ci qui est maintenant exploité dans les traitements de suppression.
Reste juste à corriger ce nom au niveau du paramètre du lien :

<a href="supprimera.php?RubriqueID='.$donnee['RubriqueID'].'">';?> Supprimer</a>

**VIRGINIE87** · 13/04/2010, 10h20

Je veux bien que tu m'en offre une paire, des triples foyers devraient faire l'affaire

Bon nickel, la suppression de la rubrique se fait correctement dans la base.
Maintenant je dois supprimer les images contenuent dans l'album que l'on vient de supprimer.

comment supprimer un album photo dans une galerie

Langage PHP

Discussions similaires

Partager

Partager