GALERIE PHOTO - 'PHENHP' AMÉLIORÉ + LOOK 'ZENPHOTO'

Commentaire de dth_2003 le 10/09/2006 11:56:52

Bonjour Zefo, je vais te poser la même question qu'a Yoadsl. Faire le système de tel sorte que l'on puisse spécifier le nom du répertoire ou se trouverons les photos. En d'autre termes, mes répertoires de photos sont toutes au même endroits mais pas avec ton index.php !!!

Merci

Commentaire de alxou le 10/09/2006 12:04:47

Bonjour
très bon travail, mais j'ai une question : pourquoi les photos miniature n'apparaissent pas sur l'ecran?
Si tu peux me répondre c'est très gentil de ta part
Merci 9.5/10

Commentaire de Palleas_44 le 11/09/2006 10:37:05

Pas mal mais le HTML est pas beau ^^
Verification w3c à l'arache : presque une centaine d'erreur :)
si jamais c'est modifié je noterai je pense mais pour l'instant la source me pique les yeux :p

bon courage =)

Commentaire de younes371 le 12/09/2006 02:25:59

j ai enregistré  index.php & style.css ds un repertoire, et puis j ai ajouté un sous repertoir s'appel 'image'  qui contient les photos,
Mais je vois rien sur la page sauf  Ma galerie et les liens next et prev desactivé.
Pouvez vous m aider?
Merci

Commentaire de cacoucatatonique le 12/09/2006 09:04:56

rebonjour, nouvelle petite question
comment generer le fichier suivant:
$fichier_parametres_gallerie = '_infos_gallerie.php';  // Fichier contentant les Informations sur chaque Gallerie (présent dans les Répertoires correspondant aux Galleries).
????

Commentaire de cacoucatatonique le 12/09/2006 10:24:26

je suis dans le meme cas que toi avec la meme avec la meme config
et snif moi non plus ca ne fonctionne pas.

Commentaire de kankrelune le 12/09/2006 12:55:05

@ grosminou_55... installe la lib GD

@ younes371... "Merci de bine repondre dans un dèlai court" ...on aura tout vu (ou plutot lu)... bientot faudra répondre juste, rapide et par téléphone... muarf... .. .

@ tchaOo°

Commentaire de younes371 le 12/09/2006 14:23:05

j ai demandé la reponse ds un dèlai court, car si ça retarde je vais oublier ce code ...lol

Commentaire de dth_2003 le 12/09/2006 19:47:50

Salut, et encore merci pour la petite modif que j'avais demandé.

Je voulais juste te signaler qu'il manque au moins une fermeture de div (cela donnais bizarre dans mon design a cause de cela) lorsque que l'on visionne une des galeries.

J'ai d'alleurs un autre souci pour le moment c'est que le reste de ma page ne s'affiche plus au moment ou je visionne une des images elle s'arrete juste après les détails de la photo, puis le reste qui devrait s'afficher ne vient pas ... je cherche ... et je trouverais c'est sur.

Une dernière petite chose, pour la beauté du geste disons. Tu utilise les css, c'est génial, mais malgré cela on trouve plein de balise style <center>, etc.. c'est dommage (sniff, sniff). Et puis tu code ta page en xhtml 1.0 strict et on retrouve plein de balise en majuscule, pas bien cela en strict si je ne m'abuse.

Très bon boulot. Merci encore 8/10

Commentaire de tombeur62 le 13/09/2006 03:50:32

superbe travail pour ma part fonctionne tres bien voici mon lien pour voir

http://escalles.new.fr/GALERIEPHOTO

et merci encore
a+

Commentaire de zefo le 13/09/2006 12:33:02

Bonjour à tous,

Je viens de faire une modif qui devrait répondre à pas mal de vos questions (repertoire photo, librairie GD). Ajout d'un fichier txt dans le zip. code validé w3c (plus de problème avec les balises). Correction d'un bug sur les galeries protégées par mot de passe suite à l'ajout de l'option pour modifier le répertoire photos par défaut.

Commentaire de younes371 le 13/09/2006 13:35:29

Salut,
de ma part j ai un probleme, dans le local ça marche tres bien l ajout des commentaires, mais dans le web ça marche plus, je me suis que le probleme esr reside dans le fait de "CHMOD" du repertoir photo, j ai fs 777(donc read,write,modif).mais ça fait rien => le meme probleme
Merci

Commentaire de younes371 le 13/09/2006 14:45:00

tombeur62 j ai visité ton stiteweb, ça marche bien.
dis moi, comment t as fait pour les commentaires, chez moi ça marche pas !!. http://younes-bouamout.orgfree.com

