Bug validation alertes - Githubissues

nlenglet-ign commented 1 year ago

Bug constaté sur master, commit 365e14c0a62922fff112b451bf73f1cd6fc13761

Etapes à reproduire pour faire apparaître le bug :

créer une branche sur un cache quelconque
saisir une alerte et la valider (état 1), puis la supprimer
saisir une nouvelle alerte (impossible de la sélectionner en faisant défiler avec les flèches ou en cliquant dessus). Pour pouvoir la sélectionner, il faut appuyer sur 'c' (raccourci pour valider une alerte), on se retrouver alors dans l'état 2, avec -1 alerte validée.
quand on valide effectivement cette alerte, on se retrouve alors avec 0 alerte validée (état 3)

(1) bug_alertes_1 (2) bug_alertes_2 (3) bug_alertes_3

ftoromanoff commented 1 year ago

J'ai l'impression que ce n'est pas lié au fait de valider ou non l'alerte mais juste a cause de la suppression d'une alerte unique.

amrosu commented 1 year ago

Ce bug semble avoir été introduit en septembre avec la branche ClassAlert (PR #324) mise dans master -> v1.4.4 (7990db38) Il y a un problème sur la gestion de la validation : la nouvelle alerte, ajoutée après la suppression, est validée par défaut avec Progress à 0/1 (0 validés) et quand on refait 'validated [c]' on a l'état 2 décrit plus haut avec Progress à 0/1 (-1 validés)

ign-packo / PackO

Bug validation alertes #368