J’ai beau y retourner maintes et maintes fois, je trouve encore des horreurs dans le code de Dotclear et aujourd’hui c’est du côté de Javascript, où l’évolution d’une fonction il y a quelques années a laissé des scories et des trucs inutilisés…
Du coup un peu de ménage, et du code en moins, c’est toujours ça de pris \o/
Tu sais, c’est le genre de code que tu reprends pour ajouter ou modifier une fonctionnalité, et puis dès que ça a l’air de fonctionner quand tu testes, tu l’oublies pour passer à autre chose — ce que je n’aime pas beaucoup comme pratique[1] —, sans vérifier que le superflu avait été viré :-)
Note
[1] Je ne jette la pierre à personne, ça m’arrive aussi !
1 De Da Scritch -
T’utilises pas un linter, au moins pour réduire l’usage des “var” ?
Perso, je vois tellement pire ailleurs que je trouve le code honorable pour l’époque.
2 De Franck -
Si j’ai un linter mais et j’ai déjà pas mal basculé de choses du côté cool de la force, let/const, mais ne te fies pas au code de l’image, c’est un vieux truc :-)