tu as mélangé une modification de l'indentation et tes propres modifs dans ce commit unique, ce qui fait que le diff est illisible, on ne peut pas lire ce que tu as modifié (sauf manip acrobatique).
C'est vraiment à éviter, il faut faire deux commits séparés dans ces cas là : formatage PSR > commit, modifs de code > commit.
Ah je suis désolé. Je ne sais pas bien ce qui a provoqué ça.
En fait
- Je croyais corriger un bug à l'origine du signalement fait sur contrib
- mais c'était une erreur de ma part
- et j'ai annnulé par la suite presqu'aussitôt.
À noter que cette fonction a évolué dans phpCake depuis le temps
alors si jamais il y a un pb dedans il est peut être réglé par la nouvelle version
(mais pour compat spip, il faudra garder l'ancien nommage des options).
Puis je faire qqchose pour améliorer ça ?
(il va peut être me falloir des détails)
Il n'est pas interdit de reformater le code source selon les règles de SPIP avec son éditeur ou un linter (tabulation, etc.), au contraire même, mais dans ce cas, on fait juste le reformatage, rien de plus, et on commite tout de suite avec un message du genre "Reformatage du code".
Ensuite, on fait ses modifs de code dans d'autres commits.