DraftBot icon indicating copy to clipboard operation
DraftBot copied to clipboard

[BUG]: affichage temps négatif

Open KirIcare opened this issue 2 years ago • 4 comments

Le bug n'a t-il pas déjà été signalé ?

  • [X] J'ai vérifié et je n'ai pas trouvé d'issue pour ce problème.

Description du bug.

Il m'est arrivé à deux reprises que le bot m'affiche un temps négatif (-1h59) au moment du /report pour faire mon évent. J'ai repéré quelques similitudes entre les 2 fois:

  1. les deux trajets d'après étaient des trajets de 2h
  2. j'ai fais la même séquence de commandes décrite plus bas Voilà, j'ai pensé qu'il s'agissait de l'équivalent de l'affichage "0min0sec" quand on fais un mini évent mais version évent et comme le compteur n'affiche jamais "00h00" (parcequ'il peut pas rester 0sec de trajet, sinon t'es arrivé) il compense en affichant le temps en négatif. Ci-joint les screens des 2 fois en questions IMG_20220814_221941 IMG_20220814_215623

Reproduction du bug.

(pas sur que ça marche, c'est juste la manipulation qui semble le provoquer mais aucune preuve) -faire plusieurs rapport à 00h01 de l'arrivée Exemple: -faire un rapport à 21h00 où il est indiqué qu'il reste moins d'une minute de trajet -faire un rapport à 21h01 avec toujours le même message -faire un rapport à 21h02 (donc la minute est théoriquement passé) et le temps négatif s'affiche au lieu de l'évent -(facultatif) faire un rapport, même quelques secondes après affiche l'évent

Dans quelle condition le bug a-t-il été trouvé ?

Sur le serveur officiel du bot avec le bot officiel.

KirIcare avatar Aug 14 '22 20:08 KirIcare

C'est fix ça @romain22222 ?

Feiryn avatar Sep 08 '22 17:09 Feiryn

faudrait retester afin d'être sur, mais je pense fix avec la #820

romain22222 avatar Sep 08 '22 18:09 romain22222

Le truc c'est que je le fais pas volontairement et comme je sais pas ce qui le cause, je pourrais pas savoir s'il aura été fix ou pas... Quand je vais essayer de le faire, je pourrais pas savoir si soit j'arrive pas à la faire, soit il est pas fix😔

KirIcare avatar Sep 08 '22 18:09 KirIcare

Ce ne sera corrigé que dans la prochaine mise à jour dans tous les cas

Feiryn avatar Sep 08 '22 18:09 Feiryn

Alors j'ai réussi à reproduire le bug, c'est assez précis tout de même. Si on fait un /report quelques millisecondes avant la fin du voyage, la commande report voit qu'il faut envoyer l'embed du trajet. Dans la fonction de construction de l'embed, quelques millisecondes se sont écoulées et donc le temps de voyage est dépassé de quelques millisecondes. À cause d'un calcul (je vais voir lequel), le temps est affiché négativement.

Pour info j'ai ajouté cette ligne avant le sendTravelPath à la fin de la commande report : await Maps.advanceTime(entity.Player, millisecondsToMinutes(minutesToMilliseconds((await MapLinks.getById(entity.Player.mapLinkId)).tripDuration) - Maps.getTravellingTime(entity.Player)), NumberChangeReason.TEST);

Feiryn avatar Sep 29 '22 14:09 Feiryn

Ahhh merci d'avoir enfin trouvé d'où ça venais (j'étais curieux et puis comme personne d'autre arrivais à le faire, on était pas sorti de l'auberge) En gros y'a un certain timing que j'ai eu à deux reprises. En vrai si c'est si précis que ça pour le faire, c'était pas le bug le plus urgent à corrigé je suppose, il dérangeait personne mdr

KirIcare avatar Sep 29 '22 14:09 KirIcare