quiet quartz
await cursor.execute(f"SELECT * FROM antiNuke WHERE guildid = ? AND punishment = ?", (ctx.guild.id, 0))
check = await cursor.fetchone()
if check is None:
print("1")
await cursor.execute("INSERT INTO antiPunishment VALUES (?,?)", (ctx.guild.id, 0))
I know i did this wrong lol
how would i check if theres nothing in a sqlite row
@echo lava
icy wagon
hm
why not iterate it
i think async for item in cursor should work, given you are using aiosqlite
quiet quartz
i think `async for item in cursor` should work, given you are using aiosqlite
Im using asqlite
echo lava
idk postgres
learn it
echo lava
sqlite is 💩 for bots
quiet quartz
too much work
^
late matrix
Sqlite still works
quiet quartz
i think `async for item in cursor` should work, given you are using aiosqlite
would it
ill test hold on
Nvm
not even gonna waste my time
I know that wouldnt work
late matrix
I'm not 100% sure so take with a grain of salt but I think it's: ```py
await cursor.execute(f"SELECT * FROM antiNuke WHERE guildid = ? AND punishment = ?", (ctx.guild.id, 0))
check = await cursor.fetchone()
if None in check:
print("1")
await cursor.execute("INSERT INTO antiPunishment VALUES (?,?)", (ctx.guild.id, 0))
Cuz afaik that returns a tuple
quiet quartz
I'm not 100% sure so take with a grain of salt but I think it's: ```py
a...
pretty sure this is literally what i have
whats the difference betweeen if None in check: and if check is None
echo lava
whats the difference betweeen `if None in check:` and `if check is None`
first means is check contains None?, second means is check points to None?
quiet quartz
okay ill test
Traceback (most recent call last):
File "C:\Users\nlotg\AppData\Local\Programs\Python\Python311\Lib\site-packages\discord\ext\commands\core.py", line 235, in wrapped
ret = await coro(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\nlotg\OneDrive\Desktop\selfbot\cogs\antinuke.py", line 252, in pungishment
if None in check:
^^^^^^^^^^^^^
TypeError: argument of type 'NoneType' is not iterable
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "C:\Users\nlotg\AppData\Local\Programs\Python\Python311\Lib\site-packages\jishaku\features\invocation.py", line 165, in jsk_debug
await alt_ctx.command.invoke(alt_ctx)
File "C:\Users\nlotg\AppData\Local\Programs\Python\Python311\Lib\site-packages\discord\ext\commands\core.py", line 1650, in invoke
await ctx.invoked_subcommand.invoke(ctx)
File "C:\Users\nlotg\AppData\Local\Programs\Python\Python311\Lib\site-packages\discord\ext\commands\core.py", line 1029, in invoke
await injected(*ctx.args, **ctx.kwargs) # type: ignore
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\nlotg\AppData\Local\Programs\Python\Python311\Lib\site-packages\discord\ext\commands\core.py", line 244, in wrapped
raise CommandInvokeError(exc) from exc
discord.ext.commands.errors.CommandInvokeError: Command raised an exception: TypeError: argument of type 'NoneType' is not iterable
if None in check:
^^^^^^^^^^^^^
TypeError: argument of type 'NoneType' is not iterable
@late matrix '
late matrix
Ok so its returning None not a tuple so you were right at the start
echo lava
what u mean i was right at the start
have you tried UPSERT?
late matrix
await cursor.execute(f"SELECT * FROM antiNuke WHERE guildid = ? AND punishment = ?", (ctx.guild.id, 0))
check = await cursor.fetchone()
if check is None:
print("1")
await cursor.execute("INSERT INTO antiPunishment VALUES (?,?)", (ctx.guild.id, 0))
I know i did this wrong lol
That one
quiet quartz
huh
echo lava
quiet quartz
have you tried UPSERT?
idk how to do that
send an example
@late matrix what are you saying though
echo lava
idk how to do that
learn it. https://www.sqlite.org/lang_upsert.html
late matrix
```py
await cursor.execute(f"SELECT * FROM antiNuke WHERE guildid = ? AN...
This original one should be right it's saying check is an instance of NoneType meaning the if check is None: is right
Are you using aiosqlite or sqlite3
echo lava
use aiosqlite.
quiet quartz
No
echo lava
not rapptz asqlite
echo lava
theres no difference
asqlite isn't aiosqlite
quiet quartz
theres still no difference bro
echo lava
yes there a difference
quiet quartz
This original one should be right it's saying check is an instance of NoneType m...
didnt work though
here ill try it again
spice owl
Your original code should work.
There are docs you can read, btw.
It either returns a Row or None if there is no data.
icy wagon
sqlite is 💩 for bots
stop shitting on sqlite, a discord bot which has maximum 5 tables does not need PostgresSQL lmao, sure i love postgres too but it would be overkill for a simple discord bot
icy wagon
Im gonna block u ngl
to be honest, aiosqlite is a better library than asqlite
spice owl
Oh that actually exists? I thought they were just mistyping aiosqlite
quiet quartz
stop shitting on sqlite, a discord bot which has maximum 5 tables does not need ...
I have more than 5 tables
i have like 30
icy wagon
very interesting, is ur bot big?
quiet quartz
No lol im recoding a old bot of mine
icy wagon
hmm also dont u think, the anti nuke / raid features are a bit obsolete, now since discord themselves have implemented that?
quiet quartz
1 server rn and its just a test server
quiet quartz
hmm also dont u think, the anti nuke / raid features are a bit obsolete, now sin...
no
spice owl
Even still, sqlite is fine and can handle multiple tables with 10s of thousands of rows.
quiet quartz
discord has no antiNuke features just antiraid
icy wagon
I'm assuming this is for like ur skill and improvement then?
spice owl
It just doesn't scale.
icy wagon
It just doesn't scale.
yes you are right
quiet quartz
what do you mean lol
icy wagon
by scaling or skill improvement
quiet quartz
lol
so is aiosqlite better than asqlite?
I could switch but bro was being annoying about it so i blocked him
icy wagon
yes yes yes
quiet quartz
how is it better
icy wagon
aiosqlite has 10x more stars but they both are pretty similar
so it doesn't really matter, you shouldn't care about changes this small, tbh, its an sqlite wrapper under the hood, sqlite being a SYNChronous database system so
its just a wrapper
spice owl
It's just more commonly used, really. asqlite is something Rapptz (the creator of d.py) rolled out. But, aiosqlite already existed.
icy wagon
exactly the point ^
icy wagon
yeah sure
spice owl
Personally, I use aiosqlite (when needed) in conjuction with SQLAlchemy (or some other ORM ifyou wanted)
quiet quartz
Your original code should work.
There are docs you can read, btw.
Back to this though it didn’t work
quiet quartz
LOL
spice owl
Back to this though it didn’t work
Then it's becuase asqlite is different.
quiet quartz
😭
quiet quartz
Then it's becuase asqlite is different.
no no
await cursor.execute(f"SELECT * FROM antiNuke WHERE guildid = ? AND punishment = ?", (ctx.guild.id, 0))
could it be because of this
im supposed to be checking if theres no row or if its None
icy wagon
oh also aiosqlite supports async iterators over cursors
icy wagon
idk if it does
spice owl
Yeah, it's not, it should still return None
quiet quartz
it does not
icy wagon
but rapptz made dpy, he couldn't pay much attention to asqlite i guess?
quiet quartz
Yeah, it's not, it should still return None
hold on ill rewrite the code & try again
if check is not None:
?
spice owl
You're trying to select from, and if there isn't a result, create a result, right?
quiet quartz
You're trying to select from, and if there isn't a result, create a result, righ...
Yes
Also
icy wagon
also a tip for writing queries:
# NO F STRINGS or sql injection and sad life :,(
await cursor.execute("""
SELECT *
FROM anti_nuke
WHERE
guild_id = :guild_id
AND punishment = :punishment
""", {
"guild_id": ctx.guild.id,
"punishment": 0
})
result = await cursor.fetchone()
spice owl
Couldn't you just INSERT OR UPDATE?
quiet quartz
```py
await cursor.execute(f"SELECT * FROM antiNuke WHERE guildid = ? AND punis...
^ it’s definitely this
0 = ban
1 = kick
btw
quiet quartz
Couldn't you just `INSERT OR UPDATE`?
Yeah
but
minor issue with that i noticed but idk how to explain it so im just finding another way
you cant update the row if theres no data
quiet quartz
im still new to sqlite but it’s definitely this
await cursor.execute(f"SELECT * FROM antiNuke WHERE guildid = ? AND punishment = ?", (ctx.guild.id, 0))
how about update or ignore
quiet quartz
Maybe because im checking if the punishment has 0 in it?
quiet quartz
Maybe because im checking if the punishment has 0 in it?
^
spice owl
Did you create the column as an int or a bool?
quiet quartz
int
spice owl
Hmm.
quiet quartz
@late matrix
spice owl
sec
quiet quartz
Maybe because im checking if the punishment has 0 in it?
^^^^
idk how to rewrite it to where it checks if theres a row at all instead of something specific
SELECT * from antiPunishment where guildid = ?
spice owl
You're currently trying to fetch where guild_id == something AND punishment==0
quiet quartz
await cursor.execute(f"SELECT * FROM antiPunishment WHERE guild_id = ?", (ctx.guild.id))
guild_id should be unique.
quiet quartz
would this work
late matrix
it should be ctx.guild.id
echo lava
guys. dont do select and insert if not exists. use UPSERT
late matrix
what is it kindly repeat
spice owl
that would return the row where the guild_id matches, if it exists
quiet quartz
im trying to check if a row doesn’t exist or if its None
late matrix
there is a method called fetch all ig
quiet quartz
that would return the row where the guild_id matches, if it exists
so that would work for what im trying to do
quiet quartz
im trying to check if a row doesn’t exist or if its None
^ if it doesn’t exist it creates the row
late matrix
how does your tables looks like?
send me the sql query
and the row you wanna check
quiet quartz
await cursor.execute("CREATE TABLE IF NOT EXISTS antiPunishment (guild_id INTEGER, punishment INTEGER)")
it was set to text for some reason but i just changed it to integer
quiet quartz
Yeah check if theres anything in punishment
if theres nothing it returns None correct?
spice owl
Well you don't have a default value, so the row can't exist without both guild and punishment.
late matrix
-d aiosqlite.fetch_all
tired tendonBOT
late matrix
it returns a list you could iterate over it
icy wagon
mommy python 🤤🤤🤤
quiet quartz
like the row is empty or not
Yeah
spice owl
guild_id should be unique, so select where guild_id == ctx.gulid.id
If the guild is not there, it will return None
quiet quartz
Well you don't have a default value, so the row can't exist without both guild a...
default value?
icy wagon
you can set defaults with sql yes
quiet quartz
Could i set punishment to 0 by default upon creating it?
icy wagon
yes yes yes
quiet quartz
i tried that earlier didnt work
late matrix
yesssssssssssss
spice owl
You could, yeah.
icy wagon
what have you tried
late matrix
i told thousands times
in dms
and you are still repeating same thing
i gave you syntax too
quiet quartz
CREATE TABLE IF NOT EXISTS antiPunishment (guild_id INTEGER, punishment INTEGER default 0)
late matrix
yeah
quiet quartz
and you are still repeating same thing
i went into dpy server and they told me what i was doing was wrong lol
late matrix
they are mf i hate dpy
quiet quartz
so i just removed it tried upsert instead and it didnt fix my problem
they said something like “ you cant update the row if it isnt created “
quiet quartz
told me default doesnt work for what im trying to do
i already had it
but alright thanks
ill try
late matrix
and close this thread if your query got resolved
spice owl
^^ /solved
spice owl
How so?
quiet quartz
await cursor.execute("CREATE TABLE IF NOT EXISTS antiPunishment (guild_id INTEGER, punishment INTEGER default 0)")
elif anti.lower() == "ban":
await cursor.execute(f"SELECT * FROM antiPunishment WHERE guild_id = {ctx.guild.id} AND punishment = 0")
check = await cursor.fetchone()
if check is not None:
e = discord.Embed(description=f"")
await ctx.send(embed=e)
return
await cursor.execute("UPDATE antiPunishment SET punishment = ? WHERE guild_id = ?", (0, ctx.guild.id))
await self.bot.db.commit()
e = discord.Embed(description=f"")
await ctx.send(embed=e)
ill remove the strings later yes i know bad way to do it
but
quiet quartz
```py
elif anti.lower() == "ban":
await cursor.execute(f"SELEC...
yeah but when i check it ignores the check even though the punishment is set to 0 which is ban
so it sends the 2nd embed instead of the 1st even though the punishment is already set to 0/ban
spice owl
But you have a check for not None and then are trying to update if it IS None
Your syntax is backwards.
spice owl
check = ...
if check: # basically what you have:
# do some stuff
return
# from this point on, check is None
update check...
this doesn't work, becasue check is None
check = ...
if check:
# update check
# do more stuff, I guess.
return
# check is None
# insert new guild_id
# do more stuff
Is what it should look like
icy wagon
im deleting discord & this bot this is lame
ahahaha optimal outcome
frail crag
didnt work either
show the code pls