Remplacer dirname(__FILE__) par __DIR__

[bazooka07]

No description provided.

[bazooka07]

Utiliser les constantes magiques adéquates

[haruka-7]

J’hésite a intégrer cette modification dans PluXml 5.7, car la constante __DIR__ est gérée à partir de la version 5.3 de PHP. Bien que la current stable soit la version 5.6, je ne sais pas quelle est la part de serveurs/hébergeurs actuellement dans une version inférieur à 5.3...

[bazooka07]

La version 5.3 de PHP n'est plus maintenue depuis plus de 4 ans :
http://php.net/eol.php
En clair, il n'existe plus aucun patch de sécurité depuis cette date.
Je doute sincèrement du professionalisme des hébergeurs qui proposent encore cette version.
Même Free.fr qui a longtemps trainé les pieds avec la version 5.1 a basculé depuis 2 ans à la version 5.6 (version maintenue la plus agée). Et on pouvait leur trouver une excuse: c'était gratuit.

Le problème est ailleurs. Ce sont ces gens sous WIndows qui utilisent des bundles WAMP (Apache + PHP + MySQL sous Windows) avec des versions anciennes de PHP. Soit ils ne sont pas capables de mettre à jour leurs bundles, soit ces bundles ne sont plus maintenus.

Soit on veut assurer la comptabilté de PluXml avec des versions obsolétes de PHP, soit on veut améliorer PluXml en tenant compte des nouvelles fonctionnalités du langage

Il faut savoir faire un choix

Je doute également que ces sites développés avec PHP 5.3 soient en "responsive design" ou en "mobile first". A l'heure où la majorité des gens utilisent un smartphone ou une tablette pour surfer sur Internet, il y a peu de chance que ces sites soient beaucoup visités.

C'est une manière aussi de pousser les gens à maintenir leurs sites avec des versions à jour de PHP et de PluXml.

[haruka-7]

J'ai ajouté un commit pour faire la même chose sur le thème par défaut, mis à jour le changelog et mergé la PR.

[bazooka07]

Include n'est pas une fonction !!
C'est unedirective de programmation.
Les parenthèses n'ont pas lieu d'être
voir example sur :
http://php.net/manual/en/function.include.php
A la racine du site, faire :
grep include {core/admin,core/lib,themes/defaut,.}/*.php grep include {core/admin,core/lib,themes/defaut,.}/*.php |wc -l
Il n'y a plus qu'à corriger avec sed et l'expression qui va bien.