perform vote message check in reaction handler

server/events/handleMessageReactionAdd.ts

@@ -3,13 +3,16 @@ import { Message, MessageReaction, User } from "discord.js";
 import { logger, newRequestId, noGuildId } from "../logger";
 import { NONE_OF_THAT } from "../constants";
 import { client } from "../..";
+import { messageIsVoteMessage } from "../helper/messageIdentifiers";
 
 
 export const name = 'messageReactionAdd'
 
 export async function execute(messageReaction: MessageReaction, user: User) {
-	if (user.id == client.user?.id)
+	if (user.id == client.user?.id) {
 		logger.info('Skipping bot reaction')
+		return
+	}
 	const requestId = newRequestId()
 	const guildId = messageReaction.message.inGuild() ? messageReaction.message.guildId : noGuildId
 	const reactedUponMessage: Message = messageReaction.message.partial ? await messageReaction.message.fetch() : messageReaction.message
@@ -23,8 +26,11 @@ export async function execute(messageReaction: MessageReaction, user: User) {
 
 	logger.info(`emoji: ${messageReaction.emoji.toString()}`)
 	if (messageReaction.emoji.toString() === NONE_OF_THAT) {
-		logger.info(`Reaction is NONE_OF_THAT. Handling`, { requestId, guildId })
+		if (messageIsVoteMessage(reactedUponMessage)) {
+			logger.info(`Reaction is NONE_OF_THAT on a vote message. Handling`, { requestId, guildId })
 			return client.VoteController.handleNoneOfThatVote(messageReaction, user, reactedUponMessage, requestId, guildId)
+
+		}
 	}
 
 	return

server/helper/vote.controller.ts

@@ -10,9 +10,7 @@ export default class VoteController {
 
 	public async handleNoneOfThatVote(messageReaction: MessageReaction, user: User, reactedUponMessage: Message, requestId: string, guildId: string) {
 		if (!messageReaction.message.guild) return 'No guild'
-		if (messageIsVoteMessage(reactedUponMessage)) {
 		logger.debug(`${reactedUponMessage.id} is vote message`, { requestId, guildId })
-			if (messageReaction.message.reactions.cache.find(reaction => reaction.emoji.toString() == NONE_OF_THAT)) {
 		const watcherRoleMember = await getMembersWithRoleFromGuild(config.bot.announcement_role, messageReaction.message.guild)
 		logger.info("ROLE MEMBERS " + JSON.stringify(watcherRoleMember), { requestId, guildId })
 		const watcherRoleMemberCount = watcherRoleMember.size
@@ -27,7 +25,4 @@ export default class VoteController {
 		}
 		logger.info(`No reroll`, { requestId, guildId })
 	}
-		}
-
-	}
 }