Sapphire - Imagine a framework•16mo ago

Listener with `once: true` appears to be triggering multiple times

import { ApplyOptions } from '@sapphire/decorators';
import { Events, Listener } from '@sapphire/framework';
import { ChannelType } from 'discord.js';
import { DBMessageLogs } from '../lib/database/DBMessageLogs';

@ApplyOptions<Listener.Options>({ once: true, event: Events.ClientReady })
export class ReadyMessageLogsDBListener extends Listener {
    public override async run() {
        const { client, database } = this.container;
        const guilds = client.guilds.cache.map(g => g.id);

        guilds.forEach(guild_id => {
            const guild = client.guilds.cache.get(guild_id);
            const channelsCollection = guild?.channels.cache;

            if (!channelsCollection) {
                return;
            }

            const channels = [...channelsCollection.values()]
                .filter(channel => channel.type === ChannelType.GuildText);

            let logDB = new DBMessageLogs(database);
            channels.forEach(async (channel) => {
                console.log(`ADDING CHANNEL NAME ${channel.name} with ID ${channel.id}`);
                await logDB.channelAdd(BigInt(channel.id), channel.name, BigInt(guild.id));
            });
        })
    }
}

import { ApplyOptions } from '@sapphire/decorators';
import { Events, Listener } from '@sapphire/framework';
import { ChannelType } from 'discord.js';
import { DBMessageLogs } from '../lib/database/DBMessageLogs';

@ApplyOptions<Listener.Options>({ once: true, event: Events.ClientReady })
export class ReadyMessageLogsDBListener extends Listener {
    public override async run() {
        const { client, database } = this.container;
        const guilds = client.guilds.cache.map(g => g.id);

        guilds.forEach(guild_id => {
            const guild = client.guilds.cache.get(guild_id);
            const channelsCollection = guild?.channels.cache;

            if (!channelsCollection) {
                return;
            }

            const channels = [...channelsCollection.values()]
                .filter(channel => channel.type === ChannelType.GuildText);

            let logDB = new DBMessageLogs(database);
            channels.forEach(async (channel) => {
                console.log(`ADDING CHANNEL NAME ${channel.name} with ID ${channel.id}`);
                await logDB.channelAdd(BigInt(channel.id), channel.name, BigInt(guild.id));
            });
        })
    }
}

29 Replies

abeOP•16mo ago

I have this listener which adds all the text channels in a discord server to a table in a database. It appears to be triggering twice with the second time containing the right data from the channel.:

RECEIVED CHANNEL general with ID 1144586568296824800
ADDING CHANNEL NAME general with ID 1144586568296824888
RECEIVED CHANNEL NAME general with ID 1144586568296824888

RECEIVED CHANNEL general with ID 1144586568296824800
ADDING CHANNEL NAME general with ID 1144586568296824888
RECEIVED CHANNEL NAME general with ID 1144586568296824888

This is the code to add:

async channelAdd(channel_id: bigint, channel_name: string, guild_id: bigint) {
    try {
        console.log(`RECEIVED CHANNEL NAME ${channel_name} with ID ${channel_id}`);
        await this.db.$connect();
        const channel = await this.db.channels.findFirst({ where: { channel_id } });
        if (!channel) {
            await this.db.channels.create({
                data: {
                    channel_id,
                    channel_name,
                    guild_id
                }
            });
        }
        await this.db.$disconnect();
    } catch (error) {
        throw error;
    }
}

async channelAdd(channel_id: bigint, channel_name: string, guild_id: bigint) {
    try {
        console.log(`RECEIVED CHANNEL NAME ${channel_name} with ID ${channel_id}`);
        await this.db.$connect();
        const channel = await this.db.channels.findFirst({ where: { channel_id } });
        if (!channel) {
            await this.db.channels.create({
                data: {
                    channel_id,
                    channel_name,
                    guild_id
                }
            });
        }
        await this.db.$disconnect();
    } catch (error) {
        throw error;
    }
}

abeOP•16mo ago

it creates two records

abeOP•16mo ago

@Helpers please

vladdy•16mo ago

Firstly you are showing two different listeners for stuff

abeOP•16mo ago

wdym two different listeners

vladdy•16mo ago

But it also looks like whatever you're doing with ids isn't going smoothly because i see them encountering rounding errors Ready and channel create I'd check your sql stuff instead

abeOP•16mo ago

i have two listener files that have onready

abeOP•16mo ago

this is the structure

vladdy•16mo ago

We'll need more info because i don't see how it's related to once Each listener will only run once if one is set to true You have two ready listeners -> each will run only once

abeOP•16mo ago

yep

abeOP•16mo ago

but i end up triggering ReadyMessageLogsDBListener twice cos of that RECEIVED CHANNEL NAME... thing its being output twice

vladdy•16mo ago

Two different ids tho

abeOP•16mo ago

you suspect a rounding error?

abeOP•16mo ago

this pair is correct btw

abeOP•16mo ago

that id is an actual channel id in my testing server idk where the one ending in 800 came from

vladdy•16mo ago

Are you sure there's not just two channels with very close ids? Maybe log the channel name too

abeOP•16mo ago

i did log the channel name

abeOP•16mo ago

see

abeOP•16mo ago

this is the server im testing on

vladdy•16mo ago

Interesting Oh Hmm Why do you get it twice then

abeOP•16mo ago

1144586568296824888 1144586568296824889 these are the IDs of the text and voice channel respectively that's what i also would like to know 😅

vladdy•16mo ago

Where else do you call your channel add function Because it seems to be called outside ready Since there's no logs for adding

abeOP•16mo ago

only 1 invocation 2nd search result

vladdy•16mo ago

rm -rf dist, rebuild And check again

abeOP•16mo ago

omg

abeOP•16mo ago

idk how that fixed it but the ghost id is gone

vladdy•16mo ago

Stale code or a previous listener

abeOP•16mo ago

Thank you! much appreciated would've banged my head on this for ages 💀

Gaming

Programming

Listener with `once: true` appears to be triggering multiple times