From a8aa39d89c21fa267ba63e4972b07828505291f1 Mon Sep 17 00:00:00 2001
From: Zevaryx <zevaryx@gmail.com>
Date: Thu, 2 Sep 2021 12:12:36 -0600
Subject: [PATCH] Change setting.save() location

---
 jarvis/cogs/rolegiver.py | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/jarvis/cogs/rolegiver.py b/jarvis/cogs/rolegiver.py
index 149bd29..423b3ba 100644
--- a/jarvis/cogs/rolegiver.py
+++ b/jarvis/cogs/rolegiver.py
@@ -39,6 +39,9 @@ class RolegiverCog(commands.Cog):
         if not setting:
             setting = Setting(guild=ctx.guild.id, setting="rolegiver", value=[])
 
+        setting.value.append(role.id)
+        setting.save()
+
         roles = []
         for role_id in setting.value:
             e_role = ctx.guild.get_role(role_id)
@@ -53,8 +56,6 @@ class RolegiverCog(commands.Cog):
             Field(name="New Role", value=f"{role.mention}"),
             Field(name="Existing Role(s)", value=value),
         ]
-        setting.value.append(role.id)
-        setting.save()
 
         embed = build_embed(
             title="Rolegiver Updated",
@@ -96,6 +97,7 @@ class RolegiverCog(commands.Cog):
             return
 
         setting.value.remove(role.id)
+        setting.save()
 
         roles = []
         for role_id in setting.value:
@@ -107,7 +109,6 @@ class RolegiverCog(commands.Cog):
         if roles:
             roles.sort(key=lambda x: -x.position)
 
-        setting.save()
         value = "\n".join([r.mention for r in roles]) if roles else "None"
         fields = [
             Field(name="Removed Role", value=f"{role.mention}"),