Seuls certains (comme typo()) sont ignorés car ils semblent avoir des spécificités ($connect = null).
De même, typage de quelques $serveur, mais il faudrait généraliser.
* 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: #4868
Co-authored-by: JamesRezo <jamesrezo@noreply.git.spip.net>
Co-committed-by: JamesRezo <jamesrezo@noreply.git.spip.net>
- 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)
Notice PHP en moins lorsqu'on calcule le titre d'un objet, mais que celui ci est vide : cela appelle historiquement
un champ 'surnom' (qui était utilisé dans la table document il fut un temps (fichier AS surnom).
Je laisse cette possibilité, même si ce n'est plus utilisé sur la zone (medias utilise maintenant
en déclaration de titre : `CASE WHEN length(titre)>0 THEN titre ELSE fichier END as titre, '' AS lang`
pour mettre en titre 'fichier' si un titre de document n'est pas présent).
+phpdoc.
Cette fonction peut etre utilisee pour encapsuler ce contenu dans un markup qui l'indique comme non editable dans un editeur wysiwyg et qui porte la trace du raccourci source pour permettre la conversion html=>raccourcis SPIP
Dans ce cas, renvoyer un <html></html> encapsulant est pour le moins incongru, même si en général safehtml nettoie cela.
Le problème apparait notamment si on veut faire un autre traitement sur la balise.
le message de r13858
C'est a sql_quote de se debrouiller pour deviner si il doit ecrire '' ou 0 quand il reçoit un null si jamais son SQL ne tolere pas l'un a la place de l'autre.
- ajout d'un champ virtuel dans la table articles
- virtuel<>'' = redirection
- evite la perte du chapo lors d'une redirection temporaire ...
- suppression des fonctions inutiles chapo_redirigetil et nettoyer_chapo
- suppression de la balise calculée #CHAPO puisque le chapo redevient un champ normal
- la fonction chapo_redirige est renommee virtuel_redirige et renvoit une chaine vide si pas de redirection
- upgrade de la base
- modification de la version_code pour mettre a jour les squelettes compilés
Au passage on generalise la fonctionnalite pour qu'elle s'applique a tout objet traductible (qui a un champ id_trad et un champ lang)
On peut donc ecrire [{}->rub2] si on a active les traductions de rubriques
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;,
}}}
Cerdic
Committo,Ergo:sum