Merge branch 'main' into betterLogs

Author: Kathund

.eslintrc

@@ -12,7 +12,8 @@
   "globals": {
     "bot": true,
     "client": true,
-    "guild": true
+    "guild": true,
+    "imgurUrl": true
   },
   "rules": {
     "curly": ["warn", "multi-line", "consistent"],

src/discord/DiscordManager.js

@@ -22,6 +22,7 @@ class DiscordManager extends CommunicationBridge {
   }
 
   connect() {
+    global.imgurUrl = "";
     global.client = new Client({
       intents: [GatewayIntentBits.Guilds, GatewayIntentBits.GuildMessages, GatewayIntentBits.MessageContent],
     });
@@ -108,6 +109,10 @@ class DiscordManager extends CommunicationBridge {
       Logger.errorMessage(`Channel ${chat} not found!`);
       return;
     }
+    if (username === bot.username && message.endsWith("Check Discord Bridge for image.")) {
+      channel.send(imgurUrl);
+      imgurUrl = "";
+    }
 
     switch (mode) {
       case "bot":

src/discord/commands/blacklistCommand.js

@@ -3,7 +3,7 @@ const { SuccessEmbed } = require("../../contracts/embedHandler.js");
 
 module.exports = {
   name: "blacklist",
-  description: "Demotes the given user by one guild rank.",
+  description: "Ignore add or remove the given user.",
   moderatorOnly: true,
   requiresBot: true,
   options: [

src/discord/commands/updateCommand.js

@@ -27,6 +27,7 @@ module.exports = {
 
       if (user !== undefined) {
         interaction.user = user;
+        interaction.member = await guild.members.fetch(interaction.user.id);
       }
 
       if (!interaction.member) {
@@ -47,7 +48,7 @@ module.exports = {
           }
 
           if (interaction.member.roles.cache.has(role)) {
-            interaction.member.roles.remove(role, "Updated Roles");
+            await interaction.member.roles.remove(role, "Updated Roles");
           }
         }
 
@@ -57,7 +58,7 @@ module.exports = {
       }
 
       if (!interaction.member.roles.cache.has(config.verification.verifiedRole)) {
-        interaction.member.roles.add(config.verification.verifiedRole, "Updated Roles");
+        await interaction.member.roles.add(config.verification.verifiedRole, "Updated Roles");
       }
 
       const [hypixelGuild, player] = await Promise.all([
@@ -71,29 +72,29 @@ module.exports = {
 
       const guildMember = hypixelGuild.members.find((m) => m.uuid === uuid);
       if (guildMember) {
-        interaction.member.roles.add(config.verification.guildMemberRole, "Updated Roles");
+        await interaction.member.roles.add(config.verification.guildMemberRole, "Updated Roles");
 
         if (config.verification.ranks.length > 0 && guildMember.rank) {
           const rank = config.verification.ranks.find((r) => r.name.toLowerCase() == guildMember.rank.toLowerCase());
           if (rank) {
             for (const role of config.verification.ranks) {
               if (interaction.member.roles.cache.has(role.role)) {
-                interaction.member.roles.remove(role.role, "Updated Roles");
+                await interaction.member.roles.remove(role.role, "Updated Roles");
               }
             }
 
-            interaction.member.roles.add(rank.role, "Updated Roles");
+            await interaction.member.roles.add(rank.role, "Updated Roles");
           }
         }
       } else {
         if (interaction.member.roles.cache.has(config.verification.guildMemberRole)) {
-          interaction.member.roles.remove(config.verification.guildMemberRole, "Updated Roles");
+          await interaction.member.roles.remove(config.verification.guildMemberRole, "Updated Roles");
         }
 
         if (config.verification.ranks.length > 0) {
           for (const role of config.verification.ranks) {
             if (interaction.member.roles.cache.has(role.role)) {
-              interaction.member.roles.remove(role.role, "Updated Roles");
+              await interaction.member.roles.remove(role.role, "Updated Roles");
             }
           }
         }
@@ -130,14 +131,14 @@ module.exports = {
           skywarsWLRatio: player.stats.skywars.WLRatio,
           skywarsPlayedGames: player.stats.skywars.playedGames,
 
-          duelsTitle: player.stats.duels.division,
-          duelsKills: player.stats.duels.kills,
-          duelsDeaths: player.stats.duels.deaths,
-          duelsKDRatio: player.stats.duels.KDRatio,
-          duelsWins: player.stats.duels.wins,
-          duelsLosses: player.stats.duels.losses,
-          duelsWLRatio: player.stats.duels.WLRatio,
-          duelsPlayedGames: player.stats.duels.playedGames,
+          duelsTitle: player.stats?.duels?.division || 0,
+          duelsKills: player.stats?.duels?.kills || 0,
+          duelsDeaths: player.stats?.duels?.deaths || 0,
+          duelsKDRatio: player.stats?.duels?.KDRatio || 0,
+          duelsWins: player.stats?.duels?.wins || 0,
+          duelsLosses: player.stats?.duels?.losses || 0,
+          duelsWLRatio: player.stats?.duels?.WLRatio || 0,
+          duelsPlayedGames: player.stats?.duels?.playedGames || 0,
 
           level: player.level,
           rank: player.rank,

src/discord/commands/verifyCommand.js

@@ -63,7 +63,7 @@ module.exports = {
         throw new HypixelDiscordChatBridgeError("This player does not have a Discord linked.");
       }
 
-      if (discordUsername !== interaction.user.username && bypassChecks !== true) {
+      if (discordUsername?.toLowerCase() != interaction.user.username && bypassChecks !== true) {
         throw new HypixelDiscordChatBridgeError(
           `The player '${nickname}' has linked their Discord account to a different account ('${discordUsername}').`,
         );
@@ -93,7 +93,7 @@ module.exports = {
         throw new HypixelDiscordChatBridgeError("The update command does not exist. Please contact an administrator.");
       }
 
-      await updateRolesCommand.execute(interaction);
+      await updateRolesCommand.execute(interaction, user);
     } catch (error) {
       errorMessage(error);
       // eslint-disable-next-line no-ex-assign

src/minecraft/commands/auctionHouseCommand.js

@@ -91,7 +91,8 @@ class AuctionHouseCommand extends minecraftCommand {
         string += string === "" ? upload.data.link : " | " + upload.data.link;
       }
 
-      this.send(`/gc ${`${username}'s Active Auctions: ${string}`}`);
+      imgurUrl = string;
+      this.send(`/gc $${username}'s Active Auctions: Check Discord Bridge for image.`);
     } catch (error) {
       errorMessage(error);
       this.send(`/gc [ERROR] ${error}`);

src/minecraft/commands/chickenCommand.js

@@ -25,7 +25,8 @@ class ChickenCommand extends minecraftCommand {
         throw "An error occured while fetching the image. Please try again later.";
       }
 
-      this.send(`/gc Funny Chicken: ${data.url}`);
+      imgurUrl = data.url;
+      this.send('/gc Funny Chicken: Check Discord Bridge for image.');
     } catch (error) {
       this.send(`/gc [ERROR] ${error ?? "Something went wrong.."}`);
     }

src/minecraft/commands/dinosaurCommand.js

@@ -25,7 +25,8 @@ class DinosaurCommand extends minecraftCommand {
         throw "An error occured while fetching the image. Please try again later.";
       }
 
-      this.send(`/gc Funny dino: ${data.url}`);
+      imgurUrl = data.url;
+      this.send("/gc Funny dino: Check Discord Bridge for image.");
     } catch (error) {
       this.send(`/gc [ERROR] ${error ?? "Something went wrong.."}`);
     }

src/minecraft/commands/duckCommand.js

@@ -24,8 +24,8 @@ class DuckCommand extends minecraftCommand {
         // eslint-disable-next-line no-throw-literal
         throw "An error occured while fetching the image. Please try again later.";
       }
-
-      this.send(`/gc Funny Duck: ${data.url}`);
+      imgurUrl = data.url;
+      this.send("/gc Funny Duck: Check Discord Bridge for image.");
     } catch (error) {
       this.send(`/gc [ERROR] ${error ?? "Something went wrong.."}`);
     }

src/minecraft/commands/kittyCommand.js

@@ -24,7 +24,8 @@ class KittyCommand extends minecraftCommand {
       const link = data[0].url;
       const upload = await uploadImage(link);
 
-      this.send(`/gc Cute Cat: ${upload.data.link}`);
+      imgurUrl = upload.data.link;
+      this.send(`/gc Cute Cat: Check Discord Bridge for image.`);
     } catch (error) {
       this.send(`/gc [ERROR] ${error ?? "Something went wrong.."}`);
     }