Skip to content

Commit 9165e57

Browse files
committed
Fix deleting config keys properly
1 parent 9d636e4 commit 9165e57

File tree

2 files changed

+3
-6
lines changed

2 files changed

+3
-6
lines changed

cogs/modmail.py

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -87,9 +87,6 @@ async def _add(self, ctx, name: str.lower, *, value):
8787
async def __del(self, ctx, *, name: str.lower):
8888
"""Removes a snippet from bot config."""
8989

90-
if 'snippets' not in self.bot.config.cache:
91-
self.bot.config['snippets'] = {}
92-
9390
em = discord.Embed(
9491
title='Removed snippet',
9592
color=discord.Color.green(),
@@ -101,7 +98,7 @@ async def __del(self, ctx, *, name: str.lower):
10198
em.color = discord.Color.red()
10299
em.description = f'Snippet `{name}` does not exist.'
103100
else:
104-
self.bot.config['snippets'][name] = None
101+
del self.bot.config['snippets'][name]
105102
await self.bot.config.update()
106103

107104
await ctx.send(embed=em)

cogs/utility.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -392,7 +392,7 @@ async def _del(self, ctx, key: str.lower):
392392
valid_keys = [f'`{k}`' for k in self.bot.config.allowed_to_change_in_command]
393393
em.add_field(name='Valid keys', value=', '.join(valid_keys))
394394
else:
395-
self.bot.config.cache[key] = None
395+
del self.bot.config.cache[key]
396396
await self.bot.config.update()
397397

398398
await ctx.send(embed=em)
@@ -498,7 +498,7 @@ async def __del(self, ctx, *, name: str.lower):
498498
em.color = discord.Color.red()
499499
em.description = f'Alias `{name}` does not exist.'
500500
else:
501-
self.bot.config['aliases'][name] = None
501+
del self.bot.config['aliases'][name]
502502
await self.bot.config.update()
503503

504504
await ctx.send(embed=em)

0 commit comments

Comments
 (0)