DraftBot-A-Discord-Adventure / DraftBot

A Discord adventure bot that speaks english and french
MIT License
229 stars 61 forks source link

[BUG]: La mission havePotions ne se valide pas quand il le faut #1560

Closed Nargacaura closed 2 years ago

Nargacaura commented 2 years ago

Description du bug.

Visiblement, la mission havePotions ne veut pas valider, même si on a 2 potions sur soi... mais il se peut aussi qu'elle se valide alors qu'on a bu la seconde potion avant de faire la mission précédente!

Selon vous, quelle est l'importance de ce bug ?

😨 Grave

Reproduction du bug.

Se mettre sur la mission 30 de la campagne sans l'avoir automatiquement complétée dans le bot de tests. Récupérer 2 potions. OU Boire une potion de vie ou de temps avec une autre potion dans l'inventaire. Faire la mission précédente. Voir sa mission validée avec une seule potion.

Captures d'écran

Cas de test (mission non validée avec 2 potions):

Capture d’écran 2022-11-01 à 19 04 00 Capture d’écran 2022-11-01 à 19 04 09

Cas de réalisation de la mission précédente (boire une potion de vie/temps, puis faire les 10 fights tout en laissant l'autre potion dans sa réserve):

Capture d’écran 2022-11-01 à 19 23 14 Capture d’écran 2022-11-01 à 19 23 21

NOTE: j'ai bu une potion de vie avant de faire la mission des 10 combats.

Sur quel shard avez vous rencontré le bug ?

🤖 bot de test 'officiel' (alpha ou bêta)

romain22222 commented 2 years ago

alors étonnamment, j'ai aucune idée la comme ca

voltou91 commented 2 years ago

Ce serait pas parce qu'il compte que les items de l'inventaire ? Donc pour lui y a qu'une potion. Et de ce que je voit quand le mec boit une potion, on compte le nombre de potions dans son inventaire, mais c'est la potion du profil qui est bu.

Nargacaura commented 2 years ago

Ce serait pas parce qu'il compte que les items de l'inventaire ? Donc pour lui y a qu'une potion. Et de ce que je voit quand le mec boit une potion, on compte le nombre de potions dans son inventaire, mais c'est la potion du profil qui est bu.

Sauf que la potion a été bue avant même de faire les 10 combats, du coup, j'ai plus l'impression qu'il ne fait que checker s'il y a une deuxième potion dans l'inventaire, mais juste quand il ne faut pas x)

romain22222 commented 2 years ago

@Nargacaura 1 - Comment as tu obtenu ta 2nde potion ? 2 - On est d'accord que t'as validé la mission en faisant un fight en ayant 2 potions sur toi ?

Nargacaura commented 2 years ago

@Nargacaura 1 - Comment as tu obtenu ta 2nde potion ? 2 - On est d'accord que t'as validé la mission en faisant un fight en ayant 2 potions sur toi ?

  1. Via le shop (les 2, d'ailleurs), plus précisément en choisissant l'équipement aléatoire.
  2. Même pas, j'ai bu la première potion avant même de démarrer le premier des 10 combats.
BastLast commented 2 years ago

@voltou91 je déplace en 4.1.0 du coup ?

voltou91 commented 2 years ago

@voltou91 je déplace en 4.1.0 du coup ?

Comme tu veux, je vais push dans quelques minutes.

BastLast commented 2 years ago

bah je vois pas l'interet d'attendre alors go, je te laisse faire la pr

voltou91 commented 2 years ago

A l'évidence elle est tjr pété T-T

QQtin3 commented 2 years ago

Shop + Fight apparemment (à étudier tho)

Nargacaura commented 2 years ago

Pour info: La mission est assez bizarre à accomplir. En cas de fight, tout est bon (gagner le 10è fight avec 1 potion sur soi et 1 dans la réserve). Cependant, si un joueur n'a eu qu'une seule potion lors de l'accomplissement de la mission des fights, la potion qu'on avait n'est pas comptée (même si elle a été achétee via le shop!)... De ce fait, racheter une autre potion n'accomplit pas cette mission: elle compte comme étant 1 potion sur 2. Remplacer la potion ne fonctionne pas, et pour l'accomplir, il faut:

voltou91 commented 2 years ago

Ok j'ai compris le problème X) En gros on check les potions de ton inventaire avant de te la give. Du coup il compte une potion puis te give ta 2e potion.