Bonjour Pascal,
- Ce que je propose de faire pour le nettoyage pour le moment, c'est de
faire passer le code de Dotclear dans PHP-CS-Fixer et pourquoi pas phpcs
... regarder ce que ça donne et en faire un petit point écrit
- répertorier, en regardant les différents sites sur le sujet, les
quelques optimisations que l'on peut réaliser facilement, comme count()
dans une boucle eh oui ! il y en a quelques uns
- supprimer les balises ?> de fin de fichier si c'est décidé ?
- faire passer tous les fichiers PHP en en utf8, sans BOM
Qu'en penses tu ? je ne risques pas de trop faire de dégât ?
Bernard
Le 13/08/2013 11:07, pascal chevrel a écrit :
Le 13/08/2013 10:49, Nicolas a écrit :
Bonjour,
oui il faudrait voir si Pascal veut toujours s'en occuper.
SInon PHP-CS-Fixer permet de nettoyer de manière automatique. Par
exemple on peut se débarrasser des balises fermantes en une action.
Nicolas
Salut,
Là je fais le pont demain soir pour partir dans le sud et mercredi
prochain je pars en congés au bord de la mer, donc dans un premier
temps, non :)
Pour l'instant je compte m'impliquer dans le code ponctuellement, je
n'ai réalistiquement pas le temps de m'investir sur des tâches qui
prennent du temps en continu, donc je propose des petits patchs sur
Clearbricks et l'antispam pour le moment et je suis disposé à écrire
des tests de temps en temps pour Clearbricks aussi.
Pour le nettoyage de code, ça serait bien effectivement de nettoyer
tout le code pour des conventions de code PSR (ma préférence, mais du
Zend, ou du PEAR ça m'irait aussi), ça serait plus lisible pour les
contributeurs occasionnels au moins. Rien que supprimer toutes les
balises de fermeture de php en fin de fichier, potentiellement ça peut
éviter des bugs aléatoire genre 'headers already sent' s'il y a un
espace en fin de fichier ou si un include a un BOM utf8.
Je connaissais phpcs mais pas phpcs-fixer, je découvre,
(https://github.com/fabpot/PHP-CS-Fixer) ça m'a l'air effectivement
bien cool !
A+
Pascal
--
Dev mailing list - [email protected] - http://ml.dotclear.org/listinfo/dev
