search

DraftBot-A-Discord-Adventure / DraftBot

A Discord adventure bot that speaks english and french
MIT License
222 stars 60 forks source link

[BUG]:évènement qui donne argent au lieu de l’enlever #2319

Closed Trolleur17200 closed 10 months ago

Trolleur17200 commented 10 months ago

Description du bug.

A Mergagnan ds la quête ou l’on rencontre Price je n’ai pas répondu je me suis fait tapé mais au lieu de perdre les 250 je les ai gagnés Ce Bug c’est passé le mardi 28/11 a 21h54

Reproduction du bug.

-Aller à Mergagnan -Avoir la quête de Price -ne pas répondre -se faire taper et voler

Captures d'écran

IMG_7743 IMG_7742

Sur quel shard avez vous rencontré le bug ?

3

symsym-1629 commented 10 months ago

selon le code, la monnaie est bien retirée, c'est chelou

https://github.com/DraftBot-A-Discord-Adventure/DraftBot/blob/dd8ace6749b9f2a024094abd71f208a6d04bd797/resources/text/events/39.json#L116

Nargacaura commented 10 months ago

Je viens de vérifier le code et... effectivement, ça donne la valeur absolue du changement, qui est toujours positive, et comme cet event n'a pas de tag spendMoney, ça l'ajoute au lieu de le retirer: https://github.com/DraftBot-A-Discord-Adventure/DraftBot/blob/master/src%2Fcore%2Fevents%2FPossibilityOutcome.ts#L89-L95

symsym-1629 commented 10 months ago

y'a possiblement d'autres events dcp ?

BastLast commented 10 months ago

@romain22222 URGENT c'est pr toi ça

Nargacaura commented 10 months ago

y'a possiblement d'autres events dcp ?

C'est fort probable, voire certain qu'il y en ait d'autres. En gros, tous ceux qui ont un money négatif sans le tag spendMoney sont affectés.

BastLast commented 10 months ago

y'a possiblement d'autres events dcp ?

C'est fort probable, voire certain qu'il y en ait d'autres. En gros, tous ceux qui ont un money négatif sans le tag spendMoney sont affectés.

Le genre de bugs qui me font pleurer de pas avoir de test unitaires