Commentaire de zefo le 13/09/2006 18:24:17

dth_2003 : lorsqu'une image est plus petite que la taille des miniatures prévues par défaut, s'agit-t-il réellement d'une photo ? As-t-elle la place dans une galerie ?
J'ai modifié aussi la fonction Menu_Navigation, je ferai une mise à jour du zip quand il y aura plus de modif.
younes371 : J'ai vu le problème sur ton site, apparemment le script n'est pas executé jusqu'à la fin, le problème vient peut-être de ton hébergeur. à voir....
@+

Commentaire de zefo le 14/09/2006 02:58:09

- Petite Astuce : vous pouvez commenter toutes les photos de votre site, il suffit d'éditer les propriétés IPTC de la photo grâce à un logiciel graphique (Irfanview le fait très bien). Ainsi les infos ne seront jamais perdues, même après copie, déplacement, réinstallation....
- pour les emoticones, je vois que ce n'est pas 'très' génant, mais je regarderai un peu le boulot que ça représente....
- Je suis en train de finir la page administration.
@+

Commentaire de johann1 le 16/09/2006 14:36:56

Juste un petit compliment ! SUPER ! Merci !

Commentaire de grosminou_55 le 17/09/2006 06:32:29

je suis d'accord avec Batistoss59  il serai super de pouvoir , dans une galerie voir des sous galeries ceci faciliterai le clacement par famille (ma soeur   l'autre soeur   leur enfant   ect )
romain

Commentaire de grosminou_55 le 17/09/2006 19:53:44

ok  zefo mais j'y connais pas grand chose alors j'aimerai pas defigurer ( voir detruire ) tou ton travail   lol

Commentaire de kankrelune le 18/09/2006 13:50:34

Bon j'ais jetté un coup d'oeil et j'avou que j'ais été surppris et malheureusement pas en bien... beaucoups beaucoups d'erreurs... .. .

-1- pas de vérification de l'éxistance des répertoires et fichiers avant leur ouverture/lecture/modification/ecriture... générant de nombreux warning et étant surment la cause des arrèt prématurés d'execution... .. .

-2- codé avec register globals à on empechant certaines actions de s'executer (tel que l'édition des gualleries dans l'admin) et si on met le error-reporting à E_ALL on pourra noter pas mal de notices sur des variables non initialisées

-3- en relation avec le point 2 un affreux

while (list($var,$val) = @each($HTTP_GET_VARS)) { ${$var} = $val; }
while (list($var,$val) = @each($HTTP_POST_VARS)) { ${$var} = $val; }

dans la page d'admin... d'une part c'est crade d'autre part c'est une faille de sécurité potentiel... .. .

-4- des includes,mkdir,fopen basés (en partie) sur des $_GET non (ou peu) controlés... c'est la porte ouverte à des includes sauvage, des créations, suppressions de fichiers,répertoire incontrolés... de plus tu fais passer le chemin "complet" de la gallerie dans l'url alors que seul son nom suffit

ex :

//on définit le chemin du script pour tous les includes,fopen et companie
define("MAIN_PATH", dirname(__FILE__).DIRECTORY_SEPARATOR);

// on définit le path des galleries en fonction de l'entrée utilisateur
if(is_dir($Rep_Photos))
{
   // on vérifie que le dernier caractère du chemin est un séparateur
   if(substr($Rep_Photos,-1) != DIRECTORY_SEPARATOR)
     $Rep_Photos .= DIRECTORY_SEPARATOR;

   define("GALERIES_PATH",$Rep_Photos);
}
else if(is_dir(MAIN_PATH.$Rep_Photos))
{
   // on vérifie que le dernier caractère du chemin est un séparateur
   if(substr($Rep_Photos,-1) != DIRECTORY_SEPARATOR)
     $Rep_Photos .= DIRECTORY_SEPARATOR;

   define("GALERIES_PATH",MAIN_PATH.$Rep_Photos);
}
else define("GALERIES_PATH",MAIN_PATH.'Photos'.DIRECTORY_SEPARATOR);

ensuite une petite fonction

function cleanPath($path)
{
  return str_replace(array('./','../','.\\','..\\'),'',$path);
}

Au final tu n'as plus qu'a faire... si je prend l'exemple de ta fonction trier_images... .. .

