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
* 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
Super, pour la passe 2, je trouve étrange le diff du second fichier, cf :
if ($redirect = _request('redirect') and !$arg == sql_getfetsel('email', 'spip_auteurs',
qui devient
if (
$redirect = _request('redirect') and !$arg == sql_getfetsel('email', 'spip_auteurs',
Étonnant que le saut de ligne ne se fasse pas après le and, mais ça semble être le fonctionnement du fixer en question, qui se déclenche uniquement car il y a un saut de ligne dans le if et donc passe ce qui suit la ( à la ligne. Au final, c'est de notre faute car on posait des sauts de lignes dans des appels de fonction dans les conditions, on aura tout le temps de reprendre ce genre de cas à la main plus tard.
Super, pour la passe 2, je trouve étrange le diff du second fichier, cf :
`if ($redirect = _request('redirect') and !$arg == sql_getfetsel('email', 'spip_auteurs',`
qui devient
```
if (
$redirect = _request('redirect') and !$arg == sql_getfetsel('email', 'spip_auteurs',
```
Étonnant que le saut de ligne ne se fasse pas après le and, mais ça semble être le fonctionnement du fixer en question, qui se déclenche uniquement car il y a un saut de ligne dans le if et donc passe ce qui suit la ( à la ligne. Au final, c'est de notre faute car on posait des sauts de lignes dans des appels de fonction dans les conditions, on aura tout le temps de reprendre ce genre de cas à la main plus tard.
b_b
Super, pour la passe 2, je trouve étrange le diff du second fichier, cf :
if ($redirect = _request('redirect') and !$arg == sql_getfetsel('email', 'spip_auteurs',qui devient
Étonnant que le saut de ligne ne se fasse pas après le and, mais ça semble être le fonctionnement du fixer en question, qui se déclenche uniquement car il y a un saut de ligne dans le if et donc passe ce qui suit la ( à la ligne. Au final, c'est de notre faute car on posait des sauts de lignes dans des appels de fonction dans les conditions, on aura tout le temps de reprendre ce genre de cas à la main plus tard.
WIP:cs-autofixesto cs-autofixes 1 year agoSuper :)
Tout ça me semble très bien.
Bon ya trop de modifs, j'arrive même pas à afficher la page pour faire une vraie révision sur mon ordi, mais super :)
7fa313e1a4into master 1 year agoReviewers
7fa313e1a4.