En date de : Ven 24.7.09, Committo,Ergo:sum <esj@rezo.net> a écrit :
> Alors si dans le cas des boucles sur les tables SPIP
il semble obligatoire de faire ce passage en minuscules (le
nom de la table étant en majuscules et sans préfixe), en
revanche lorsqu'il s'agit d'interroger une table non-spip
(i.e. lorsqueil y a un ":" dans le paramètre nom de la
table de la boucle) on peut imaginer que cette mise en
minuscule ne soit pas obligatoire? ou peut être seulement
si le nom est *complètement* en majuscules seulement?Bonne prémices mais mauvaises conclusions. Les "tables
SPIP" sont celles listéee par la globale $tables_de_tables,
on peut envisager de ne faire le strtolower que pour elles
(i.e. que si le strtolower sur le type de la boucle donne un
index défini dans ce tableau, on le prend, sinon on garde
l'original). Regarde ce que ça donne avec cette stratégie.
OK, ça semble une bonne manière de restreindre la mise en minuscule "automatique" aux tables SPIP & assimilées uniquement
Ce qui pourrait être fait par les patch ci-dessous
Mais celle-là ou une autre, ça m'inquiète un peu sur le
plan de la compatibilité: combien de gens on écrit des
boucles en majuscules, sachant qu'il y aurait un strtolower
derrière ?
A priori on peut supposer qu'il n'y a pas forcément beaucoup de sites qui utilisent la fonctionnalité d'interrogation de bases externes... et que ceux qui l'utilisent sont suffisamment experts pour ne pas avoir codé leurs boucles de façon "négligente"!
D'une façon plus générale ça me semble limite de conserver un truc qui fera s'arracher les cheveux à tout développeur de squelette qui va se retrouver à interroger une base avec des noms de tables ayant des majuscules et qui n'y arrivera pas (ici on est clairement à la limite du bogue par rapport à la fonctionnalité attendue) uniquement pour conserver une tolérance avec ceux qui ont codé des boucles avec des noms de tables en majuscules alors qu'il n'y a pas lieu (là on est au contraire dans l'exploitation d'un effet de bord qui permet une dérogation "laxiste" à la rigueur nécessaire à l'utilisation de la fonctionnalité...)
Bref, en ce qui me concerne pour ce point: "Pas de pitié pour les squelettes laxistes"! 
SPIP 2.1.0 dev SVN [14270]:
Index: phraser_html.php