Ben, si tu n'avais pas viré le paramètre que j'avais rajouté tu l'aurais eue,
parce que la variable globale use_cache, elle n'existe plus.
Bon bah moi, j'ai fini par comprendre que la classe Link était vivable si on faisait un rawurlencode
aux endroits adéquat, ce qui n'est pas complètement trivial (faut pas le refaire sur un déjà fait).
Du coup, j'ai pu faire un mix entre l'ancien et le nouveau, et il semble que ça marche dans tous les
cas de figure et proprement.
Ben, si tu n'avais pas viré le paramètre que j'avais rajouté tu l'aurais
eue, parce que la variable globale use_cache, elle n'existe plus.
Ah, bah voilà, c'est ça qui m'avait échappé.
Bon bah moi, j'ai fini par comprendre que la classe Link était vivable si
on faisait un rawurlencode aux endroits adéquat, ce qui n'est pas
complètement trivial (faut pas le refaire sur un déjà fait). Du coup, j'ai
pu faire un mix entre l'ancien et le nouveau, et il semble que ça marche
dans tous les cas de figure et proprement.
Je ne suis pas d'accord pour que tu remettes la méthode javascript comme ça,
à partir du moment où l'ancienne méthode peut fonctionner sans impact
quelconque ni sur les perfs ni sur ton concept de "pages 100%" ni rien
d'autre.
Déjà avec le nouveau compilo et la nouvelle gestion du cache il y a beaucoup
de perturbations et de bugs (à bon escient, hein, je ne dis pas !) pour
qu'on ne rajoute pas en plus des inconnues dans d'autres bouts du script.
L'énorme [au moins 
] désagrément de la méthode javascript, c'est qu'elle
bidouille les pages servies aux visiteurs normaux. La méthode ancienne ne
modifie que les pages servies aux admins, elle est donc plus fine.
-- Fil
Si ce n'est que ça, je rajoute un test de fonction définie et comme ça il n'y a
plus besoin de bidouiller pour les visiteurs non anormaux ....
esj
Si ce n'est que ça, je rajoute un test de fonction définie et comme ça il
n'y a plus besoin de bidouiller pour les visiteurs non anormaux ....
Non, on va revenir à ma méthode, qui est (en sortie) la même que celle
d'avant, et qui respecte le cahier des charges que tu nous as donné, de ne
pas mettre de php.
Tant que tu n'as pas expliqué en quoi ta méthode est meilleure (et
convaincu !), on conserve l'ancienne. C'est prudent, c'est conservateur,
c'est "si ça marche on ne touche pas".
-- Fil
Comment ça l'ancienne ?
L'ancienne interpolait du PHP, ce que tu as posté tout à l'heure ne le fait plus,
nous avons donc deux méthodes nouvelles, laquelle choisir ?
La tienne demande de parcourir toutes la table à la recherche d'une chaine dont
on n'espère que seul le compilo l'aura générée.
La mienne (avec le correctif ci-dessus) évite cette analyse.
Pour moi, le choix est clair.
esj
nous avons donc deux méthodes nouvelles, laquelle choisir ? La tienne
demande de parcourir toutes la table à la recherche d'une chaine dont on
n'espère que seul le compilo l'aura générée.
La page, tu veux dire, pas "toutes la table". Cela dit un strpos (juste pour
les admins), ça doit bien coûter une pico-seconde.
Pour moi, le choix est clair.
Peut-être, mais tu n'es pas tout seul !
-- Fil