Message essentiellement pour Joseph. 
Donc Joseph, je voulais te demander si tu utilisais vraiment le deuxième argument de la fonction.
D'après moi, il est inutile car la fonction est utilisée pour parser automatiquement des chaines venant soit de fichiers de conf, soit de l'utilisateur dans un champ. Dans les deux cas, le tableaux des remplacements éventuels n'est jamais utilisé, vu que c'est pour traduire des chaines qui se suffisent à elles-mêmes. Je ne vois pas de cas d'utilisation où il y aurait un contexte supplémentaire rempli non-automatiquement (vu qu'on ne gère que <:truc:> et pas <:truc{param=machin}:>, et même si c'était le cas, ça serait alors automatique et pas en argument).
Bref, s'il n'est pas utilisé, est-ce que je peux le supprimer ? En revanche là je voudrais ajouter un argument (et donc en deuxième, car plus souvent utilisé) pour choisir le mode lorsque c'est PAS _T().
En effet quand on utilise cette fonction sur des options, ya plein de cas où typo() est super emmerdante ! La fonction est surtout utile lorsqu'il y a des <multi> mais sinon ça peut donner n'importe quoi !...
Par exemple imagines que l'option est une expression régulière. Avec des "?". Et bam, ça casse tout vu que ça corrige de la typo dessus.
Le deuxième paramètre serait donc plutôt le "mode" de typo() :
- "toujours" : dès que c'est pas _T() c'est typo() (par défaut ?)
- "jamais" : ne rien faire si c'est pas _T()
- "multi" : ne passer par typo() seulement si on trouve au moins un "<multi>" dans la chaine
C'est cette troisième option "intelligente" qui serait bien pratique quand on parse automatiquement des options de config avec cette fonction.
--
RastaPopoulos
Aucun souci pour ma part. Personnellement, je n'utilise pas ce second argument et il peut donc être modifié.
Bien cordialement
Joseph
Le 01/03/2010 14:25, RastaPopoulos a écrit :
Message essentiellement pour Joseph. 
Donc Joseph, je voulais te demander si tu utilisais vraiment le deuxième argument de la fonction.
D'après moi, il est inutile car la fonction est utilisée pour parser automatiquement des chaines venant soit de fichiers de conf, soit de l'utilisateur dans un champ. Dans les deux cas, le tableaux des remplacements éventuels n'est jamais utilisé, vu que c'est pour traduire des chaines qui se suffisent à elles-mêmes. Je ne vois pas de cas d'utilisation où il y aurait un contexte supplémentaire rempli non-automatiquement (vu qu'on ne gère que <:truc:> et pas <:truc{param=machin}:>, et même si c'était le cas, ça serait alors automatique et pas en argument).
Bref, s'il n'est pas utilisé, est-ce que je peux le supprimer ? En revanche là je voudrais ajouter un argument (et donc en deuxième, car plus souvent utilisé) pour choisir le mode lorsque c'est PAS _T().
En effet quand on utilise cette fonction sur des options, ya plein de cas où typo() est super emmerdante ! La fonction est surtout utile lorsqu'il y a des <multi> mais sinon ça peut donner n'importe quoi !...
Par exemple imagines que l'option est une expression régulière. Avec des "?". Et bam, ça casse tout vu que ça corrige de la typo dessus.
Le deuxième paramètre serait donc plutôt le "mode" de typo() :
- "toujours" : dès que c'est pas _T() c'est typo() (par défaut ?)
- "jamais" : ne rien faire si c'est pas _T()
- "multi" : ne passer par typo() seulement si on trouve au moins un "<multi>" dans la chaine
C'est cette troisième option "intelligente" qui serait bien pratique quand on parse automatiquement des options de config avec cette fonction.