Le 13/08/2013 11:24, Kozlika a écrit :
Le 13 août 2013 10:49, Nicolas <[email protected]> a écrit :
SInon PHP-CS-Fixer permet de nettoyer de manière automatique. Par
exemple on peut se débarrasser des balises fermantes en une action.
Je me souviens que ça avait été évoqué il y a quelques années et
qu'Olivier y tenait beaucoup à cette balise fermante. Je ne dis pas ça
genre "touche pas à l'héritage de Dieu" mais parce qu'il y avait
peut-être une (bonne) raison à ça ?
S'il en avait une dans le cadre de Dotclear, ce n'est pas documenté. La
doc php elle-même conseille de ne pas mettre de balise de fermeture dans
les fichiers purement php :
http://www.php.net/manual/en/language.basic-syntax.phptags.php
"If a file is pure PHP code, it is preferable to omit the PHP closing
tag at the end of the file. This prevents accidental whitespace or new
lines being added after the PHP closing tag, which may cause unwanted
effects because PHP will start output buffering when there is no
intention from the programmer to send any output at that point in the
script. "
C'est aussi la bonne pratique prônée par les conventions de code des
frameworks php (zend, drupal, symphony...) et les conventions de code
PEAR ou PSR.
Dans la mesure où on sait que ça peut introduire des bugs et que
personne ne sait ce que ça peut apporter comme bénéfice dans le cadre de
Dotclear, je vote pour les virer :)
Pascal
--
Dev mailing list - [email protected] - http://ml.dotclear.org/listinfo/dev
