Question

129

Discord.js ban/kick commands available to all users. How can I fix this?

rated 0 times [ 130] [ 1] / answers: 1 / hits: 103961 / 5 Years ago, tue, february 26, 2019, 12:00:00

I'm making my own Discord Bot because I don't trust the bigger ones (Dyno, Hime, NosoBot, etc.) And my bot is pretty much done. The only problem is that my code allows all members to use these commands. I only want people to be able to use the functions they have permissions to. The code works, but how can I make it allow only people with permission to kick/ban?

if (msg.content.startsWith("$kick ")) {

    if (msg.mentions.members.first()) {

        msg.mentions.members.first.kick().then((member) => {

            msg.channel.send(":wave: " + member.displayName + " has been successfully kicked :point_right: ");

        }).catch(() => {

            msg.channel.send("I do not have permissions to do this");

        });

    }

}else if (msg.content.startsWith("$ban ")) {

   if (!message.member.hasPermission("MANAGE_MESSAGES")) return;

    if (msg.mentions.members.first()) {

        msg.mentions.members.first.ban().then((member) => {

            msg.channel.send(":wave: " + member.displayName + " has been successfully banned :point_right: ");

        }).catch(() => {

            msg.channel.send("I do not have permissions to do this");

        });

    }

}

Answers

Only authorized users can answer the question. Please sign in first, or register a free account.

reedmustafam

Add To Favorites

Follow

Total Points: 211

Total Questions: 83

Total Answers: 105

Location: Vanuatu

Member since Wed, Oct 14, 2020

4 Years ago

answered 5 Years ago emiliano · Accepted Answer

The "KICK_MEMBERS" permission tells you if they have the permission to kick members, hence the name.

The "BAN_MEMBERS" permission tells you if they have the permission to ban members, hence the name.

Your Kick Command:

if (msg.member.hasPermission("KICK_MEMBERS")) {

    if (msg.members.mentions.first()) {

        try {

            msg.members.mentions.first().kick();

        } catch {

            msg.reply("I do not have permissions to kick " + msg.members.mentions.first());

        }

    } else {

        msg.reply("You do not have permissions to kick " + msg.members.mentions.first());

    }

}

Your Ban Command:

if (msg.member.hasPermission("BAN_MEMBERS")) {

    if (msg.members.mentions.first()) {

        try {

            msg.members.mentions.first().ban();

        } catch {

            msg.reply("I do not have permissions to ban" + msg.members.mentions.first());

        }

    } else {

        msg.reply("You do not have permissions to ban" + msg.members.mentions.first());

    }

}

The reason for the try and catch ensures that if the bot does not have permissions to kick or ban that user, it will not cause an error.

Another note:

You do not have to create another bot.on('message') event. Instead just use an elseif