Cohérence des define…
Rector m’a fait des misères car les constantes sont un peu définies sans convention dans ce plugin.
La convention :
- dans PHP est : une constante est en majuscule… (et sans souligné devant).
- dans SPIP c’est en majuscule logiquement, et avec un souligné devant.
Dans ce plugin il y a une grande quantité de define, dont une partie est en majuscule, l’autre en minuscule.
2 remarques :
- ça devrait être en majuscule, quitte à en avoir (ça éviterait que des outils automatiques fassent des bêtises :p).
- l’utilisation réelle est plus de l’ordre d’un tableau de configuration (figé une fois construit), avec des valeurs par défaut qui peuvent être surchargées et définies dans mes_options.php ou je ne sais quoi. Un peu comme symfony/config ou des outils proches ou plus simples.
Pas sûr qu’il faille changer quelque chose là maintenant… mais c’est à noter.