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 :

  1. ça devrait être en majuscule, quitte à en avoir (ça éviterait que des outils automatiques fassent des bêtises :p).
  2. 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.

Personne assignée Chargement en cours
Suivi du temps Chargement en cours

Forge communautaire SPIP | Charte d'utilisation | Signaler un problème sur ce site