Discord Bots are not recognized as Bots in Python code

Question

I am writing a discordbot bot the function user.bot isnt working correctly

@client.event
async on_reaction_add(reaction, user):
     if not user.bot:
          await client.wait_until_ready()
     try:
          channel = client.get_channel(welcome_channel_id)
              try:
                  await channel.send(message)
                  await reaction.remove(user)
              except Exception as e
                  raise e
              except Exception as e
                  raise e

Everything works accept the if statement, I create a message with a reaction with a bot but the bots reaction is removed. I am using Python 3.8. If u need any extra information pls tell me.

reaction.remove() sure does sound like something that removes a reaction, right? — gshpychka
– gshpychka, Commented Nov 15, 2021 at 20:52
Yes but it still should not do it if the bot placed the reaction thats why there is a if not user.bot: — Naomi
– Naomi, Commented Nov 15, 2021 at 20:54

gshpychka · Accepted Answer · 2021-11-15 21:03:53Z

2

Your try block executed every time, after the if statement is evaluated.

I'm assuming it should only execute if the if statement evaluates to true:

if not user.bot:
     await client.wait_until_ready()
     try:
          channel = client.get_channel(welcome_channel_id)
          await channel.send(message)
          await reaction.remove(user)
      except Exception as e
              raise e

But raise e negates the whole point of using a try/catch block - it has no effect.

The above code is equivalent to

if not user.bot:
     await client.wait_until_ready()
     channel = client.get_channel(welcome_channel_id)
     await channel.send(message)
     await reaction.remove(user)

edited Nov 15, 2021 at 21:03

answered Nov 15, 2021 at 20:57

gshpychka

12.1k1 gold badge22 silver badges39 bronze badges

Sign up to request clarification or add additional context in comments.

5 Comments

Naomi Over a year ago

Thank u for your fast help. It works :)

gshpychka Over a year ago

Consider accepting the answer if it works.

Naomi Over a year ago

U where to fast I will as soon I can :)

gshpychka Over a year ago

Edited the answer to remove the redundant try/catch block. The try/catch block has no effect, though, as it just reraises every exception - it would work the same without it.

Naomi Over a year ago

In a example code I had it was done that way so I thought it was better to do so thanks for correcting it :)

Collectives™ on Stack Overflow

Discord Bots are not recognized as Bots in Python code

1 Answer 1

5 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

5 Comments

Your Answer

Sign up or log in

Post as a guest

Related