#1 2006-12-27 00:09:50

strotti
Membre
2006-07-28
22

Mise a jour bug track 529

Bonjour tout le monde,

j'utilise la galerie depuis 2 ans et demi et depuis la version 1.6 (moi etant à la 1.5) je rencontrais des problemes quand je cliquais sur la vignette. J'avais cherché sur le forum et j'avais decouvert que plusieurs personnes avaient le meme souci. Finalement un bug track a ete ouvert.

http://bugs.phpwebgallery.net/view.php?id=529

Le probleme est que meme avec la version 1.6.2 le probleme persistait et donc, en faisant plusieurs recherches sur le forum j'ai trouvé une "solution". Enfin, pas moi, mais Yves ;-).

http://forum.phpwebgallery.net/viewtopic.php?id=8858

Ma question etait:

pourquoi le bug track n'a pas ete mis a jour par les developpeurs? Est-ce car ce n'est pas la bonne solution?

Merci pour vos reponses.

IIS6
        * System d'exploitation: WINNT
        * PHP: 4.3.9 (Montrer les informations)
        * MySQL: 4.0.18-nt

Hors ligne

#2 2006-12-27 07:46:30

VDigital
Former Piwigo Team
Montpellier (FR)
2005-05-04
15127

Re: Mise a jour bug track 529

C'est la solution pour vous très bien.
Il faut qu'on essaie cette configuration sur d'autres serveurs.
Les variables systèmes ne répondent pas toujours de la même façon suivant les environnements.
Il ne faudrait pas en solutionnant votre problème que nous placions d'autres utilisateurs de PWG devant un autre problème.
Mais le bug 529 est suivi comme tous les autres.

(Rub fait un travail énorme dans le suivi par exemple).
8-)


Vincent -« Plus vidéaste averti que photographe amateur... »
La galerie - Le blog   

Piwigo est une application libre de gestion de photos en ligne.

Hors ligne

#3 2006-12-27 10:41:42

strotti
Membre
2006-07-28
22

Re: Mise a jour bug track 529

Merci pour votre réponse.

Bonnes fêtes.

Hors ligne

#4 2006-12-27 11:07:46

rvelices
Équipe Piwigo
2005-12-29
1417

Re: Mise a jour bug track 529

D'apres ce que je sache aucun d'entre nous utilise IIS pour developper.

La solution proposee peut marcher sous IIS et pour certaines configuraitons d'Apache, mais j'ai en tete au moins une configuration ou ca ne fonctionne pas.

Hors ligne

#5 2006-12-27 11:20:22

rub
Former Piwigo Team
Lille
2005-08-26
5239

Re: Mise a jour bug track 529

VDigital a écrit:

Mais le bug 529 est suivi comme tous les autres.

Et tu as raison de faire un petit up de temps en temps et surtout avant la prochaine sortie d'une version.
Car entre chaque version, on n'a pas le temps de traiter toutes les fiches et le choix est parfois abitraire.

En tout cas, tu as reéouvert le "débat" ;-)

Hors ligne

#6 2006-12-27 11:21:28

VDigital
Former Piwigo Team
Montpellier (FR)
2005-05-04
15127

Re: Mise a jour bug track 529

Merci, rvelices, c'est bien ce que je craignais.
8-)

[HS]
IIS n'est pas utilisé pour développer, c'est un peu normal: mise en oeuvre, sécurité, coût.
Je ne peux que vérifier ponctuellement.
Je n'ai pas l'espace IIS nécessaire pour monter une version de PWG pour faire des simples tests.
Si un hébergeur veut nous offrir un espace IIS gratuit pour des tests...
8-)
[/HS]


Vincent -« Plus vidéaste averti que photographe amateur... »
La galerie - Le blog   

Piwigo est une application libre de gestion de photos en ligne.

Hors ligne

#7 2006-12-27 11:33:31

rub
Former Piwigo Team
Lille
2005-08-26
5239

Re: Mise a jour bug track 529

rvelices a écrit:

D'apres ce que je sache aucun d'entre nous utilise IIS pour developper.

Et ca ne risque pas d'arriver car il n'y a pas de version non payante, à ce qu'il me semble.
Par contre, si l'équipe de dev avait à disposition par un tierce personne d'un environnement de test sur un config IIS, ca pourrait être utile.

rvelices a écrit:

La solution proposee peut marcher sous IIS et pour certaines configuraitons d'Apache, mais j'ai en tete au moins une configuration ou ca ne fonctionne pas.

