Miniatures de gallerie d'image : second GET gênant ?

Marsh Posté le 27-02-2008 à 16:35:39

Bonjour,

Je travaille sur un code en php qui affiche une gallerie d'images à partir du contenu d'un dossier et qui génère des miniatures à la volée.
Voici ci-dessous la portion du code qui génère (ou devrait générer) les miniatures.
Le problème vient en fait du second GET ($_GET['id']), et je ne saisis pas pourquoi.
Une fois celui-ci enlevé (si je le remplace par une chaine de caractères adequat par exemple), mes miniatures s'affichent.
Quelqu'un aurait une explication à ça :??: il y a une règle que je n'ai pas respectée ?

Code :

<?php
$idgal = $_GET['id'];
if(!empty($_GET['img'])) {
$img = get_magic_quotes_gpc() ? stripslashes($_GET['img']) : $_GET['img'];
if(!function_exists('gd_info')) { die('Erreur : la librairie GD n\'est pas activée sur votre serveur.'); }
$tempext = strtolower(substr(strrchr($img, '.'),1));
if(!in_array($tempext, array('png', 'jpg', 'gif', 'jpeg')));
if($tempext == 'jpg' || $tempext == 'jpeg') {
$source = @imagecreatefromjpeg('./img/gallery/'.$idgal.'/'.$img);
} elseif($tempext == 'gif') {
$source = @imagecreatefromgif('./img/gallery/'.$idgal.'/'.$img);
} elseif($tempext == 'png') {
$source = @imagecreatefrompng('./img/gallery/'.$idgal.'/'.$img);
} else { die('Error !'); }
$infos = getimagesize('./img/gallery/'.$idgal.'/'.$img);
$largeur_original = $infos[0];
$hauteur_original = $infos[1];
if($infos[0] > 120 || $infos[1] > 120) {
$pl = $infos[0] / 120;
$ph = $infos[1] / 120;
if ($ph > 1 || $pl > 1) {
if ($ph > $pl) {
$largeur_new = $infos[0] / $ph;
$hauteur_new = $infos[1] / $ph;
} else {
$largeur_new = $infos[0] / $pl;
$hauteur_new = $infos[1] / $pl;
}
}
$img = @imagecreatetruecolor($largeur_new, $hauteur_new);
@imagecopyresized($img, $source, 0, 0, 0, 0, $largeur_new, $hauteur_new, $largeur_original, $hauteur_original);
@header('Content-type: image/jpeg');
@imagejpeg($img, '', 100);
} else {
@header('Content-type: '.$infos['mime']);
echo file_get_contents('./img/gallery/'.$idgal.'/'.$img);
}
} else {

Et voici la seconde moitié du script (le else du début est répété, donc évidemment a supprimer si vous le testez) :

Code :

} else {
$html_header = "<style type=\"text/css\">\r\na:link, a:visited, a:active {\r\ntext-decoration: none;\r\ncolor: #FFFFFF\r\n}\r\na:hover {\r\ntext-decoration: underline;\r\ncolor: #FFFFFF\r\n}\r\n</style>\r\n</head>\r\n<body text=\"#FFFFFF\">\r\n<div align=\"center\">\r\n<table width=\"100%\" border=\"0\">\r\n";
$html_footer = "</table></div>";
$main = opendir('./img/gallery/'.$idgal);
while(($n = readdir($main)) !== false) {
$tempext = strtolower(substr(strrchr($n, '.'),1));
if(is_file('./img/gallery/'.$idgal.'/'.$n) && in_array($tempext, array('png', 'jpg', 'gif', 'jpeg'))) {
$images[] = $n;
}
}
if(empty($images)) { die($html_header.'<tr><td align=\"center\">'.$idgal.'Dossier vide.</td></tr>\r\n').$html_footer; }
sort($images);
$afficher_nom = 0;
$images_p_ligne = 4;
echo $html_header;
$x = 0;
$enlever_extention = 1;
$remplacer_ = 0;
$afficherpoids = 0;
while(isset($images[$x])) {
echo "<tr>\r\n";
for($i=1; $i <= $images_p_ligne; $i++) {
if(!empty($images[$x])) {
if($afficher_nom = 1) {
$nom = $images[$x];
if($enlever_extention = 1) { $nom = ereg_replace(substr(strrchr($images[$x], '.'), 0), '', $images[$x]); }
if($remplacer_ = 1) { $nom = ereg_replace('_', ' ', $nom); }
$nom = " ".$nom."";
if($afficherpoids = 1) { $size = " (".round(filesize('./img/gallery/'.$idgal.'/'.$images[$x]) / 1024, 2)." Ko)"; }
}
echo '<td width="'.intval(100 / $images_p_ligne).'%" align="center"><a href="./img/gallery/'.$idgal.'/'.$images[$x].'" id="mb1" class="mb" title="le titre"><img src="gallery.php?img='.urlencode($images[$x]).'" alt="IMG" border="0" style="filter:alpha(opacity=50); -moz-opacity: .5;" onMouseover="makevisible(this,0)" onMouseout="makevisible(this,1)" />'.$nom.'</a>'.$size.' </td>'."\r\n";
$x++;
}
}
echo "</tr>\r\n";
}
echo $html_footer;
}
?>

