🌍
Français
Juste un p'tit message pour la personne qui veut s'en occuper.
Je propose de faire deux fonctions comme l'a suggérer je en sais plus qui. Une fonction redirect() qui fait un header("Location: ...") tout bêtement et une deuxième qui s'appelerait refresh() et qui correspondrait à la fonction redirect() avant que je ne la modifie. Avec ces deux fonctions on résoud tous les problèmes.
Pour ce qui est du problème potentiel de l'utilisation de header() je ne pense pas que cela en soit un réellement. On livre une version où il n'y a pas de problème. Si la personne qui utilise pwg et modifie des fichiers php (y compris config_local.php) c'est à ses risques et périls. Quoi qu'il en soit sur ma galerie personnelle j'ai toujours utilisé header() et je n'ai pas le moindre problème.
rub a écrit:
nicolas a écrit:
Je me suis désassigné de la fonctionnalité si cela intéresse quelqu'un de faire les développements nécessaires.
Et pour le problème que j'avais indiqué?
rub a écrit:
J'ai ma BSF qui boucle à l'infini avec $conf['guest_access'] = false;
Et j'ai ca qui s'affiche sur fond blanc:Code:
Notice: Undefined index: language in d:\dev\bsf\include\functions_user.inc.php on line 527 Redirection... Cliquez ici si votre navigateur ne vous redirige pas.
En fait, le problème ne provient pas de la redirection mais des sessions.
Ayant descendu, les 2 modifications en même temps, il y a eu amalgame!
nicolas a écrit:
Je me suis désassigné de la fonctionnalité si cela intéresse quelqu'un de faire les développements nécessaires.
Et pour le problème que j'avais indiqué?
rub a écrit:
J'ai ma BSF qui boucle à l'infini avec $conf['guest_access'] = false;
Et j'ai ca qui s'affiche sur fond blanc:Code:
Notice: Undefined index: language in d:\dev\bsf\include\functions_user.inc.php on line 527 Redirection... Cliquez ici si votre navigateur ne vous redirige pas.
nicolas a écrit:
Je me suis désassigné de la fonctionnalité si cela intéresse quelqu'un de faire les développements nécessaires.
Pourquoi ?
Simplifier le redirect est probablement une bonne idée s'il est trop complexe et/ou mal utilisé.
Tu peux sans doute y arriver en tenant compte de ce qu'a dit z0rglub (header/pas header , je n'ai pas regardé, ça me dépasse un peu),
et ce que d'autres ont dit sur l'utilisation du template.
<:o)
Je me suis désassigné de la fonctionnalité si cela intéresse quelqu'un de faire les développements nécessaires.
rvelices a écrit:
Sinon je ne vois aucun interet de la mettre en template-common et ensuite la personnaliser selon le theme.
Moi non plus.
Je n'ai pas tout analysé, mais je ne vois pas en quoi cette modif du template aide à simplifier le redirect.
En quoi c'est beaucoup plus simple que d'appeler header.tpl, redirect.tpl, et enfin footer.tpl comme c'est fait partout dans PWG ?
<:o|
J'ai ma BSF qui boucle à l'infini avec $conf['guest_access'] = false;
Et j'ai ca qui s'affiche sur fond blanc:
Notice: Undefined index: language in d:\dev\bsf\include\functions_user.inc.php on line 527 Redirection... Cliquez ici si votre navigateur ne vous redirige pas.
A l'origine, les redirections utilisaient la directive "header". Devant le nombre de retour sur le forum à l'époque à cause des problèmes de caractères écrits avant le header, comme un caractère après le "?>" de include/mysql.inc.php, j'ai switché la redirection en meta-refresh HTML. Je préfère de beaucoup cette méthode, mais alors franchement beaucoup. C'est d'ailleurs la méthode que l'on retrouve sur de très nombreux projets et c'est à la fois efficace et ça marche partout.
Bref, je ne vois pas l'intérêt de repasser à header :-/
rvelices a écrit:
Si le but final est d'utiliser le header Location, d'accord - mais ca ne va pas trop marcher si les entetes http ont deja ete envoye.
Je sais bien mais je compte bien trouver un moyen de le gérer correctement sans cette redirection bancale.
rvelices a écrit:
Sinon je ne vois aucun interet de la mettre en template-common et ensuite la personnaliser selon le theme.
Je ne veux pas intégrer de thème.
rvelices a écrit:
En plus il faut que tu remettes les choses en page_header, car le slideshow et la notification par mail utilisent le meta refresh.
c'est fait. [Subversion] r1512
Si le but final est d'utiliser le header Location, d'accord - mais ca ne va pas trop marcher si les entetes http ont deja ete envoye. Sinon je ne vois aucun interet de la mettre en template-common et ensuite la personnaliser selon le theme.
En plus il faut que tu remettes les choses en page_header, car le slideshow et la notification par mail utilisent le meta refresh.
nicolas a écrit:
Je ne comprends pas pourquoi tu le fais ici mais quoi qu'il en soit on peut remettre les informations du footer.
Pour le debug des requetes sql: il y a plusieurs redirections en picture suite au "set as representative", "add to caddie", "rate" ... En desactivant le meta redirect sur le navigateur je me suis servi pour debugger les requetes sql.
Attention, je suis ca de loin, mais je sais que j'ai utlisé la notion de redirection pour la notification par mail.
Mais pas dans un cas "classique".
Je l'ai utilisé dans mon traitement pour outrepasser le timemout (les "vrais" fonctions le permettant étant non dispo chez Free, par exemple).
Donc une redirection avec comme url une page particulière et un temps d'attente de zéro.
On utilise les mêmes procédures pour chaque cas, mais je penses que plusieurs fonctions seraient peut-être mieux!
Qq'1 peut-il faire un point de l'endroit ou est utilisé un redirect et pour faire quoi?
En effet, je me suis rendu compte a posteriori que les messages de rvelices et chrisaga que tu cites sont dans le bugtracker. Difficile de suivre la conversation quand elle a lieu à 2 endroits différents :-/.
z0rglub a écrit:
(on a perdu les posts de rvelices et chrisaga dans ce topic ?)
Pourquoi tu dis ça ? On peut les recopier. Il n'y en avait pas. J'ai repris le message de rvelices postés dans Mantis.
(on a perdu les posts de rvelices et chrisaga dans ce topic ?)
rvelices a écrit:
C'est peut etre une amelioration au niveau de code, mais
1. Je travaille toujours en theme dark et maintenant chaque fois qui'il y a redirection, il y a une page entierement blanche affichee rapidement dans mon navigateur.
Je propose de régler ça dans une deuxième étape.
rvelices a écrit:
2. Notice: Undefined variable: user in D:\pwg\bsf\include\functions.inc.php on line 584
commit trop rapide. J'ai oublié d'enlever du code qui m'a servit à débugger!
Corrigé: [Subversion] r1509
rvelices a écrit:
3. Pendant le dev je me suis deja servi pour debugger les requetes sql et le temps d'execution affichees normalement dans le footer (c'est perdu maintenant)
Je ne comprends pas pourquoi tu le fais ici mais quoi qu'il en soit on peut remettre les informations du footer.
rvelices a écrit:
4. le titre de ta page t'aurais du faire un strip_tags
oui.
rvelices a écrit:
Je suis d'accord avec chrisaga que c'est mieux si on utilise le header/footer du template
Je le répète: le header et le footer y sont mais sous une forme simplifié pour ne pas dépendre du thème.
Mon idée est de réduire la fonction redirect() à celle qu'elle devrait être, à savoir celle du premier message de ce sujet.