je m'apprêtais à commiter la suppression des ?> donc je retranscris les échanges IRC pour discussion ici même
Pour Rasta et azzerty on commiterait sur
svn://trac.rezo.net/spip/spip/spip
directement avant report sur le stable
L'idée de fil est de contenter les tenants de la suppression du ?> final et ceux qui préfèrent savoir que le fichier se termine en remplaçant le ?> par ## END/n
je m'apprêtais à commiter la suppression des ?> donc je retranscris les
échanges IRC pour discussion ici même
[...]
L'idée de fil est de contenter les tenants de la suppression du ?> final
et ceux qui préfèrent savoir que le fichier se termine en remplaçant le
?> par ## END/n
On a vraiment besoin d'avoir ce ## END de remplacement ???
Parce que bon, savoir que le fichier se termine… ça se voit non ? c'est la fin, le bout, la finitude, là où le texte s'arrête, là où il n'y a plus de lignes après, le bout du monde quoi… non ?
je suggérais ça car il me semble qu'il y a deux religions sur le sujet :
celleux qui veulent être sûrEs d'avoir "tout", et celleux qui veulent pas
de saut de ligne plantogène ; perso en cette matière comme en bien
d'autres, je suis non seulement agnostique mais athée
Parce que bon, savoir que le fichier se termine… ça se voit non ? c'est la fin, le bout, la finitude, là où le texte
s'arrête, là où il n'y a plus de lignes après, le bout du monde quoi… non ?
Ben oui, mais si on a un doute ?
Ah tiens, ça se termine comme ça ce fichier ?
bizzarre, ya pas telle fonction dans ce fichier ?
Ah, peut être ça continue, je peux scroller ?
Hmm...
Il aurait pas été tronqué par erreur ce fichier ??
Voir ##END explicite le truc et lève le doute.
Ben oui et non… ce n'est juste qu'une habitude de codage a avoir. J'utilise dans la mesure du possible la norme PSR-2 quand je code, surtout si je commence de zéro.
De plus, avec l'utilisation d'un IDE, il est facile de configurer son environnement pour que cela fasse de façon intuitive.
J'utilise aussi bien Éclipse que Netbeans ou Sublime Text. Un jeu d'enfants de configurer le système pour avoir PHPMD, PHPCS et cie. (Windows, c'est une autre histoire. C'est du vécu.)
Je pense que pour le moment remplacer un ?> par #END risque de poser
plus de problème sur le suivi. Du genre mais eux c'est quoi la regle
d'où ça vient .... Coté explication, communication, .... je crois
qu'on ne gagne pas grand chose.
Remarque le script ne semble considérer que si il y a des des espaces
après le ?> terminal, il serait mieux d'avoir une règle égale pour
tous les fichiers. C'est plus compréhensible quand on limite les
exceptions.
Le fait de supprimer purement et simplement ?> me semble plus clair et
plus explicable vu la littérature présente. De plus cela me semble
plus facile à scanner par la suite pour corriger. Moins d'élément à
analyser.
Je suis d'accord. Et si c'est pour tester qu'on a bien les données qu'on doit avoir, il y a d'autres outils pour cela, dédiés à ce besoin (les signatures md5 ou autre de paquets, pour être sûr qu'on a la même version, les outils de versionnage qui permettent des DIFF, etc).
Mettre "NON MAIS ATTENTION LA C'EST LA VRAIE FIN DU FICHIER" sur chaque fichier de SPIP… ça me parait être du bidouillage.
Ben oui et non… ce n'est juste qu'une habitude de codage a avoir.
J'utilise dans la mesure du possible la norme PSR-2 quand je code, surtout
si je commence de zéro.
De plus, avec l'utilisation d'un IDE, il est facile de configurer son
environnement pour que cela fasse de façon intuitive.
J'utilise aussi bien Éclipse que Netbeans ou Sublime Text. Un jeu
d'enfants de configurer le système pour avoir PHPMD, PHPCS et cie.
(Windows, c'est une autre histoire. C'est du vécu.)
Tu as oublié ta conclusion. Peux tu préciser quel est le rapport ce fil ?
Oui pardon…
Ce que je disais, le fait de ne pas avoir de ?> à la fin du fichier et ne
pas avoir de ##END n'est pas un problème en soit. Si on (la core-team entre
autres) décide qu'on codera de telle ou telle façon, chacun prendra au fur
et à mesure l'habitude de cette "norme".
Pour la référence à PHPMD, PHPCS et cie, c'était pour que nos IDE (si ce
n'est pas natif) puissent prendre en compte la norme de codage de SPIP
(ici, la fin du fichier sans ?>).
La discussion de la fin de fichier PHP dans SPIP me fait largement penser
au PSR-2. Cela ne serait pas idiot d'adopter dans la mesure du possible
cette norme dans SPIP.
Il y a aussi un outil dédié pour psr1 et psr2 : http://cs.sensiolabs.org/ et qui permet de changer seulement certains éléments, dont les tags fermants.