function trier_images()
{
    $rep = cleanPath($_GET['rep']); // on nettoie
    
    if(is_dir(GALERIES_PATH.$rep)) // on vérifie que le répertoire existe sinon on renvoie false
    {
if (!$dir = @opendir(GALERIES_PATH.$rep)) // on l'ouvre et si ça rate on renvoie false
return false;
else
{
$image = array(); // on initialise toujours un tableau avant de le remplir

while(false !== ($f = readdir($dir))) // on fait une comparaison typé pour la lecture
{
$ext_file = strtolower(array_pop(explode(".", $f)));
if ($ext_file == "jpg" || $ext_file == "gif" || $ext_file == "png" || $ext_file == "wbmp")
$image[] = $f;
}

closedir($dir); // ON OUBLIE PAS DE FERMER LE POINTEUR DE LECTURE DU REPERTOIRE
natcasesort($image);

return array_values($image);
}
}
return false;
}

-5- dans la ligné de -4- virer


// Security by phpSecure.info
if ($_GET["rep"]){ $_GET["rep"]="./".$_GET["rep"]; }
//End

qui ne t'offre qu'une sécurité illusoire et qui est incompatible avec des path de guallerie externe au path du script... .. .

-6- problème dans le parsing/écriture des tes fichiers _infos_galerie.php... d'une part il ne sont pas tout le temps vérifié/créé (par exemple dans l'admin)... d'autre part après quelque visites/tentative de modification voiici ce que j'obtiens

<?
$nbp = "14"; // Nombre de Photos.
?><?
$nbp = "14"; // Nombre de Photos.
?><?
$nbp = "14"; // Nombre de Photos.
?>

Déja c'est pô beau ensuite <? ne fonctionne pas chez moi... il faut mettre <?php pour finir mais c'est un détail plutot que de définir X variables mieux vaut faire un tableau $galerieInfos... tu l'écrit dans le fichier dès la première visite (coté visiteurs ou coté admin) avec les valeur non définit mise par défaut et quand tu veux réécrire le fichier il te suffit de faire...

// on l'inclu pour récupérer son contenu ou on le crée
if(is_file(MAIN_PATH.$rep.'_infos_galerie.php'))
    include(MAIN_PATH.$rep.'_infos_galerie.php');
else
{
    $galerieInfos = array(
                'Description' => 'Ecrivez ici la Description de votre Galerie.',
                'nbp' => 0,
                // toutes les infos avec leur valeur par défaut
              );  
}

/*
* On modifie s'il y a lieux le contenu de $galerieInfos
*/

// on ouvre le fichier en  virant tout son contenu
if($handle = @fopen(MAIN_PATH.$rep.'_infos_galerie.php',"w"))
{
   if(!@fwrite($handle,'<?php'."\n\n".'$galerieInfos = '.var_export($galerieInfos,true).";\n\n".'?>'))
      // erreur
  
   fclose($handle);
}
else // erreur

C'est plus sur au moin tu est sur que les variables sont initialisés (plutot que de les tester à tout va)... et puis le tableau permet une simplification et une meilleur lecture du code (mais d'où vient donc cette £$%ùµ*§ de variable $nbp)... .. .

-7- sinon pour tester l'existance d'une variable on fait...

if(isset($mavar)) pour voir si elle existe
if(empty($mavar)) pour voir si elle est vide ( '', 0, null ou false)

et

if($mavar) ou if($mavar === true) pour voir si $mavar est égale à true

$mavar = 'pwet';
if($mavar) echo $mavar;

fonctionne mais c'est un abus de langage et une mauvaise habitude...

$mavar = 'pwet';
if(!empty($mavar)) echo $mavar;

-8- tu utilise indifférament les simples et doubles quote... préfère les simple quote et concatène tes variables.. tu gagnera en clareté et (très légèrement) en perf

echo 'Bonjour '.$name.' nous sommes le '.$date.' il fait beau les oiseaux chantent';

-9- je serais toi j'externaliserais le déclaration de mes fonctions dans un fichier functions.php... l'interet ? clarifier le code, réutiliser les fonctions dans plusieurs fichiers (si ton projet grossi et contient plusieurs fichiers)

-10- ne passe surtout pas les mot de pass dans l'url... c'est bof bof en plus de ne pas être très sécurisé (imaginons que le pc soit commun à plusieurs personnes) utilise les sessions... .. .

Voila en gros... il y a encore plein de choses à dire mais dans un premier temps je pense que c'est suffisant... cependant j'ajouterais que je ne dis pas tout ça pour descendre bètement ton code... loin de là... l'idée est somme toute basique mais bonne, les bases sont là... si je dis ça c'est juste pour souligner les erreurs pour que tu puisse modifier ta source qui n'en sera qu'améliorée... .. .

@ tchaOo°

Commentaire de maxouplus le 18/09/2006 14:56:44

