Confusion

Dockerfile

@@ -5,4 +5,4 @@ WORKDIR /usr/src/app
 COPY package.json ./
 RUN npm install
 COPY . .
-CMD [ "node", "main.js" ]
+CMD ["/bin/sh", "-c", "node main.js 2> /logs/error.log 1> /logs/silvanus.log"]

modules/CustomClasses.js

@@ -54,9 +54,9 @@ module.exports = {
             this.watchChannelId = watchChannelId;
             return this;
         }
-        setRoles(waterRoleId, fruitRoleId) {
+        setRoles(waterRole, fruitRole) {
-            this.waterRoleId = waterRoleId;
+            this.waterRoleId = waterRole.id;
-            if (fruitRoleId) this.fruitRoleId = fruitRoleId;
+            if (fruitRole) this.fruitRoleId = fruitRole.id;
             return this;
         }
         queryBuilder(query) {
@@ -127,11 +127,18 @@ module.exports = {
                     return queryParts.join('');
                     break;
                 case "setRoles":
-                    queryParts = [
+                    if (this.fruitRoleId != "") {
-                        `UPDATE guild_info SET water_role_id = ${db.escape(this.waterRoleId)}, `,
+                        queryParts = [
-                        `fruit_role_id = ${db.escape(this.fruitRoleId)} `,
+                            `UPDATE guild_info SET water_role_id = ${db.escape(this.waterRoleId)}, `,
-                        `WHERE guild_id = ${db.escape(this.guildId)}`
+                            `fruit_role_id = ${db.escape(this.fruitRoleId)} `,
-                    ];
+                            `WHERE guild_id = ${db.escape(this.guildId)}`
+                        ];
+                    } else {
+                        queryParts = [
+                            `UPDATE guild_info SET water_role_id = ${db.escape(this.waterRoleId)} `,
+                            `WHERE guild_id = ${db.escape(this.guildId)}`
+                        ];
+                    }
                     return queryParts.join('');
                     break;
                     default:

modules/functions.js

@@ -70,12 +70,12 @@ const functions = {
 					);
 				return refreshActionRow;
 			},
-			treeRoleMenu() {
+			treeRoleMenu(fruit) {
-				return new ActionRowBuilder()
+				let actionRow = new ActionRowBuilder().addComponents(this.buttons.waterPing());
-					.addComponents(
+				if (fruit) {
-						this.buttons.waterPing(),
+					actionRow.addComponents(this.buttons.fruitPing());
-						this.buttons.fruitPing()
+				}
-					);
+				return actionRow;
 			},
 			buttons: {
 				acceptRules() {
@@ -100,10 +100,10 @@ const functions = {
 		},
 		embeds: {
 			treeRoleMenu(guildInfo) {
-				const actionRow = functions.builders.actionRows.treeRoleMenu();
+				const actionRow = functions.builders.actionRows.treeRoleMenu(guildInfo.fruitRoleId == "" ? false : true);
 				let tempStrings = strings.embeds.treeRoleMenu;
 				let description = tempStrings[0] + tempStrings[1] + `<@&${guildInfo.waterRoleId}>` + tempStrings[2];
-				if (guildInfo.fruitRoleId != undefined) {
+				if (guildInfo.fruitRoleId != "") {
 					description += tempStrings[3] + `<@&${guildInfo.fruitRoleId}>` + tempStrings[4];
 				}
 				const embed = new EmbedBuilder()

slash-commands/rolemenu.js

@@ -19,7 +19,7 @@ module.exports = {
 		await interaction.deferReply().catch(err => console.error(err));
         if (interaction.client.guildInfos.has(interaction.guildId)) {
             let guildInfo = interaction.client.guildInfos.get(interaction.guildId);
-            guildInfo.setRoles(interaction.options.getRole('waterrole').id, interaction.options.getRole('fruitrole').id);
+            guildInfo.setRoles(interaction.options.getRole('waterrole'), interaction.options.getRole('fruitrole'));
             await dbfn.setGuildInfo(guildInfo.queryBuilder("setRoles"));
             await fn.collectionBuilders.guildInfos(interaction.client);
             await interaction.editReply(fn.builders.embeds.treeRoleMenu(guildInfo)).catch(err => console.error(err));