Error Command | discord.js - imagine an app | Page 1

elder burrow Apr 10, 2023, 3:38 AM

#

create a react-role command, but when I run it it gives me the following error in the console.

      throw er; // Unhandled 'error' event
      ^

TypeError: Cannot read properties of null (reading 'name')
    at Object.execute (c:\Users\tiuel\Desktop\Yuji\command\utilidades\reactionrole.js:87:26)
    at Object.execute (c:\Users\tiuel\Desktop\Yuji\event\interactionCreate.js:7:25)
    at Client.<anonymous> (c:\Users\tiuel\Desktop\Yuji\funcao\handleCommands.js:14:45)
    at Client.emit (node:events:513:28)
    at InteractionCreateAction.handle (c:\Users\tiuel\Desktop\Yuji\node_modules\discord.js\src\client\actions\InteractionCreate.js:97:12)
    at module.exports [as INTERACTION_CREATE] (c:\Users\tiuel\Desktop\Yuji\node_modules\discord.js\src\client\websocket\handlers\INTERACTION_CREATE.js:4:36)
    at WebSocketManager.handlePacket (c:\Users\tiuel\Desktop\Yuji\node_modules\discord.js\src\client\websocket\WebSocketManager.js:352:31)
    at WebSocketShard.onPacket (c:\Users\tiuel\Desktop\Yuji\node_modules\discord.js\src\client\websocket\WebSocketShard.js:494:22)
    at WebSocketShard.onMessage (c:\Users\tiuel\Desktop\Yuji\node_modules\discord.js\src\client\websocket\WebSocketShard.js:328:10)
    at callListener (c:\Users\tiuel\Desktop\Yuji\node_modules\ws\lib\event-target.js:290:14)
Emitted 'error' event on Client instance at:
    at emitUnhandledRejectionOrErr (node:events:394:10)
    at process.processTicksAndRejections (node:internal/process/task_queues:84:21)```

📎 reactionrole.js

desert valeBOT Apr 10, 2023, 3:38 AM

#

• What's your exact discord.js npm list discord.js and node node -v version?
• Post the full error stack trace, not just the top part!
• Show your code!
• Explain what exactly your issue is.
• Not a discord.js issue? Check out #useful-servers.

jade pine Apr 10, 2023, 3:39 AM

#

elder burrow create a react-role command, but when I run it it gives me the following error i...

Show full stack trace

elder burrow Apr 10, 2023, 3:41 AM

#

jade pine Show full stack trace

edited the post

jade pine Apr 10, 2023, 3:48 AM

#

elder burrow edited the post

cargo4 is sometimes null, so the error occurs when accessing .name. Either require that option, or check if it exists before accessing it

elder burrow Apr 10, 2023, 3:50 AM

#

jade pine cargo4 is sometimes null, so the error occurs when accessing .name. Either requi...

it exists, but I created it to be a non-mandatory option. by case should I put a required: false?

jade pine Apr 10, 2023, 3:51 AM

#

When someone runs the command, if they don't supply a value for cargo4, you are going to have errors when trying to access it here .setLabel(${cargo4.name}).

You either need to make that option required, or check if cargo4 exists before accessing it

elder burrow Apr 10, 2023, 3:53 AM

#

it exists, as I said, I want to make it a position that you are not obliged to put in, and just one more option if the person wants to put it in or not

jade pine Apr 10, 2023, 3:56 AM

#

If the command option isn't required then cargo4 might not exist sometimes. So again, check it exists before accessing it

elder burrow Apr 10, 2023, 3:57 AM

#

and how am i going to do that? you don't understand very well.. if it's to check the code, I did it just now

jade pine Apr 10, 2023, 3:58 AM

#

I understand perfectly fine. I am telling you how to fix it.

Check if cargo4 exists before accessing the name property

#

Don't physically go to your code and check, I mean write code to check if it exists

if (!exists) {
  // don't check it
}

elder burrow Apr 10, 2023, 4:01 AM

#

Ok

umbral token Apr 10, 2023, 4:01 AM

#

elder burrow it exists, but I created it to be a non-mandatory option. by case should I put ...

~~try put required: true at cargo4 option~~

jade pine Apr 10, 2023, 4:01 AM

#

umbral token ~~try put `required: true` at cargo4 option~~

He wants it to be optional

elder burrow Apr 10, 2023, 4:03 AM

#

umbral token ~~try put `required: true` at cargo4 option~~

if you do it like this, the person who is going to execute it will have to put 4 instead of 3

elder burrow Apr 10, 2023, 4:10 AM

#

jade pine Don't physically go to your code and check, I mean write code to check if it exi...

I'm sorry, but I'm a layman, there are times and I still don't understand what to do with this here or where to put it

jade pine Apr 10, 2023, 4:16 AM

#

elder burrow I'm sorry, but I'm a layman, there are times and I still don't understand what t...

Try something like this

        let botao = new ActionRowBuilder()
            .addComponents(
                new ButtonBuilder()
                    .setCustomId('botao1')
                    .setLabel(`${cargo1.name}`)
                    .setStyle(ButtonStyle.Primary),

                new ButtonBuilder()
                    .setCustomId('botao2')
                    .setLabel(`${cargo2.name}`)
                    .setStyle(ButtonStyle.Primary),

                new ButtonBuilder()
                    .setCustomId('botao3')
                    .setLabel(`${cargo3.name}`)
                    .setStyle(ButtonStyle.Primary),
            );

        if (cargo4) {
            botao.addComponents(
                new ButtonBuilder()
                    .setCustomId('botao4')
                    .setLabel(`${cargo4.name}`)
                    .setStyle(ButtonStyle.Primary)
            )
        }

elder burrow Apr 10, 2023, 8:56 AM

#

jade pine Try something like this ```js let botao = new ActionRowBuilder() ...

