Merci Nikrou pour m’avoir conseillé PHPStan, ça m’a permis de trouver deux ou trois bugs qui étaient restés sous les radars dans Clearbricks depuis des années ! Je me suis cependant limité au niveau 5 de l’analyse, qui en compte 8, déjà bien suffisant.
Du côté de Dotclear je vais faire de même, peut-être d’ailleurs en m’arrêtant encore plus bas vu que la structure de l’appli ne s’y prête pas toujours bien ; on a pas mal de code hors classes, plus quelques constantes et variables globales qui compliquent l’analyse.
Du coup j’ai conservé le typage que j’avais mis en place côté Clearbricks et suis tombé sur quelques bugs côté Dotclear à cause d’une utilisation « baroque », c’est-à-dire pas franchement prévue à l’origine ; effet de bord d’un code utilisé et parfois détourné depuis des années.
En tout cas il faudra faire quelques tests d’installation propre, de mise à jour et voir comment se comportent les quelques dizaines de plugins et de thèmes avant de publier la prochaine 2.19 !