edit : ajout du script complet

Message édité par san_ le 28-02-2008 à 11:45:51

Reply

Marsh Posté le 27-02-2008 à 16:35:39

Reply

Marsh Posté le 27-02-2008 à 16:48:03

Es-tu sût que ton $_GET['id] a bien une valeur ?

---------------
Feedback : http://forum.hardware.fr/hfr/Achat [...] 2666_1.htm

Reply

Marsh Posté le 27-02-2008 à 17:40:24

Oui, mais même si le $_GET['id'] n'est affecté à aucune variable, (si je met $_GET['id']; tout seul, bien qu'évidemment ça ne servirai à rien), les miniatures ne sont pas générées.
C'est la présence d'un second GET qui pose problème, et je ne sais pas pourquoi.

Message cité 1 fois

Reply

Marsh Posté le 27-02-2008 à 17:42:57

san_ a écrit :

Oui, mais même si le $_GET['id'] n'est affecté à aucune variable, (si je met $_GET['id']; tout seul, bien qu'évidemment ça ne servirai à rien), les miniatures ne sont pas générées.
C'est la présence d'un second GET qui pose problème, et je ne sais pas pourquoi.

Que donne un print_r($_GET) ?

Message cité 1 fois

---------------
Feedback : http://forum.hardware.fr/hfr/Achat [...] 2666_1.htm

Reply

Marsh Posté le 27-02-2008 à 18:11:09

babasss a écrit :

Que donne un print_r($_GET) ?

Array ( [p] => gallery )

Message cité 1 fois

Reply

Marsh Posté le 27-02-2008 à 18:14:35

Tu l'as placé où ? Juste avant if(!empty($_GET['img'])) ?

Message cité 1 fois
Message édité par babasss le 27-02-2008 à 18:14:43

---------------
Feedback : http://forum.hardware.fr/hfr/Achat [...] 2666_1.htm

Reply

Marsh Posté le 27-02-2008 à 18:20:42

babasss a écrit :

Tu l'as placé où ? Juste avant if(!empty($_GET['img'])) ?

Non, tout à la fin de mon script. Si je le met avant le if(!empty($_GET['img'])), la même chose s'affiche, seulement mes miniatures ne s'affichent plus. A cause de la présence du print_r($_GET)... étrange n'est ce pas [:gratgrat]

Reply

Marsh Posté le 27-02-2008 à 18:46:40

Si quand tu le mets au début de ton script, il n'affiche que ce que "Array ( [p] => gallery )". cela veut dire que uniquement $_GET['gallery'] est défini et pas les autres....
Es-tu sur que dans l'URL, tu as id= et img= ?

---------------
Feedback : http://forum.hardware.fr/hfr/Achat [...] 2666_1.htm

Reply

Marsh Posté le 27-02-2008 à 19:44:50