On peut aussi passer par une solution sans utilisation $_SERVER?
Mais ca ne résoudra pas les autres utilisations de $_SERVER (s'il y en a!).

Dernière modification par rub (2006-12-27 21:02:08)

Hors ligne

#8 2006-12-27 11:38:07

rub
Former Piwigo Team
Lille
2005-08-26
5239

Re: Mise a jour bug track 529

VDigital a écrit:

[HS]
Si un hébergeur veut nous offrir un espace IIS gratuit pour des tests...
[/HS]

C'est ce qu'on appelle un message croisé ;-)

Hors ligne

#9 2006-12-27 14:35:02

rub
Former Piwigo Team
Lille
2005-08-26
5239

Re: Mise a jour bug track 529

Extrait du bug tracker:

Code:

Pour corriger tous les $_SERVER['SCRIPT_FILENAME'], 2 solutions me viennent à l'esprit:
  1 Affectation d'une valeur à $_SERVER['SCRIPT_FILENAME'] si elle est vide (Solution de elguaro mais en effectuant des tests sur les variables de substitutions ou en utilisant $_SERVER['PHP_SELF'])
  2 Gestion interne d'un variable pour chaque .php

1
Avantages:
  Transparent au niveau du code
  Peut-être appliqué à d'autres variables $_SERVER
Inconvénients:
  Ne fonctionnera pas forcément sur tous les environnements
  Un serveur IIS de test est nécessaire


2
Avantages:
  Plus de contraintes d'environnement
Inconvénients:
  Modification mineure de l'ensemble des pages
  Résout que $_SERVER['SCRIPT_FILENAME']

Dernière modification par rub (2006-12-27 21:02:28)

Hors ligne

#10 2006-12-27 20:59:04

mathiasm
Former Piwigo Team
2006-02-06
2692

Re: Mise a jour bug track 529

IIS est dispo sur toute machine équipée d'un windows XP Professionnel (et pas Home)...

Certains en ont donc p-ê au boulot, même si ce n'est pas forcément autorisé ...

Hors ligne

#12 2006-12-31 03:04:22

strotti
Membre
2006-07-28
22

Re: Mise a jour bug track 529

Salut Rub,

je t'ai envoyé un mail avec les résultats des tests.

Bàt

Hors ligne

#13 2006-12-31 11:52:13

rub
Former Piwigo Team
Lille
2005-08-26
5239

Re: Mise a jour bug track 529

bugtracker a écrit:

Merci à strotti et VDigital pour leurs tests.
La casse n'est pas la même suivant les systèmes (et les variables aussi en passant).
J'ai mis à jour le fichier test_IIS.php pour de nouveau tests.

Merci bien! ;-)

Hors ligne

#14 2006-12-31 18:06:49

rub
Former Piwigo Team
Lille
2005-08-26
5239

Re: Mise a jour bug track 529

Le problème devrait être résolu avec les commits décrits dans la fiche 539

Pour tester/intégrer les modifications avant la prochaine version:


Ajouter la fonction suivante dans le fichier /include/functions.inc.php (juste avant le ?>)

Code:

/**
 * Return basename of the current script
 * Return value are chnage to loawer case
 *
 * @param void
 *
 * @return script basename
 */
function script_basename()
{
  if (!empty($_SERVER['SCRIPT_NAME']))
  {
    $file_name = $_SERVER['SCRIPT_NAME'];
  }
  else if (!empty($_SERVER['PHP_SELF']))
  {
    $file_name = $_SERVER['PHP_SELF'];
  }
  else if (!empty($_SERVER['SCRIPT_FILENAME']))
  {
    $file_name = $_SERVER['SCRIPT_FILENAME'];
  }
  else if (!empty($_SERVER['PATH_TRANSLATED']))
  {
    $file_name = $_SERVER['PATH_TRANSLATED'];
  }
  else
  {
    $file_name = '';
  }

  // $_SERVER return lower string following var ans systems
  return basename(strtolower($file_name));
}

Dans les fichiers
  /include/common.inc.php,
  /include/functions_calendar.inc.php,
  /include/section_init.inc.php
Remplacer partout

Code:

basename($_SERVER['SCRIPT_FILENAME'])

par

Code:

script_basename()

Sur ce, une bonne année 2007 à tous!

Hors ligne

#15 2006-12-31 19:07:09

VDigital
Former Piwigo Team
Montpellier (FR)
2005-05-04
15127

Re: Mise a jour bug track 529

[HS]Un bon réveillon à toi aussi Rub... Les voeux pour 2007 devront attendre encore quelques heures.
8-)
[HS]


Vincent -« Plus vidéaste averti que photographe amateur... »
La galerie - Le blog   

Piwigo est une application libre de gestion de photos en ligne.

Hors ligne

Pied de page des forums

Propulsé par FluxBB

github twitter newsletter Faire un don Piwigo.org © 2002-2024 · Contact