Change authentication message to be an error instead of normal ephemeral message for all faculty commands

VehvilainenPooki · VehvilainenPooki · commit f120436a7083 · 2025-05-23T03:03:32.000+03:00
diff --git a/src/discordBot/commands/faculty/add_instructors.js b/src/discordBot/commands/faculty/add_instructors.js
@@ -8,7 +8,7 @@ const { courseAdminRole, facultyRole } = require("../../../../config.json");
 
 const execute = async (interaction, client, models) => {
   if (!interaction.member.permissions.has("ADMINISTRATOR") && !interaction.member.roles.cache.some(r => r.name === facultyRole)) { // Olisiko parempi tarkistaa tietokannasta eikä discordista?
-    await sendEphemeral(interaction, "You do not have permission to use this command.");
+    await sendErrorEphemeral(interaction, "You do not have permission to use this command.");
     return
   }
 
diff --git a/src/discordBot/commands/faculty/create_channel.js b/src/discordBot/commands/faculty/create_channel.js
@@ -7,7 +7,7 @@ const { facultyRole } = require("../../../../config.json");
 
 const execute = async (interaction, client, models) => {
   if (!interaction.member.permissions.has("ADMINISTRATOR") && !interaction.member.roles.cache.some(r => r.name === facultyRole)) {
-    await sendEphemeral(interaction, "You do not have permission to use this command.");
+    await sendErrorEphemeral(interaction, "You do not have permission to use this command.");
     return
   }
 
diff --git a/src/discordBot/commands/faculty/create_course.js b/src/discordBot/commands/faculty/create_course.js
@@ -8,8 +8,8 @@ const { sendErrorEphemeral, sendEphemeral, editEphemeral } = require("../../serv
 const { facultyRole } = require("../../../../config.json");
 
 const execute = async (interaction, client, models) => {
-if (!interaction.member.permissions.has("ADMINISTRATOR") && !interaction.member.roles.cache.some(r => r.name === facultyRole)) {
-    await sendEphemeral(interaction, "You do not have permission to use this command.");
+  if (!interaction.member.permissions.has("ADMINISTRATOR") && !interaction.member.roles.cache.some(r => r.name === facultyRole)) {
+    await sendErrorEphemeral(interaction, "You do not have permission to use this command.");
     return
   }
 
diff --git a/src/discordBot/commands/faculty/create_poll.js b/src/discordBot/commands/faculty/create_poll.js
@@ -8,7 +8,7 @@ const numbers = [ "1️⃣", "2️⃣", "3️⃣", "4️⃣", "5️⃣", "6️
 
 const execute = async (interaction, client) => {
   if (!interaction.member.permissions.has("ADMINISTRATOR") && !interaction.member.roles.cache.some(r => r.name === facultyRole)) {
-    await sendEphemeral(interaction, "You do not have permission to use this command.");
+    await sendErrorEphemeral(interaction, "You do not have permission to use this command.");
     return
   }
 
diff --git a/src/discordBot/commands/faculty/delete_bridge.js b/src/discordBot/commands/faculty/delete_bridge.js
@@ -8,7 +8,7 @@ const { sendMessageToTelegram } = require("../../../telegramBot/bridge/service")
 
 const execute = async (interaction, client, models) => {
   if (!interaction.member.permissions.has("ADMINISTRATOR") && !interaction.member.roles.cache.some(r => r.name === facultyRole)) {
-    await sendEphemeral(interaction, "You do not have permission to use this command.");
+    await sendErrorEphemeral(interaction, "You do not have permission to use this command.");
     return
   }
 
diff --git a/src/discordBot/commands/faculty/delete_channel.js b/src/discordBot/commands/faculty/delete_channel.js
@@ -8,7 +8,7 @@ const { facultyRole } = require("../../../../config.json");
 
 const execute = async (interaction, client, models) => {
   if (!interaction.member.permissions.has("ADMINISTRATOR") && !interaction.member.roles.cache.some(r => r.name === facultyRole)) {
-    await sendEphemeral(interaction, "You do not have permission to use this command.");
+    await sendErrorEphemeral(interaction, "You do not have permission to use this command.");
     return
   }
 
diff --git a/src/discordBot/commands/faculty/disable_bridge.js b/src/discordBot/commands/faculty/disable_bridge.js
@@ -8,7 +8,7 @@ const { facultyRole } = require("../../../../config.json");
 
 const execute = async (interaction, client, models) => {
   if (!interaction.member.permissions.has("ADMINISTRATOR") && !interaction.member.roles.cache.some(r => r.name === facultyRole)) {
-    await sendEphemeral(interaction, "You do not have permission to use this command.");
+    await sendErrorEphemeral(interaction, "You do not have permission to use this command.");
     return
   }
 
diff --git a/src/discordBot/commands/faculty/edit_course.js b/src/discordBot/commands/faculty/edit_course.js
@@ -67,7 +67,7 @@ const changeCourseNick = async (interaction, client, models, courseName, newValu
 
 const execute = async (interaction, client, models) => {
   if (!interaction.member.permissions.has("ADMINISTRATOR") && !interaction.member.roles.cache.some(r => r.name === facultyRole)) {
-    await sendEphemeral(interaction, "You do not have permission to use this command.");
+    await sendErrorEphemeral(interaction, "You do not have permission to use this command.");
     return
   }
 
diff --git a/src/discordBot/commands/faculty/edit_topic.js b/src/discordBot/commands/faculty/edit_topic.js
@@ -12,7 +12,7 @@ const { saveChannelTopicToDb } = require("../../../db/services/channelService");
 
 const execute = async (interaction, client, models) => {
   if (!interaction.member.permissions.has("ADMINISTRATOR") && !interaction.member.roles.cache.some(r => r.name === facultyRole)) {
-    await sendEphemeral(interaction, "You do not have permission to use this command.");
+    await sendErrorEphemeral(interaction, "You do not have permission to use this command.");
     return
   }
 
diff --git a/src/discordBot/commands/faculty/enable_bridge.js b/src/discordBot/commands/faculty/enable_bridge.js
@@ -8,7 +8,7 @@ const { facultyRole } = require("../../../../config.json");
 
 const execute = async (interaction, client, models) => {
   if (!interaction.member.permissions.has("ADMINISTRATOR") && !interaction.member.roles.cache.some(r => r.name === facultyRole)) {
-    await sendEphemeral(interaction, "You do not have permission to use this command.");
+    await sendErrorEphemeral(interaction, "You do not have permission to use this command.");
     return
   }
 
diff --git a/src/discordBot/commands/faculty/hide_channel.js b/src/discordBot/commands/faculty/hide_channel.js
@@ -6,7 +6,7 @@ const { confirmChoice } = require("../../services/confirm");
 
 const execute = async (interaction, client, models) => {
   if (!interaction.member.permissions.has("ADMINISTRATOR") && !interaction.member.roles.cache.some(r => r.name === facultyRole)) {
-    await sendEphemeral(interaction, "You do not have permission to use this command.");
+    await sendErrorEphemeral(interaction, "You do not have permission to use this command.");
     return
   }
 
diff --git a/src/discordBot/commands/faculty/hide_course.js b/src/discordBot/commands/faculty/hide_course.js
@@ -10,7 +10,7 @@ const { facultyRole } = require("../../../../config.json");
 
 const execute = async (interaction, client, models) => {
   if (!interaction.member.permissions.has("ADMINISTRATOR") && !interaction.member.roles.cache.some(r => r.name === facultyRole)) {
-    await sendEphemeral(interaction, "You do not have permission to use this command.");
+    await sendErrorEphemeral(interaction, "You do not have permission to use this command.");
     return
   }
 
diff --git a/src/discordBot/commands/faculty/lock_chat.js b/src/discordBot/commands/faculty/lock_chat.js
@@ -11,7 +11,7 @@ const { facultyRole } = require("../../../../config.json");
 
 const execute = async (interaction, client, models) => {
   if (!interaction.member.permissions.has("ADMINISTRATOR") && !interaction.member.roles.cache.some(r => r.name === facultyRole)) {
-    await sendEphemeral(interaction, "You do not have permission to use this command.");
+    await sendErrorEphemeral(interaction, "You do not have permission to use this command.");
     return
   }
 
diff --git a/src/discordBot/commands/faculty/remove_instructors.js b/src/discordBot/commands/faculty/remove_instructors.js
@@ -8,7 +8,7 @@ const { courseAdminRole, facultyRole } = require("../../../../config.json");
 
 const execute = async (interaction, client, models) => {
   if (!interaction.member.permissions.has("ADMINISTRATOR") && !interaction.member.roles.cache.some(r => r.name === facultyRole)) {
-    await sendEphemeral(interaction, "You do not have permission to use this command.");
+    await sendErrorEphemeral(interaction, "You do not have permission to use this command.");
     return
   }
 
diff --git a/src/discordBot/commands/faculty/rename_channel.js b/src/discordBot/commands/faculty/rename_channel.js
@@ -8,7 +8,7 @@ const { confirmChoice } = require("../../services/confirm");
 
 const execute = async (interaction, client, models) => {
   if (!interaction.member.permissions.has("ADMINISTRATOR") && !interaction.member.roles.cache.some(r => r.name === facultyRole)) {
-    await sendEphemeral(interaction, "You do not have permission to use this command.");
+    await sendErrorEphemeral(interaction, "You do not have permission to use this command.");
     return
   }
 
diff --git a/src/discordBot/commands/faculty/status.js b/src/discordBot/commands/faculty/status.js
@@ -15,7 +15,7 @@ const { findAllCourseMembers } = require("../../../db/services/courseMemberServi
 
 const execute = async (interaction, client, models) => {
   if (!interaction.member.permissions.has("ADMINISTRATOR") && !interaction.member.roles.cache.some(r => r.name === facultyRole)) {
-    await sendEphemeral(interaction, "You do not have permission to use this command.");
+    await sendErrorEphemeral(interaction, "You do not have permission to use this command.");
     return
   }
 
diff --git a/src/discordBot/commands/faculty/unhide_channel.js b/src/discordBot/commands/faculty/unhide_channel.js
@@ -6,7 +6,7 @@ const { confirmChoice } = require("../../services/confirm");
 
 const execute = async (interaction, client, models) => {
   if (!interaction.member.permissions.has("ADMINISTRATOR") && !interaction.member.roles.cache.some(r => r.name === facultyRole)) {
-    await sendEphemeral(interaction, "You do not have permission to use this command.");
+    await sendErrorEphemeral(interaction, "You do not have permission to use this command.");
     return
   }
 
diff --git a/src/discordBot/commands/faculty/unhide_course.js b/src/discordBot/commands/faculty/unhide_course.js
@@ -10,7 +10,7 @@ const { facultyRole } = require("../../../../config.json");
 
 const execute = async (interaction, client, models) => {
   if (!interaction.member.permissions.has("ADMINISTRATOR") && !interaction.member.roles.cache.some(r => r.name === facultyRole)) {
-    await sendEphemeral(interaction, "You do not have permission to use this command.");
+    await sendErrorEphemeral(interaction, "You do not have permission to use this command.");
     return
   }
 
diff --git a/src/discordBot/commands/faculty/unlock_chat.js b/src/discordBot/commands/faculty/unlock_chat.js
@@ -10,7 +10,7 @@ const { facultyRole } = require("../../../../config.json");
 
 const execute = async (interaction, client, models) => {
   if (!interaction.member.permissions.has("ADMINISTRATOR") && !interaction.member.roles.cache.some(r => r.name === facultyRole)) {
-    await sendEphemeral(interaction, "You do not have permission to use this command.");
+    await sendErrorEphemeral(interaction, "You do not have permission to use this command.");
     return
   }
 

Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@ const { courseAdminRole, facultyRole } = require("../../../../config.json");`
`8`	`8`
`9`	`9`	`const execute = async (interaction, client, models) => {`
`10`	`10`	`if (!interaction.member.permissions.has("ADMINISTRATOR") && !interaction.member.roles.cache.some(r => r.name === facultyRole)) { // Olisiko parempi tarkistaa tietokannasta eikä discordista?`
`11`		`- await sendEphemeral(interaction, "You do not have permission to use this command.");`
	`11`	`+ await sendErrorEphemeral(interaction, "You do not have permission to use this command.");`
`12`	`12`	`return`
`13`	`13`	`}`
`14`	`14`
Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@ const { facultyRole } = require("../../../../config.json");`
`7`	`7`
`8`	`8`	`const execute = async (interaction, client, models) => {`
`9`	`9`	`if (!interaction.member.permissions.has("ADMINISTRATOR") && !interaction.member.roles.cache.some(r => r.name === facultyRole)) {`
`10`		`- await sendEphemeral(interaction, "You do not have permission to use this command.");`
	`10`	`+ await sendErrorEphemeral(interaction, "You do not have permission to use this command.");`
`11`	`11`	`return`
`12`	`12`	`}`
`13`	`13`
Original file line number	Diff line number	Diff line change
`@@ -8,8 +8,8 @@ const { sendErrorEphemeral, sendEphemeral, editEphemeral } = require("../../serv`
`8`	`8`	`const { facultyRole } = require("../../../../config.json");`
`9`	`9`
`10`	`10`	`const execute = async (interaction, client, models) => {`
`11`		`-if (!interaction.member.permissions.has("ADMINISTRATOR") && !interaction.member.roles.cache.some(r => r.name === facultyRole)) {`
`12`		`- await sendEphemeral(interaction, "You do not have permission to use this command.");`
	`11`	`+ if (!interaction.member.permissions.has("ADMINISTRATOR") && !interaction.member.roles.cache.some(r => r.name === facultyRole)) {`
	`12`	`+ await sendErrorEphemeral(interaction, "You do not have permission to use this command.");`
`13`	`13`	`return`
`14`	`14`	`}`
`15`	`15`
Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@ const numbers = [ "1️⃣", "2️⃣", "3️⃣", "4️⃣", "5️⃣", "6️`
`8`	`8`
`9`	`9`	`const execute = async (interaction, client) => {`
`10`	`10`	`if (!interaction.member.permissions.has("ADMINISTRATOR") && !interaction.member.roles.cache.some(r => r.name === facultyRole)) {`
`11`		`- await sendEphemeral(interaction, "You do not have permission to use this command.");`
	`11`	`+ await sendErrorEphemeral(interaction, "You do not have permission to use this command.");`
`12`	`12`	`return`
`13`	`13`	`}`
`14`	`14`
Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@ const { sendMessageToTelegram } = require("../../../telegramBot/bridge/service")`
`8`	`8`
`9`	`9`	`const execute = async (interaction, client, models) => {`
`10`	`10`	`if (!interaction.member.permissions.has("ADMINISTRATOR") && !interaction.member.roles.cache.some(r => r.name === facultyRole)) {`
`11`		`- await sendEphemeral(interaction, "You do not have permission to use this command.");`
	`11`	`+ await sendErrorEphemeral(interaction, "You do not have permission to use this command.");`
`12`	`12`	`return`
`13`	`13`	`}`
`14`	`14`
Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@ const { facultyRole } = require("../../../../config.json");`
`8`	`8`
`9`	`9`	`const execute = async (interaction, client, models) => {`
`10`	`10`	`if (!interaction.member.permissions.has("ADMINISTRATOR") && !interaction.member.roles.cache.some(r => r.name === facultyRole)) {`
`11`		`- await sendEphemeral(interaction, "You do not have permission to use this command.");`
	`11`	`+ await sendErrorEphemeral(interaction, "You do not have permission to use this command.");`
`12`	`12`	`return`
`13`	`13`	`}`
`14`	`14`
Original file line number	Diff line number	Diff line change
`@@ -67,7 +67,7 @@ const changeCourseNick = async (interaction, client, models, courseName, newValu`
`67`	`67`
`68`	`68`	`const execute = async (interaction, client, models) => {`
`69`	`69`	`if (!interaction.member.permissions.has("ADMINISTRATOR") && !interaction.member.roles.cache.some(r => r.name === facultyRole)) {`
`70`		`- await sendEphemeral(interaction, "You do not have permission to use this command.");`
	`70`	`+ await sendErrorEphemeral(interaction, "You do not have permission to use this command.");`
`71`	`71`	`return`
`72`	`72`	`}`
`73`	`73`
Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,7 @@ const { saveChannelTopicToDb } = require("../../../db/services/channelService");`
`12`	`12`
`13`	`13`	`const execute = async (interaction, client, models) => {`
`14`	`14`	`if (!interaction.member.permissions.has("ADMINISTRATOR") && !interaction.member.roles.cache.some(r => r.name === facultyRole)) {`
`15`		`- await sendEphemeral(interaction, "You do not have permission to use this command.");`
	`15`	`+ await sendErrorEphemeral(interaction, "You do not have permission to use this command.");`
`16`	`16`	`return`
`17`	`17`	`}`
`18`	`18`