$_GET['p'] tu veux sans doute dire.
"p" c'est la variable que j'utilise pour charger le script gallery.php de la gallerie (index.php?p=gallery).

Je veux faire fonctionner la variable $_GET['id'] pour choisir d'afficher le contenu d'un des répertoires placés dans "./img/gallery/". Par exemple si je charge "index.php?p=gallery&id=eurometal" je veux qu'il m'affiche le contenu de "./img/gallery/eurometal".

La variable "img", je n'ai pas à la préciser moi-même dans l'url, elle est générée pour chaque miniature (<img src="gallery.php?img=DSC01326.jpg" ...). Je sais pas si je suis clair :whistle: .

Message édité par san_ le 27-02-2008 à 19:45:18

Reply

Marsh Posté le 27-02-2008 à 23:30:51

et donc à chaque visionnage de ta galerie, les miniatures sont générées ?

Tu ferais pas mieux de les générer à l'import de tes images, ça serait fait une fois pour toutes

Message cité 1 fois

Reply

Marsh Posté le 27-02-2008 à 23:30:51

Reply

Marsh Posté le 28-02-2008 à 00:43:12

bricocoman a écrit :

et donc à chaque visionnage de ta galerie, les miniatures sont générées ?

Tu ferais pas mieux de les générer à l'import de tes images, ça serait fait une fois pour toutes

Faire en sorte que le script génère physiquement les images miniatures ? je l'ai testé avec un autre script, c'est plus rapide en effet mais je préfère quand même qu'elles soient générées à la volée, question de souplesse pour les mises à jour. Si je remplace une image originale par une autre, sa miniature, elle, ne sera pas mise à jour automatiquement.

Sinon pour mon problème, personne n'a une idée :??: je devrais peut être poster la seconde moitié du script ?

Reply

Marsh Posté le 28-02-2008 à 08:55:45

tu dois avoir une notice ou autre dans ton script qui fait foirer l'affichage, erreur fréquente.
donc un peu de debug ne ferait pas de mal, enlève le header et appelle ton script directement, tu verras les messages d'erreur

Message cité 1 fois

Reply

Marsh Posté le 28-02-2008 à 11:34:40

soju a écrit :

tu dois avoir une notice ou autre dans ton script qui fait foirer l'affichage, erreur fréquente.
donc un peu de debug ne ferait pas de mal, enlève le header et appelle ton script directement, tu verras les messages d'erreur

+1
Ce qui est clair c'est que l'URL avec lequel tu appelles ce script ne contient que la varaible $_GET['p'] et pas les autres...
Donc soit l'URL d'appel doit être mauvais... Peux-tu nous le montrer ?

---------------
Feedback : http://forum.hardware.fr/hfr/Achat [...] 2666_1.htm

Reply

Marsh Posté le 28-02-2008 à 11:48:37

c'est celle-ci http://localhost/?p=gallery&id=eurometal

J'ai ajouté la seconde moitié du code. Voyez ma variable $idgal = $_GET['id']; que j'ai déplacé tout au début. Si vous la remplacez par la chaîne de caractère qui va bien ($idgal = 'eurometal'; dans mon cas), tout fonctionne. Avec un GET, c'est pas le cas [:pingouino] .

Reply

Marsh Posté le 28-02-2008 à 11:54:39

Pour passer de http://localhost/?p=gallery&id=eurometal à ton script, tu utilises quoi : un Rewrite ?
Ce serait pas ca qui te bouffe une partie de ton URL ?

Message cité 1 fois

---------------
Feedback : http://forum.hardware.fr/hfr/Achat [...] 2666_1.htm

Reply

Marsh Posté le 28-02-2008 à 12:01:18

babasss a écrit :

Pour passer de http://localhost/?p=gallery&id=eurometal à ton script, tu utilises quoi : un Rewrite ?
Ce serait pas ca qui te bouffe une partie de ton URL ?

J'utilise ça dans mon index.php :

Code :

<?php
$page = $_GET['p'];
if ($page != '') {
include ($page. '.php');
}
?>

