Interface de l'historique des versions

Bonjour,

Je regardais l'historique des versions d'un tutoriel et j'ai constaté plusieurs choses.

• Sur la page d'historique, la date s'affiche en anglais.
• Sur l'interface Diff, les noms des nouveaux fichiers n'apparaissent pas.
• Sur l'interface Diff, on pourrait clarifier quelle version est à gauche et à droite.
• Sur l'interface Diff, j'ai l'impression qu'on compare la version actuelle à la version avant le commit, mais pas les modifications apportées par le commit. Il serait plus logique de voir ce qu'un commit a modifié, ou de laisser le choix à l'utilisateur des versions à comparer (par exemple pour voir les modifications apportées entre deux mises en validation).

Effectivement, l'historique des versions d'un tutoriel/article a grandement besoin d'être amélioré !

Je confirme, et je sais de quoi je parle, c'est carrément moche et pas pratique. Je t'avoue que j'ai un peu foutu ça au placard en essayant de limiter le nombre d'évolutions introduites par la ZEP-12, mais typiquement, c'est un truc sur lequel j'aimerai me repencher quand j'aurai du temps. Par exemple, ça implique d'utiliser convenablement difflib, et de plus utiliser la sortie "HTML"

Oui, les soucis de l'historique ont déjà été abordés dans ce sujet. Pierre_24 avait d'ailleurs dit qu'il créerait un ticket à ce sujet, mais je ne sais pas ce que c'est devenu.

Il faudrait peut-être en faire une ZEP pour lui donner une meilleure visibilité ?

J'ai du oublier

Du coup, j'ai fait ça de suite, merci de me l'avoir rappelé.

En ce qui concerne une ZEP … Si tu veux, mais je ne pense pas que ça soit nécessaire d'aller à ce point là (sauf si ça vous amuse). On peut par contre discuter de possibilités d'évolutions ici puis les remonter sur GH

En fait, la comparaison se fait non pas entre la version sélectionnée et la version actuelle, mais entre la version sélectionnée et l'avant-dernière version : lien Github

214

tdiff = hcommit.diff("HEAD~1")

A priori HEAD~1 désigne l'avant-dernier commit.

Même pas tout à fait certain, quand je vois les résultats que ça sort, mais une chose est sur, c'est pas correct.

Si, c'est bien une comparaison de la version sélectionnée avec l'avant-dernier commit. D'ailleurs, si vous cliquez sur le lien de l'avant-dernier commit, le diff est systématiquement vide.

Oui c'est bien ce que j'avais remarqué en pratique. D'ailleurs le diff du dernier commit est à l'envers du coup.

Bien vu. Je vais faire un ou deux tests approfondis, pour voir.

Pour les premiers faut que tu les compare avec le "git's semi-secret empty tree". Si tu passe sur ce code, tdiff = hcommit.diff("HEAD~1") ne marche pas, si c'est le premier commit.

J'ai fait un PR, voir le sujet suivant. Merci de l'astuce, j'ai corrigé le code pour que le premier commit soit comparable à cet arbre là