Sans chercher des complications, c'est tout simplement très bien : le PHP fonctionne, la mise en place est très simple, le look est très agréable. C'est 10 sur 10.

Commentaire de johann1 le 18/09/2006 18:02:00

Je ne sais pas dans quelle intention younes371 demande les sources de kankrelune, mais je suis pour ma part bien content de ses interventions toujours très mésurées, mais surtout cohérentes !
Je ne pense pas qu'un pro doit obligatoirement "créer" des sources. Il a peut-être d'autres choses à faire... travailler, entr'autres... et aider les autres sur codes sources !

Grand merci kankrelune

Johann *

Commentaire de younes371 le 18/09/2006 18:21:40

Salut johann1 ,
j ai demandé ça car il critique toujours les scripts. alors ils nous empeche a remettre des nouveaux scripts ici, tjs des mauvais commentaires, il ne donne pas un cup de pouce ....
Comprend moi stp

Commentaire de younes371 le 18/09/2006 19:07:42

Ok johann1 , je suis dsl, merci,
" si je lis bien tes interventions, tu ne souhaites "critiquer de façon négative" vomes interventions !  "
Je vais mettre ça ds ma tete,
Désolé une autre fois a vous les 2

Commentaire de zefo le 19/09/2006 00:49:39

Merci kankrelune pour tes remarques très constructives !
à l'origine je cherchais un script simple d'installation pour créer une petite galerie perso, donc seul le résultat m'intéressait et puis j'ai un peu modifié le code et j'ai voulu le partager.
Lorsque j'aurais fait évolué le code je posterai sur le site.
Pour ceux qui cherchent des solutions toutes faites je vous conseille plutôt 'Simple Picture Gallery Manager' à cette adresse http://spgm.sourceforge.net/?lg=fr.
Mais le but ici c'est plutôt de faire travailler nos neuronnes, alors à vos claviers....

Commentaire de RcrivelliNET le 19/09/2006 07:37:43