(c'est bien ça que tu me demandais ?)

Message édité par san_ le 28-02-2008 à 12:02:04

Reply

Marsh Posté le 28-02-2008 à 12:31:06

Yes...
Essaye de récupérer les $_GET avant de faire le include

Code :

<?php
 $page = $_GET['p'];
 $idgal = $_GET['idgal'];
 $img= $_GET['img'];
 if ($page != '') {
  include ($page. '.php');
 }
?>

Et de l'autre coté, tu appelles les varaibles $idgal et $img

---------------
Feedback : http://forum.hardware.fr/hfr/Achat [...] 2666_1.htm

Reply

Marsh Posté le 28-02-2008 à 13:03:01

j'ai essayé de mettre $idgal = $_GET['id']; dans mon index mais le résultat est le même.
Quant à la variable $img, elle ne contient pas uniquement le GET img mais
$img = get_magic_quotes_gpc() ? stripslashes($_GET['img']) : $_GET['img'];, et ce uniquement si la condition if(!empty($_GET['img'])) est respectée, donc je ne peux pas la bouger.
Et je n'ai pas à mettre de "?img=" dans l'url ce ?img corresepond uniquement à l'url générée des miniatures comme je l'ai précisé en haut

Message édité par san_ le 28-02-2008 à 13:05:04

Reply

Marsh Posté le 28-02-2008 à 14:16:26

et si tu essayes ce lien : http://localhost/gallery.php?id=eurometal , il se passe quoi ?
Peux-tu nous donner aussi tout le fichier index.php ?

Message cité 1 fois

---------------
Feedback : http://forum.hardware.fr/hfr/Achat [...] 2666_1.htm

Reply

Marsh Posté le 28-02-2008 à 14:40:33

babasss a écrit :

et si tu essayes ce lien : http://localhost/gallery.php?id=eurometal , il se passe quoi ?
Peux-tu nous donner aussi tout le fichier index.php ?

page blanche avec le get, et si je le remplace par la chaine "eurometal", j'ai ma gallerie sur fond blanc, c'est toujours le même problème.

Le reste de l'index n'est pas très interessant, c'est presque que du html pour l'instant.
Mais si tu es motivé a comprendre voila le site "complet" (j'en suis qu'au tout début) : http://sweedigel.free.fr/tmp/site.7z

Message cité 2 fois

Reply

Marsh Posté le 28-02-2008 à 14:43:32

san_ a écrit :

page blanche avec le get, et si je le remplace par la chaine "eurometal", j'ai ma gallerie sur fond blanc, c'est toujours le même problème.

Le reste de l'index n'est pas très interessant, c'est presque que du html pour l'instant.
Mais si tu es motivé a comprendre voila le site "complet" (j'en suis qu'au tout début) : http://sweedigel.free.fr/tmp/site.7z

Je regarde ca ce soir.... Ton pb me turlupine sacrément...

---------------
Feedback : http://forum.hardware.fr/hfr/Achat [...] 2666_1.htm

Reply

Marsh Posté le 28-02-2008 à 17:35:27

san_ a écrit :

page blanche avec le get, et si je le remplace par la chaine "eurometal", j'ai ma gallerie sur fond blanc, c'est toujours le même problème.

Le reste de l'index n'est pas très interessant, c'est presque que du html pour l'instant.
Mais si tu es motivé a comprendre voila le site "complet" (j'en suis qu'au tout début) : http://sweedigel.free.fr/tmp/site.7z

J'ai téléchargé tes sources pour faire l'essai.

--> page blanche... moi aussi, c'est ce que j'ai "en apparence", mais si tu affiche la source, c'est pas vraiment une page blanche ^^

Code :

<style type="text/css">
a:link, a:visited, a:active {
text-decoration: none;
color: #FFFFFF
}
a:hover {
text-decoration: underline;
color: #FFFFFF
}
</style>
</head>
<body text="#FFFFFF">
<div align="center">
<table width="100%" border="0">
<tr>
<td width="25%" align="center"><a href="./img/gallery/eurometal/._DSC01326.jpg" id="mb1" class="mb" title="le titre"><img src="gallery.php?img=._DSC01326.jpg" alt="IMG" border="0" style="filter:alpha(opacity=50); -moz-opacity: .5;" onMouseover="makevisible(this,0)" onMouseout="makevisible(this,1)" /> . DSC01326</a> (51.23 Ko) </td>
<td width="25%" align="center"><a href="./img/gallery/eurometal/._DSC01327.jpg" id="mb1" class="mb" title="le titre"><img src="gallery.php?img=._DSC01327.jpg" alt="IMG" border="0" style="filter:alpha(opacity=50); -moz-opacity: .5;" onMouseover="makevisible(this,0)" onMouseout="makevisible(this,1)" /> . DSC01327</a> (63.66 Ko) </td>
<td width="25%" align="center"><a href="./img/gallery/eurometal/._DSC01332.jpg" id="mb1" class="mb" title="le titre"><img src="gallery.php?img=._DSC01332.jpg" alt="IMG" border="0" style="filter:alpha(opacity=50); -moz-opacity: .5;" onMouseover="makevisible(this,0)" onMouseout="makevisible(this,1)" /> . DSC01332</a> (55.01 Ko) </td>
<td width="25%" align="center"><a href="./img/gallery/eurometal/._DSC01334.jpg" id="mb1" class="mb" title="le titre"><img src="gallery.php?img=._DSC01334.jpg" alt="IMG" border="0" style="filter:alpha(opacity=50); -moz-opacity: .5;" onMouseover="makevisible(this,0)" onMouseout="makevisible(this,1)" /> . DSC01334</a> (53.74 Ko) </td>
</tr>
<tr>
<td width="25%" align="center"><a href="./img/gallery/eurometal/._DSC01337.jpg" id="mb1" class="mb" title="le titre"><img src="gallery.php?img=._DSC01337.jpg" alt="IMG" border="0" style="filter:alpha(opacity=50); -moz-opacity: .5;" onMouseover="makevisible(this,0)" onMouseout="makevisible(this,1)" /> . DSC01337</a> (58.61 Ko) </td>
<td width="25%" align="center"><a href="./img/gallery/eurometal/._DSC01341.jpg" id="mb1" class="mb" title="le titre"><img src="gallery.php?img=._DSC01341.jpg" alt="IMG" border="0" style="filter:alpha(opacity=50); -moz-opacity: .5;" onMouseover="makevisible(this,0)" onMouseout="makevisible(this,1)" /> . DSC01341</a> (62.3 Ko) </td>
<td width="25%" align="center"><a href="./img/gallery/eurometal/DSC01326.jpg" id="mb1" class="mb" title="le titre"><img src="gallery.php?img=DSC01326.jpg" alt="IMG" border="0" style="filter:alpha(opacity=50); -moz-opacity: .5;" onMouseover="makevisible(this,0)" onMouseout="makevisible(this,1)" /> DSC01326</a> (81.21 Ko) </td>
<td width="25%" align="center"><a href="./img/gallery/eurometal/DSC01327.jpg" id="mb1" class="mb" title="le titre"><img src="gallery.php?img=DSC01327.jpg" alt="IMG" border="0" style="filter:alpha(opacity=50); -moz-opacity: .5;" onMouseover="makevisible(this,0)" onMouseout="makevisible(this,1)" /> DSC01327</a> (167.53 Ko) </td>
</tr>
<tr>
<td width="25%" align="center"><a href="./img/gallery/eurometal/DSC01332.jpg" id="mb1" class="mb" title="le titre"><img src="gallery.php?img=DSC01332.jpg" alt="IMG" border="0" style="filter:alpha(opacity=50); -moz-opacity: .5;" onMouseover="makevisible(this,0)" onMouseout="makevisible(this,1)" /> DSC01332</a> (86.86 Ko) </td>
<td width="25%" align="center"><a href="./img/gallery/eurometal/DSC01334.jpg" id="mb1" class="mb" title="le titre"><img src="gallery.php?img=DSC01334.jpg" alt="IMG" border="0" style="filter:alpha(opacity=50); -moz-opacity: .5;" onMouseover="makevisible(this,0)" onMouseout="makevisible(this,1)" /> DSC01334</a> (104.57 Ko) </td>
<td width="25%" align="center"><a href="./img/gallery/eurometal/DSC01337.jpg" id="mb1" class="mb" title="le titre"><img src="gallery.php?img=DSC01337.jpg" alt="IMG" border="0" style="filter:alpha(opacity=50); -moz-opacity: .5;" onMouseover="makevisible(this,0)" onMouseout="makevisible(this,1)" /> DSC01337</a> (116.58 Ko) </td>
<td width="25%" align="center"><a href="./img/gallery/eurometal/DSC01341.jpg" id="mb1" class="mb" title="le titre"><img src="gallery.php?img=DSC01341.jpg" alt="IMG" border="0" style="filter:alpha(opacity=50); -moz-opacity: .5;" onMouseover="makevisible(this,0)" onMouseout="makevisible(this,1)" /> DSC01341</a> (146.11 Ko) </td>
</tr>
</table></div>
<script type="text/javascript">
var box = {};
window.addEvent('domready', function() {
box = new MultiBox('mb', {descClassName: 'multiBoxDesc'});
});
</script>

c'est plutot du "blanc sur blanc", autrement appelé le "monochrome de whiteman"

---------------
- Xav - ...There are no crimes when there are no laws... -- Xav's World

Reply

Marsh Posté le 28-02-2008 à 17:39:15

Oui j'avais vu ça. Et d'ailleurs le plus troublant c'est qu'on s'aperçoit bien que mon $_GET['id'] est bel et bien pris en compte. Seulement... la miniature ne s'affiche pas. A n'y rien comprendre

Reply

Marsh Posté le 28-02-2008 à 20:48:19

Héhé, ca marche chez moi.....

Deux problèmes :

Je ne sais pas pourquoi mais cet abruti me renvoyait le nom des images avec "_" devant => rectification dans la boucle en dessous de "while(($n = readdir($main)) !== false)", j'ai rajouté

Code :

$n = substr($n,2,strlen($n);

Là on n'a toujours pas les miniatures mais quand on clique dessus, on obtient les agrandissement.... Mais pourquoi donc

Donc le pb vient de la miniature, je chope l'url de img src et quand tu colles l'url dans le navigateur (par ex : gallery.php/?img=DSC01337.jpg), il te crache un flot d'insulte parce qu'il ne connait pas $idgal et c'est normal, tu ne l'as pas passé dans l'url, d'où la correction : src="gallery.php?id='.$idgal.'&img='.urlencode($images[$x]).'" alt="IMG">

Et maintenant tout roule

Message édité par babasss le 28-02-2008 à 20:49:21

---------------
Feedback : http://forum.hardware.fr/hfr/Achat [...] 2666_1.htm

Reply

Marsh Posté le 29-02-2008 à 01:05:24

[:diskobeck] génial ça fonctionne. Par contre je n'ai pas le problème de l'underscore [:gratgrat] . En effet c'est plutôt logique, mais il y a quelquechose que je ne m'eplique pas : pourquoi le fait de mettre un $_GET au début, quel qu'il soit, empêchait l'affichage des miniatures ? Même lorsqu'on avait déclaré $idgal = 'eurometal' par exemple.

Un grand, grand merci en tout cas :jap: .

Reply

Marsh Posté le 29-02-2008 à 09:12:10

Si tu avais testé la page gallery.php/?img=DSC01337.jpg, tu connaitrais la réponse
En fait, tu appellais la page gallery.php avec comme variable img=DSC01337.jpg et tu ne définissais pas idgal or dans la première moitié de ton script tu en as besoin pour localiser l'image (ligne 11, 13 15 et 17)

---------------
Feedback : http://forum.hardware.fr/hfr/Achat [...] 2666_1.htm

Reply

Marsh Posté le

Reply

Miniatures de gallerie d'image : second GET gênant ?

Sujets relatifs:

Leave a Replay