Discussion module:Lien interwiki

Remarques en passant[modifier le code]

Il y a une fonction mw.text.trim.
La catégorisation devrait dépendre de l'espace de noms de la page courante plutôt que de l'espace de noms de la cible du lien.
J'ai vu deux utilisations directes de {{Lien/témoin}} ailleurs, il faudra voir si on peut s'en passer ou s'il faut une fonction auxiliaire ({{Idée d'article}} : sans doute pas nécessaire, {{HBeff joueur}} : ça a l'air plus compliqué).

Merci d'avoir étudié le code :-)

Pour le mw.text.trim : je sais, mais il est peu performant (notamment car il utilise ustring) et je veux vraiment que ce module soit optimisé. Bon, là on parle du trim d'un seul paramètre au maximum, mais quand même. En fait, c'est plutôt que je souhaite "propager" dans la codebase du wiki une fonction optimisée au lieu du mw.text.trim qui est bien lent. Ça aurait davantage de sens par exemple dans Module:Liste éléments où le trim est potentiellement exécuté de nombreuses fois. Il faudra que je benchmarke pour me faire une idée des performances.
Bien vu, c'est corrigé. Le pire, c'est qu'en voyant le {{{1|}}} dans le {{Lien/témoin}} je m'étais dit « ah attention, ce n'est pas la page actuelle c'est la page cible, il ne faut pas que je me fasse avoir »… du coup j'ai bien évité l'erreur mais j'en ai fait une autre ailleurs. <shame>
C'est assez beurk ce genre d'utilisations extérieures du sous-modèle, mais en attendant de voir comment elles pourraient être traitées, j'ai découplé le code dans le module, de sorte que {{Lien/témoin}} puisse appeler la fonction du module, pour ne pas avoir de duplication de code.