a new error appeared

#

      throw er; // Unhandled 'error' event
      ^

TypeError [ColorConvert]: Unable to convert color to a number.
    at resolveColor (c:\Users\tiuel\Desktop\Yuji\node_modules\discord.js\src\util\Util.js:260:63)
    at EmbedBuilder.setColor (c:\Users\tiuel\Desktop\Yuji\node_modules\discord.js\src\structures\EmbedBuilder.js:22:36)
    at Object.execute (c:\Users\tiuel\Desktop\Yuji\command\utilidades\reactionrole.js:96:5)
    at Object.execute (c:\Users\tiuel\Desktop\Yuji\event\interactionCreate.js:7:25)
    at Client.<anonymous> (c:\Users\tiuel\Desktop\Yuji\funcao\handleCommands.js:14:45)
    at Client.emit (node:events:513:28)
    at InteractionCreateAction.handle (c:\Users\tiuel\Desktop\Yuji\node_modules\discord.js\src\client\actions\InteractionCreate.js:97:12)
    at module.exports [as INTERACTION_CREATE] (c:\Users\tiuel\Desktop\Yuji\node_modules\discord.js\src\client\websocket\handlers\INTERACTION_CREATE.js:4:36)
    at WebSocketManager.handlePacket (c:\Users\tiuel\Desktop\Yuji\node_modules\discord.js\src\client\websocket\WebSocketManager.js:352:31)
    at WebSocketShard.onPacket (c:\Users\tiuel\Desktop\Yuji\node_modules\discord.js\src\client\websocket\WebSocketShard.js:494:22)
Emitted 'error' event on Client instance at:
    at emitUnhandledRejectionOrErr (node:events:394:10)
    at process.processTicksAndRejections (node:internal/process/task_queues:84:21) {
  code: 'ColorConvert'
}```

formal gorge Apr 10, 2023, 10:20 AM

#

What color did you put in reactionrole.js line 96?

elder burrow Apr 10, 2023, 10:38 AM

#

formal gorge What color did you put in reactionrole.js line 96?

not one, I put it so that the user can put the color he wants

formal gorge Apr 10, 2023, 10:40 AM

#

Then your error is to be expected if the user puts one that is invalid

#

You should handle that

elder burrow Apr 10, 2023, 10:41 AM

#

but in case it is a string value, then it can put it in hexadecimal right? because this is the logic that I have when making the code

formal gorge Apr 10, 2023, 10:43 AM

#

If it‘s formatted as a correct hexadecimal color, yes. #ff0000 would work, #da wouldn’t for example

#

Neither would #discord

elder burrow Apr 10, 2023, 10:44 AM

#

yes, that's why I put an example in the description, but I believe it makes a mistake in thinking here. I'll show you how it is now

#

            .setColor(`${cor.name}`)
            .setTitle(`${titulo.name}`)
            .setDescription(`${descricao.name}`);```

