[SPIP Zone] r87653 - in _plugins_/saisies/trunk

Le 26.02.15 22:18, marcimat@rezo.net a écrit :

Author: marcimat@rezo.net
Date: 2015-02-26 22:18:47 +0100 (Thu, 26 Feb 2015)
New Revision: 87653

Modified:
    _plugins_/saisies/trunk/inc/saisies.php
    _plugins_/saisies/trunk/saisies_pipelines.php
Log:
Coquille dans un phpdoc ; et pour le fichier saisies_pipelines, je ne sais pas, il ne voit pas bien le phpdoc d'entête. Je change les fins de ligne comme les autres fichiers (LF) et non CRLF pour tester.

Details: Connexion · GitLab

ce pas de fermeture m'étonne toujours. Tu peux me rappeler la raison? c'est tellement bizarre ...

--
Maïeul

(j'ai encore oublié la zone !)

Le 26/02/2015 22:21, Maïeul a écrit :

ce pas de fermeture m'étonne toujours. Tu peux me rappeler la raison?
c'est tellement bizarre ...

«
Si un fichier est purement du code PHP, il est préférable de ne pas placer la balise de fermeture à la fin du fichier. Ceci permet d'éviter d'oublier un espace ou une nouvelle ligne après la balise de fermeture de PHP, ce qui causerait des effets non voulus car PHP commencera à afficher la sortie, ce qui n'est souvent pas ce qui est désiré.
»

MM.

salut,
j’avais une question justement : est-ce qu’on doit généraliser ce principe ? dès qu’on tombe sur un fichier php fermé on retire la balise de fermeture ?

Le 27/02/2015 08:41, chankalan a écrit :

salut,
j'avais une question justement : est-ce qu'on doit généraliser ce
principe ? dès qu'on tombe sur un fichier php fermé on retire la balise
de fermeture ?

Personnellement c'est une habitude que j'ai prise depuis longtemps sur tout mon code php non spip.
C'est dans les bonnes pratiques, PHPstorm indique même un warning sur ces balises de fermeture.
Dans le code spip, je ne sais pas s'il y a une charte (jamais vue en tout cas).

--
nicod_

nicod_ a écrit le 27/02/2015 09:58 :

Le 27/02/2015 08:41, chankalan a écrit :

salut,
j'avais une question justement : est-ce qu'on doit généraliser ce
principe ? dès qu'on tombe sur un fichier php fermé on retire la balise
de fermeture ?

Personnellement c'est une habitude que j'ai prise depuis longtemps sur
tout mon code php non spip.
C'est dans les bonnes pratiques, PHPstorm indique même un warning sur
ces balises de fermeture.
Dans le code spip, je ne sais pas s'il y a une charte (jamais vue en
tout cas).

C'est ici :

--
RealET