[spip-dev] r125253 - _plugins_/spip-bonux/trunk

Salut Jluc,

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.

Tutafé, et du coup, vu que c'est moi qui l'ai ajouté et que je l'utilise pas mal, jluc tu as fait quoi dessus donc ? :slight_smile:

Comme 99% de bonux, ça devrait pouvoir être ajouté au core, dès la 3.3… ça fait 5 ans c'est là "pour tester" quand même…

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.

Donc au total j'ai fait 3 commits qui se résument au 2eme
(du moins normalement, mais avec en prime des indentations différentes)
https://git.spip.net/spip-contrib-extensions/spip-bonux/commit/fa2020716deff18a3835d1801bc397cea2d93d7c
= actualisation du lien vers la librairie phpCake à l'origine de cette fonction.

À 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)

JLuc

Tu as regardé le diff dans trac ?
https://zone.spip.org/trac/spip-zone/changeset/125253/spip-zone

Simplement, après ton commit il y a deux espaces au début de chaque ligne (même pas le PSR spip en plus).

Du coup, tout le fichier est considéré comme modifié, et si tu as modifié du code quelque part, on ne peut pas le voir.

A moins de copier coller les deux parties, de les reformater et de refaire un diff dessus, ce qui n'est pas très pratique.

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.

Oui je sais pas comment j'ai fait ça mais j'ai très bien compris le pb.

Ce que je demandais c'est si je pouvais réparer ça
à coup de revert rebase merge push -onto ou autre giterie

JL

Le diff sur gitea est un peu moins violent, mais pas beaucoup plus lisible, surtout vers la fin :
https://git.spip.net/spip-contrib-extensions/spip-bonux/commit/e8fa5e8de134fc9ff988073a49b3714894d8d805