[Fix] various fixes and better error handling

commands/music/back.js

@@ -7,8 +7,8 @@ module.exports = {
     .setDescription("plays previous track from the queue"),
 
   async execute(interaction, client) {
-    const queue = client.player.getQueue(interaction.guild);
     await interaction.deferReply();
+    const queue = client.player.getQueue(interaction.guild);
 
     if (!queue || !queue.playing) {
       return await interaction.followUp({
@@ -27,14 +27,21 @@ module.exports = {
       );
     }
 
-    await queue.back();
-    await interaction.followUp({
-      embeds: [
-        embedMessage(
-          "#9dcc37",
-          `Playing Previous Track **${queue.previousTracks[0].title}**, [<@${interaction.user.id}>]`
-        ),
-      ],
-    });
+    try {
+      await queue.back();
+      await interaction.followUp({
+        embeds: [
+          embedMessage(
+            "#9dcc37",
+            `Playing Previous Track **${queue.previousTracks[0].title}**, [<@${interaction.user.id}>]`
+          ),
+        ],
+      });
+    } catch (error) {
+      client.logger(error.message, "error");
+      await interaction.followUp({
+        embeds: [embedMessage("#9dcc37", "Could not play the previous track")],
+      });
+    }
   },
 };

commands/music/clear.js

@@ -7,8 +7,8 @@ module.exports = {
     .setDescription("clears the music queue"),
 
   async execute(interaction, client) {
-    const queue = client.player.getQueue(interaction.guild);
     await interaction.deferReply();
+    const queue = client.player.getQueue(interaction.guild);
 
     if (!queue) {
       return await interaction.followUp({
@@ -20,6 +20,7 @@ module.exports = {
         ],
       });
     }
+
     try {
       if (queue) {
         await queue.clear();
@@ -33,7 +34,15 @@ module.exports = {
         });
       }
     } catch (err) {
-      console.error(err);
+      client.logger(err.message, "error");
+      await interaction.followUp({
+        embeds: [
+          embedMessage(
+            "#9dcc37",
+            "Could not clear the queue, maybe there is no queue"
+          ),
+        ],
+      });
     }
   },
 };

commands/music/disconnect.js

@@ -7,8 +7,8 @@ module.exports = {
     .setDescription("disconnects from the channel"),
 
   async execute(interaction, client) {
-    const queue = client.player.getQueue(interaction.guild);
     await interaction.deferReply();
+    const queue = client.player.getQueue(interaction.guild);
 
     if (!queue) {
       return await interaction.followUp({
@@ -31,7 +31,15 @@ module.exports = {
         });
       }
     } catch (err) {
-      console.error(err);
+      client.logger(err.message, "error");
+      await interaction.followUp({
+        embeds: [
+          embedMessage(
+            "#9dcc37",
+            "Could not disconnect the bot, Maybe you do not have permission ?"
+          ),
+        ],
+      });
     }
   },
 };

commands/music/lyrics.js

@@ -69,7 +69,7 @@ module.exports = {
           ),
         ],
       });
-      console.error(error);
+      client.logger(error.message, "error");
     }
   },
 };

commands/music/nowPlaying.js

@@ -7,8 +7,8 @@ module.exports = {
     .setDescription("shows the current music name"),
 
   async execute(interaction, client) {
-    const queue = client.player.getQueue(interaction.guild);
     await interaction.deferReply();
+    const queue = client.player.getQueue(interaction.guild);
 
     if (!queue || !queue.playing) {
       return await interaction.followUp({

commands/music/pause.js

@@ -7,8 +7,8 @@ module.exports = {
     .setDescription("pauses the song"),
 
   async execute(interaction, client) {
-    const queue = client.player.getQueue(interaction.guild);
     await interaction.deferReply();
+    const queue = client.player.getQueue(interaction.guild);
 
     if (!queue || !queue.playing)
       return await interaction.followUp({
@@ -30,7 +30,10 @@ module.exports = {
         });
       }
     } catch (err) {
-      console.error(err);
+      client.logger(err.message, "error");
+      await interaction.followUp({
+        embeds: [embedMessage("#9dcc37", "Could not pause the song")],
+      });
     }
   },
 };

commands/music/play.js

@@ -44,7 +44,6 @@ module.exports = {
       });
 
     const queue = client.player.createQueue(interaction.guildId, {
-      metadata: interaction,
       leaveOnEnd: false,
       leaveOnStop: true,
       initialVolume: 80,
@@ -117,7 +116,7 @@ module.exports = {
               embeds: [musicEmbed],
             });
       } catch (err) {
-        console.log(err);
+        client.logger(err.message, "error");
         await interaction.followUp(
           "There was an error playing this song, please try again"
         );

commands/music/resume.js

@@ -7,8 +7,8 @@ module.exports = {
     .setDescription("resumes the song"),
 
   async execute(interaction, client) {
-    const queue = client.player.getQueue(interaction.guild);
     await interaction.deferReply();
+    const queue = client.player.getQueue(interaction.guild);
 
     if (!queue || !queue.playing)
       return await interaction.followUp({
@@ -30,7 +30,10 @@ module.exports = {
         });
       }
     } catch (err) {
-      console.error(err);
+      client.logger(err.message, "error");
+      await interaction.followUp({
+        embeds: [embedMessage("#9dcc37", "I was not able to resume this song")],
+      });
     }
   },
 };

commands/music/shuffle.js

@@ -15,15 +15,22 @@ module.exports = {
         embeds: [embedMessage("#9dcc37", `❌ | There is no queue to shuffle!`)],
       });
 
-    if (queue) {
-      await queue.shuffle();
+    try {
+      if (queue) {
+        await queue.shuffle();
+        await interaction.followUp({
+          embeds: [
+            embedMessage(
+              "#9dcc37",
+              `✅ Queue has been shuffled [<@${interaction.user.id}>]`
+            ),
+          ],
+        });
+      }
+    } catch (error) {
+      client.logger(error.message, "error");
       await interaction.followUp({
-        embeds: [
-          embedMessage(
-            "#9dcc37",
-            `✅ Queue has been shuffled [<@${interaction.user.id}>]`
-          ),
-        ],
+        embeds: [embedMessage("#9dcc37", "Could not shuffle the queue")],
       });
     }
   },

commands/music/skip.js

@@ -33,7 +33,10 @@ module.exports = {
         ],
       });
     } catch (err) {
-      console.error(err);
+      client.logger(err.message, "error");
+      await interaction.followUp({
+        embeds: [embedMessage("#9dcc37", "Song could not be skipped")],
+      });
     }
   },
 };

commands/music/stop.js

@@ -23,6 +23,7 @@ module.exports = {
 
     try {
       if (queue && queue.playing) {
+        await queue.stop();
         await interaction.followUp({
           embeds: [
             embedMessage(
@@ -31,9 +32,9 @@ module.exports = {
             ),
           ],
         });
-        await queue.stop();
       }
     } catch (err) {
+      client.logger(err.message, "error");
       await interaction.followUp({
         embeds: [embedMessage("#9dcc37", `❌ | Something went wrong :/`)],
       });