En date de : Sam 25.7.09, Committo,Ergo:sum <esj@rezo.net> a écrit :
Ton patch pose 2 pbs.
D'une part, il faut prendre isset et pas in_array car tu
passes à côté de la boucle HIERARCHIE et de l'alias
SYNDICATION.
déja que c'est pas simplissime à comprendre ce truc alors si en plus il y a des cas particuliers ^^
Ensuite, il ne fait le strtolower que pour les tables
externes, mais ça pose un pb pour une table externe qui
appartient au jeu de tables d'un autre site SPIP: le
strtolower est fait pour le jeu local, mais pas pour le jeu
externe sauf les tables principales, c'est pas
démesurément intuitif.
oui, dans l'idée c'était pour n'impacter que sur les tables non-SPIP mais c'est vrai qu'on peut aussi gérer des tables SPIP comme de l'externe! (ça simplifie pas non plus les choses ça...)
J'envoie une autre solution sur la branche dev, tu vois
qu'il y a trop de pièges pour envoyer ça dans la stable.
Mais tu peux évidemment la surcharger par l'autre
facilement: c'est la fonction principale du phraseur.
clairement ce patch n'était qu'hypothétique: je me doutais bien qu'il ne faisait qu'effleurer la complexité du fonctionnement du compilo...
L'important reste que ta soluce résout le problème: je vais reporter dans la stable que j'utilise: merci!