Fix some bugs in roll.js

This commit is contained in:
Michael 2022-12-16 23:22:11 -06:00
parent 8fcb1e2e0c
commit aaf220e37d

View file

@ -5,6 +5,7 @@
const { SlashCommandBuilder } = require('discord.js'); const { SlashCommandBuilder } = require('discord.js');
const { randomInt } = require('node:crypto'); const { randomInt } = require('node:crypto');
const { abs } = require('node:math');
module.exports = { module.exports = {
data: new SlashCommandBuilder() data: new SlashCommandBuilder()
@ -56,8 +57,8 @@ module.exports = {
let rollModNota = ''; let rollModNota = '';
// Limit numbers rollable to prevent spam and API issues // Limit numbers rollable to prevent spam and API issues
if (rollCount > 99) { if (rollCount > 100) {
await interaction.reply({ content: `You want ${rollCount} numbers?! That's a bit much, even for me...\nTry 99 or less please!`, ephemeral: true }); await interaction.reply({ content: `You want ${rollCount} numbers?! That's a bit much, even for me...\nTry 100 or less please!`, ephemeral: true });
return; return;
} }
// Prevent 0 or less numbers // Prevent 0 or less numbers
@ -85,8 +86,8 @@ module.exports = {
} }
if (rollMod != 0) { if (rollMod != 0) {
rollModStr = ' ' + modSign + ' ' + rollMod; rollModStr = ' ' + modSign + ' ' + (rollMod);
rollModNota = modSign + rollMod; rollModNota = modSign + abs(rollMod);
} }
const diceRollsString = diceRolls.toString().split(',').join(' + '); const diceRollsString = diceRolls.toString().split(',').join(' + ');