Hello,
J'ai enfain eu le temps de m'attaquer un petit peut à ce problème de commantaire sur la photo.J'ai trouver une piste, j'ai observé que les info ITPC et EXIF de s'affichait pas ... en commantant la partie affichage des EXIF ... au miracle ... les commentaire s'affichent et les infos ITPC (pour autant qu'on en ai mise) marche aussi ... maintenant il faudrait s'attaquer à a ce bout de code qui affiche les EXIF, pourquoi arrête t'il l'execution du code ? ... si j'ai d'autre info je les poste... pour rappel je suis sur EasyPhp 1.8. Merci a tous.

Commentaire de kankrelune le 19/09/2006 11:58:19

Pour remettre les choses en place je voudrais préciser que si mon but avait été juste de critiquer cette source je ne me serais pas fait chier à pondre un paté détaillant, avec exemple, chaque points que je trouve critiquable... j'aurais juste dis c'est nul et mal codé... hormis ma première phrase qui peut paraitre sèche (et pourtant ce n'est pas le but) je ne vois pas où est le problème dans mon commentaire... quand au récurant "si t'es si fort fais nous voir tes sources" je ne relèverais pas, aucun interet et comme l'a dit Johann1 les connaissance de quelqu'un ne se trouve en rien dans l'étalage de ses réalisations... .. .

Pour finir je dirais qu'il est déplacé que les personnes se sentant agressées ne soit pas les auteurs de la source critiqué... ce dernier aillant, je pense, compris le but de mes critiques... .. .

@ Zefo... biensur qu'il existe des galleries déja faite très bien, voir même des usines à gaz de fonctionalités... là n'est pas la question au contraire... ton projet est une très bonne base et les modifications à apporter sont, hormis éventuellement quelques options supplémentaires (suivant le gout de chacun), principalement du coté syntaxique et structurel... pour le reste ta source à sa place ici bien plus que bon nombre de "hello word" et le fait de la partager est tout à ton honneur... j'espere que mon commentaire ne t'aura pas fait croire que je pense le contraire... .. .

@ tchaOo°

ps : pour le problème de fin d'execution prématuré remplacés le error_reporting(2039) par error_reporting(E_ALL) dans l'index.php ligne 246 et dites nous si vous voyez des messages d'erreur s'afficher

Commentaire de MrBoubou le 19/09/2006 13:40:40

Salut à tous,
j'ai une erreur lors de l'installation d'un code sur un répertoire :
"Warning: rename(images/Miss,images/) [function.rename]: Invalid argument in /homepages/40/..../htdocs/photos/admin.php on line 94
qui est :
92:    if ($modifier)
93:     {
94:         if ($nomf<>$gal) {rename($Rep_Photos.$gal,$Rep_Photos.$nomf); $gal=$nomf;}
95:         if ($descriptionf<>$Description or $passf<>$Password)

d'autre part, quand un mot de passe est entré manuellement dans le fichier "infos_gallerie.php", même en inscrivant le mot de passe, on ne peut y accéder.  On a alors le message suivant : "Cette Galerie n'existe pas ou plus."

Quelqu'un a-t-il une idée?

Merci à tous.

Commentaire de MrBoubou le 19/09/2006 14:30:59

Merci pour l'info kankrelune.

Commentaire de pdl le 19/09/2006 16:34:34

RcrivelliNET, as-tu bien chargé l'exention php_exif ? car sans celle-ci tu ne pourras pas afficher les données EXIF et le script s'arrête.

Commentaire de MrBoubou le 20/09/2006 09:57:50

Merci, Zefo, la modif du code est efficace mais le problème de l'impossibilité d'ouverture de la galerie protégée reste entier. j'ai bien le nom de la galerie avec son copyright indiquant qu'il est protégé et lors de l'entrée du mot de passe, j'ai toujours le message suivant : "Cette Galerie n'existe pas ou plus."
En tout cas, bravo et merci pour cette belle source.

Commentaire de MrBoubou le 21/09/2006 00:12:09

Salut à tous, j'ai fait quelques modifications sur le script d'Admin en utilisant les "SESSION"s :
ligne 1 :
<?php session_start();
après la ligne 13:

//-------[utilisation d'une variable de session pour l'administration]-------
if ($pass==$Mot_de_passe) {
$HTTP_SESSION_VARS['pass'] = $pass;
}
j'ai remplacé les lignes 20 à 46 (début du switch) par :

// --------------[modif pour utilisation de SESSION Asmin]--------------
if (!isset($HTTP_SESSION_VARS['pass'])){
      echo '
      <div id="gallerytitle">
          <h2>Administration</h2>
      </div>';
      //Affiche formulaire Mot de Passe
      echo '
      <div class="imgcommentform">
          <h3>Accés restreint :</h3>
          <form id="commentform" action="#" method="post">
          <div>
          <input type="hidden" name="topic" value="accueil" />
              <table border="0">
                  <tr>
                      <td><label for="website">Mot de passe:</label></td>
                      <td><input type="password" id="pass" name="pass" size="40" value="" class="inputbox" /></td>
                  </tr>
              </table>
              <br />
              <input type="submit" value="Entrer" class="pushbutton" />
          </div>
          </form>
      </div>
      ';

  }else{
//----------------------[fin de première partie SESSION]------------------------  
  switch ($topic){...

et avant l'ancienne ligne 262 //------[fin du code]----
j'ai placé :
} //-----[FIN modif SESSION]---------------------------------

Voilà, cela fonctionne et c'est beaucoup plus pratique pour administrer sans entrer à chaque fois le mot de passe administrateur.

Autre point, j'ai résolu mon problème de "mot de passe sur les galeries" qui ne fonctionnait pas, en modifiant la ligne 385 de l'index :
if ($_GET['pass'] != $Password) { par
if ($_GET['pass'] !== $Password) {
puis tous les liens où nous avions par exemple :
action=voir_gallerie&amp;rep="
par
action=voir_gallerie&rep="
le amp; ne me parait pas utile. Ai-je tort ?

Avant j'avais déporté toutes les fonctions dans un fichier extérieur, appelé en début de page index par l'instruction :
require_once('fonctions.php');
à la ligne 59 par exemple.

Voilà, encore merci à Zefo pour seon application et à Kankrelune pour ses idées et son aide.

Commentaire de zefo le 21/09/2006 12:09:06

@ MrBoubou : Merci pour la ligne 385.
Pour les sessions j'ai fait différemment de toi, mais surtout il faut retirer tous les liens faisant apparaître le mot de passe.

Sinon il me reste encore pas mal de boulot pour continuer à épurer le code, mais j'ai déjà posté le travail fait. ensuite je pourrais ajouter de nouvelles options.

@+

Commentaire de MrBoubou le 22/09/2006 00:18:28

@Kankrelune :
"...Le &amp; dans les liens est indispensable pour la conformité au normes W3C..."

j'ai essayé de les remettre à la place de "&" simple.
Résultat : je ne peux plus afficher les galeries protégées par un mot de passe.

En les remplaçant par "&" seulement, ça fonctionne !!!
Comment expliquer cela ?

A bientôt.