formal gorge Apr 10, 2023, 10:46 AM

#

You need to catch that error either way. Always account for user error

elder burrow Apr 10, 2023, 10:46 AM

#

but I think the correct formula is not this one but this one

#

            .setColor(cor)
            .setTitle(titulo)
            .setDescription(descricao);```

elder burrow Apr 10, 2023, 10:47 AM

#

formal gorge You need to catch that error either way. Always account for user error

Yes

formal gorge Apr 10, 2023, 10:48 AM

#

elder burrow ```const embed = new EmbedBuilder() .setColor(cor) .setT...

Depends on how you defined cor etc.

elder burrow Apr 10, 2023, 10:48 AM

#

elder burrow ```const embed = new EmbedBuilder() .setColor(cor) .setT...

this must be the correct form since I created a const of them

elder burrow Apr 10, 2023, 10:49 AM

#

formal gorge Depends on how you defined cor etc.

        const titulo = inter.options.getString('titulo');
        const descricao = inter.options.getString('descricao');```

formal gorge Apr 10, 2023, 10:49 AM

#

elder burrow ```const embed = new EmbedBuilder() .setColor(cor) .setT...

Then this is correct

elder burrow Apr 10, 2023, 10:50 AM

#

defines it like this, my mistake is wanting to put the name and not the declared value of the const to appear

#

soon then he can put it in hexadecimal or even write the name of the color (which I think might work)

#

I'll test here

#

            else if (i.guild.members.me.roles.highest.position < cargo4.position) {
                                                                        ^

TypeError: Cannot read properties of null (reading 'position')
    at InteractionCollector.<anonymous> (c:\Users\tiuel\Desktop\Yuji\command\utilidades\reactionrole.js:120:64)
    at InteractionCollector.emit (node:events:525:35)
    at InteractionCollector.handleCollect (c:\Users\tiuel\Desktop\Yuji\node_modules\discord.js\src\structures\interfaces\Collector.js:133:14)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)```

#

now this error is occurring to me here, I even imagine what it is. Because in my line of reasoning I created 4 options for a position, 3 of which are mandatory to place. only I want to leave one or even more being optional to the user

formal gorge Apr 10, 2023, 11:39 AM

#

Then do the same you did for cargo4 earlier. Check for it existing before trying to use it

elder burrow Apr 10, 2023, 11:51 AM

#

I did this verification method

#

        if (!cargo1 || !cargo2 || !cargo3) {
            return inter.reply({ content: 'Por favor, forneça as opções obrigatórias!', ephemeral: true });
        }

        // Verifica se a opção 4 foi fornecida
        if (cargo4) {
            // código do comando para quando a opção 4 foi fornecida
            return inter.reply({ content: `Você escolheu ${cargo1}, ${cargo2}, ${cargo3} e ${cargo4}.`, ephemeral: true });
        } else {
            // código do comando para quando a opção 4 não foi fornecida
            inter.reply({ content: `Você escolheu ${cargo1}, ${cargo2} e ${cargo3}.`, ephemeral: true });
        }```

#

but it is giving me this error

formal gorge Apr 10, 2023, 11:52 AM

#

Line 120 is where in that?

elder burrow Apr 10, 2023, 11:52 AM

#

Wait

#

      throw er; // Unhandled 'error' event
      ^

TypeError: Cannot read properties of null (reading 'name')
    at Object.execute (c:\Users\tiuel\Desktop\Yuji\command\utilidades\reactionrole.js:101:26)
    at Object.execute (c:\Users\tiuel\Desktop\Yuji\event\interactionCreate.js:7:25)
    at Client.<anonymous> (c:\Users\tiuel\Desktop\Yuji\funcao\handleCommands.js:14:45)
    at Client.emit (node:events:513:28)
    at InteractionCreateAction.handle (c:\Users\tiuel\Desktop\Yuji\node_modules\discord.js\src\client\actions\InteractionCreate.js:97:12)
    at module.exports [as INTERACTION_CREATE] (c:\Users\tiuel\Desktop\Yuji\node_modules\discord.js\src\client\websocket\handlers\INTERACTION_CREATE.js:4:36)
    at WebSocketManager.handlePacket (c:\Users\tiuel\Desktop\Yuji\node_modules\discord.js\src\client\websocket\WebSocketManager.js:352:31)
    at WebSocketShard.onPacket (c:\Users\tiuel\Desktop\Yuji\node_modules\discord.js\src\client\websocket\WebSocketShard.js:494:22)
    at WebSocketShard.onMessage (c:\Users\tiuel\Desktop\Yuji\node_modules\discord.js\src\client\websocket\WebSocketShard.js:328:10)
    at callListener (c:\Users\tiuel\Desktop\Yuji\node_modules\ws\lib\event-target.js:290:14)
Emitted 'error' event on Client instance at:
    at emitUnhandledRejectionOrErr (node:events:394:10)
    at process.processTicksAndRejections (node:internal/process/task_queues:84:21)```

