Refs: #5369

_DIR_LOG et _FILE_LOG_SUFFIX ne sont pas définis lorsque spip_log est appelé dans mes_options - fixes #5152

* Passe 1 : le Spacing générique et les déclarations de classes et de fonctions 1217 fixes dans 158 fichiers
* Passe 2 : Structures de contrôle (if, for, etc.) 967 fixes dans 168 fichiers
* Passe 3 : Appels de fonctions, instanciations de classes 1810 fixes dans 128 fichiers
* Passe 4 : Quotes, Double-quotes 3179 fixes dans 174 fichiers
* Passe 5 : Short Array syntax 2697 fixes dans 248 fichiers

Co-authored-by: JamesRezo <james@rezo.net>
Reviewed-on: https://git.spip.net/spip/spip/pulls/4868


Co-authored-by: JamesRezo <jamesrezo@noreply.git.spip.net>
Co-committed-by: JamesRezo <jamesrezo@noreply.git.spip.net>

Franck a rédigé 4 years ago et marcimat a validé 4 years ago

«avec tendresse depuis 2001»

Fixer operators_spaces : espace de part et d'autre des opérateurs binaires (règle SPIP sur les - + * / etc...)

doivraient être en minuscule.

- encoding (utf8)
- eof_ending (saut le ligne en fin de fichier)
- elseif (pas else if)
- function_call_space (espaces sur fonctions)
- function_declaration (pareil)
- function_typehint_space (pareil)

Regles : 
- un espace après chaque virgule
- un espace avant et apres chaque '='

passage du copyright en 2015

Evite une notice lors d'une action en cours interrompue qui doit reprendre, par exemple pour une restauration de dump qui demande de se reconnecter. faudrait-il indiquer <pre>:null()::</pre> ou <pre>:aucune_fonction()::</pre> dans le log ? quand $fu est vide on a <pre>:()::</pre>, ce n'est pas tres clair.

PHP Fatal error:  Call to undefined function sous_repertoire() 
lors des appels du cron

define('_LOG_FILELINE',true); permet d'ajouter dans les log le fichier, la ligne et le nom de la fonction d'ou le log est généré. Utile en debugage

Fatal error: Call to undefined function sous_repertoire() in /var/alternc/html/k/kent1/spip-mutu-3.0/ecrire/inc/log.php on line 38

Je suis tombé dessus en mutu

Déplacer les logs dans le répertoire tmp/log/ cela semble plus lisible que de nombreux fichiers éparpillés à la racine de tmp/

r5629 a introduit un echappement du contenu des log pour eviter toute injection de code executable dans les logs.
r10325 a restreint cet echappement a spip.log uniquement, rendant les logs annexes vulnerables a des injections
On retablit l'echappement généralisé, car ces logs annexes peuvent etre aussi vecteur d'injection de code, mais on introduit une constante _LOG_BRUT permettant de desactiver l'échappement en configuration de test/debug.

la separation des logs prives/publics en deux fichiers s'avere peu pratique a l'usage. On revient a un fichier unique, en prefixant les log d'un :Pri: ou :Pub: pour indiquer si le log vient du prive ou du public, et permettre un filtre grep facile.

La specification du niveau de log permet maintenant de declencher des logs verbeux uniquement dans le prive si besoin, ce qui évite le probleme du noyage des logs prives dans le flux de log public sur un serveur de production.

Je profite de cette écriture massive pour normaliser quelque chose de trompeur lorsqu'on compare deux versions, savoir l'usage de " ou ' dans le premier argument de define et defined. Comme les chaînes entre apostrophes sont plus rapidement analysées que celles entre guillemets, je choisis l'apostrophe.

Dépot obtenu avec le script Shell:
{{{
a=$(find . -name "*.php" |grep -v extensions/ | grep -v /config/ | grep -v index.php | grep -v public.php | grep -v prive.php )
echo -n "Fichiers: "
echo $a|wc -w
for i in $a
do
sed -f ~/Sites/spip/spip.sed $i > /tmp/f.php
if diff -q $i /tmp/f.php
then 
    :
else
       diff $i /tmp/f.php
#     echo $i;   php /tmp/f.php
#    mv /tmp/f.php $i
fi
done
}}}
et le script Sed:
{{{
s/Copyright (c) 2001-20../Copyright (c) 2001-2011/
s,\(if [(]!*\)*\(defined* *[(]\)"\([^"]*\)"\(.*\);[[:space:]]*[#/]*.*$,\1\2'\3'\4;,
}}}

[14396] en mieux: le spip_log par défaut doit se rappeler récursivement lui-même plutot que la version surchargeable, sinon quand celle-ci est fausse ça part en boucle.