PHP

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

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Miniatures de gallerie d'image : second GET gênant ?

san_

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

Publicité

babasss

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

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

san_

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

babasss

san_ a écrit :

Que donne un print_r($_GET) ?

Message cité 1 fois

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

san_

babasss a écrit :

Que donne un print_r($_GET) ?

Array ( [p] => gallery )

Message cité 1 fois

babasss

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

san_

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]

babasss

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

san_

$_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

bricocoman

j'aime les modos

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

Publicité

san_

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 ?

soju

One shot !

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

babasss

soju a écrit :

+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

san_

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] .

babasss

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

san_

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

babasss

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

san_

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

babasss

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

san_

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

babasss

san_ a écrit :

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

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

Xav_

The only one...

san_ a écrit :

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

san_

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

babasss

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

san_

[: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: .

babasss

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

Publicité

FORUM HardWare.fr

Programmation

PHP

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

Sujets relatifs
File get contents / Image	[RESOLU] Passer un scipt au second plan
JS : Image preloading : pas de gain de performances?	[Résolu de par moi-même][PHP] Récupérer l'url d'une image uploadée
phpMyAdmin + blob image + export Excel	popup image problemes avec iexplorer
Modifier mon scrip pour mon image	[Python] Conso CPU/Chargement Image
Problème d'alignement d'image avec CSS	création de miniatures pour vidéos WMV sur serveur
Plus de sujets relatifs à : Miniatures de gallerie d'image : second GET gênant ?

Page générée en 0.196 secondes