#

when he checks it shows that there is an error on line 101 which is where the cargo4 button is

formal gorge Apr 10, 2023, 11:53 AM

#

~~Do you know how to read stacktraces? It tells you what line in what file the issue is from,~~ so fix it in that line

formal gorge Apr 10, 2023, 11:54 AM

#

jade pine Try something like this ```js let botao = new ActionRowBuilder() ...

Did you do this as recommended?

elder burrow Apr 10, 2023, 11:54 AM

#

formal gorge Did you do this as recommended?

Yes

elder burrow Apr 10, 2023, 11:55 AM

#

elder burrow ```c:\Users\tiuel\Desktop\Yuji\command\utilidades\reactionrole.js:120 ...

but it gave me this error here

formal gorge Apr 10, 2023, 11:56 AM

#

Show your lines 90-110 then

elder burrow Apr 10, 2023, 11:59 AM

#

                    .setLabel(`${cargo4.name}`)
                    .setStyle(ButtonStyle.Primary)
            )
        }

        const embed = new EmbedBuilder()
            .setColor(cor)
            .setTitle(titulo)
            .setDescription(descricao.name);

        inter.reply({ embeds: [embed], components: [botao] });

        const coletor = inter.channel.createMessageComponentCollector();

        coletor.on('collect', async (i) => {

            const member = i.member;

            if (i.guild.members.me.roles.highest.position < cargo1.position) {
                i.update({ content: 'Meu cargo esta abaixo do cargo que estou tentando entregar', ephemeral: true });```

formal gorge Apr 10, 2023, 12:06 PM

#

elder burrow ``` .setCustomId('botao4') .setLabel(`${cargo...

You forgot to remove the .name from your descriptao (and apparently that is optional too?)

#

Please learn to look at the correct lines the stacktrace tells you about

elder burrow Apr 10, 2023, 12:10 PM

#

formal gorge Please learn to look at the correct lines the stacktrace tells you about

where can i see this? because in my course it still hasn't shown me how to identify the error line in the console

elder burrow Apr 10, 2023, 12:10 PM

#

formal gorge You forgot to remove the .name from your descriptao (and apparently that is opti...

it's not optional, forget it

formal gorge Apr 10, 2023, 12:10 PM

#

elder burrow when he checks it shows that there is an error on line 101 which is where the ca...

You were able to find it‘s line 101 here, now check the error stack and see if you find that number there too

#

Hint: it‘s near the name of the file the error is in

elder burrow Apr 10, 2023, 12:18 PM

#

Ok

elder burrow Apr 10, 2023, 12:19 PM

#

formal gorge You were able to find it‘s line 101 here, now check the error stack and see if y...

in this case, the error points to this line here:

#

    i.update({ content: 'Meu cargo esta abaixo do cargo que estou tentando entregar', ephemeral: true });
    return;
}```

#

I should remove the else and leave it like this:

#

    i.update({ content: 'Meu cargo esta abaixo do cargo que estou tentando entregar', ephemeral: true });
    return;
}```

#

by the way not and that, because the error still persists

-_-

formal gorge Apr 10, 2023, 12:32 PM

#

elder burrow I should remove the `else` and leave it like this:

No. You shouldn’t check that at all if the user didn’t put a cargo4 option

#Error Command