DraftBot-A-Discord-Adventure / DraftBot

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

[BUG]: crash de guilddaily #1339

Closed romain22222 closed 2 years ago

romain22222 commented 2 years ago

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

Description du bug.

[2022/08/10 17:30:06] Error: Erreur generateRandomProperty : nombre property invalide at generateRandomProperty (/home/draftbot/DraftBot-A-Discord-Adventure/dist/src/commands/guild/GuildDailyCommand.js:232:11) at Object. (/home/draftbot/DraftBot-A-Discord-Adventure/dist/src/commands/guild/GuildDailyCommand.js:265:94) at Generator.next () at fulfilled (/home/draftbot/DraftBot-A-Discord-Adventure/dist/src/commands/guild/GuildDailyCommand.js:5:58)

Effectué lors d'un daily

Reproduction du bug.

1- faire un guilddaily 2- boum ca crash

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

Sur un serveur perso avec le bot officiel.

BastLast commented 2 years ago

Fuck

Pietagorh commented 2 years ago

C'est pas tous les gds, j'en ai eu un à 16h11

BastLast commented 2 years ago

du coup j'ai pas compris dans quel cas ça crash @romain22222 ?

romain22222 commented 2 years ago

ca crash soit sur un type de gd, mais aucune idée duquel, soit c'est avec une certaine proba, et dans ce cas c'est la gestion des probas qui va pas

romain22222 commented 2 years ago

et si c'est un type de gd, c'est un des plus rares

romain22222 commented 2 years ago

au niveau 100 je précise

BastLast commented 2 years ago

Ce bug a l'air super peu fréquent et on a pas assez dinfos pour le fixer vite alors je le decale d'une mise à jour

Pietagorh commented 2 years ago

Hum, on l'avait pas fix ça ? De mémoire c'était juste qu'il manquait 50 dans la répartition des probas

BastLast commented 2 years ago

Hmm pourquoi l'issue a pas été tag ?

Chevalier2 commented 2 years ago

Hum, on l'avait pas fix ça ? De mémoire c'était juste qu'il manquait 50 dans la répartition des probas

Je confirme qu'il manque 50, je viens de vérifier.

BastLast commented 2 years ago

C'est toujours le cas sur develop ?

Chevalier2 commented 2 years ago

Non, c'est fix sur develop.

BastLast commented 2 years ago

Ok quel commit ?

Chevalier2 commented 2 years ago

image

BastLast commented 2 years ago

Tu as le lien ? Xd

Chevalier2 commented 2 years ago

https://github.com/DraftBot-A-Discord-Adventure/DraftBot/commit/a9af783ae4eeacd151df052edc6a0d7ba4831782 money passe de 350 à 400.

BastLast commented 2 years ago

Merci, je close du coup