cs-autofixes #4868

Merged
JamesRezo merged 5 commits from cs-autofixes into master 1 year ago
JamesRezo commented 1 year ago
Owner
  • 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
JamesRezo added 5 commits 1 year ago
b_b commented 1 year ago
Owner

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.
JamesRezo changed title from WIP:cs-autofixes to cs-autofixes 1 year ago
b_b approved these changes 1 year ago
Owner

Super :)
Tout ça me semble très bien.

Super :) Tout ça me semble très bien.
Owner

Bon ya trop de modifs, j'arrive même pas à afficher la page pour faire une vraie révision sur mon ordi, mais super :)

Bon ya trop de modifs, j'arrive même pas à afficher la page pour faire une vraie révision sur mon ordi, mais super :)
JamesRezo merged commit 7fa313e1a4 into master 1 year ago
JamesRezo referenced this issue from a commit 1 year ago
JamesRezo deleted branch cs-autofixes 1 year ago

Reviewers

b_b approved these changes 1 year ago
The pull request has been merged as 7fa313e1a4.
Sign in to join this conversation.
Loading…
There is no content yet.