Data not storing (#116)

* remove: deprecated json saving of userHandle

* fix: changed false response from bot about user creating

* fix: no data from 'add' functions was written to db

* version: 2.5.2 -> 2.5.3

Author: PantomInach

cogs/commandlistener.py

@@ -52,7 +52,7 @@ async def on_ready(self):
         # Sends message to mods, when bot is online
         print("Now Online")
         await self.utils.sendModsMessage(
-            "Bot is now online.\nVersion:\tDiscordBot DataBot v2.5.2"
+            "Bot is now online.\nVersion:\tDiscordBot DataBot v2.5.3"
         )
         # Sets the bot's presence to "Online" or "Do not Disturb" to indicate if it's logging or not.
         if self.ch.getFromConfig("log") == "True":
@@ -226,7 +226,6 @@ async def on_raw_reaction_add(self, payload):
                     and self.ch.getFromConfig("log") == "True"
                 ):
                     self.uh.addReactionXP(payload.user_id, self.xpf.randomRange(1, 5))
-                    self.uh.saveData()
 
     # When a user changes his voice state
     @commands.Cog.listener()
@@ -391,7 +390,6 @@ async def on_message(self, message):
                 if name.endswith("jpg") or name.endswith("png"):
                     # Gives XP when picture is in message
                     self.uh.addTextXP(userID, self.xpf.randomRange(20, 40))
-                    self.uh.saveData()
                     return
 
         # When Message is a String
@@ -403,7 +401,6 @@ async def on_message(self, message):
             # Give XP when message is not a command
             if self.ch.isInWhitelist(message.channel.id):
                 self.uh.addTextXP(message.author.id, self.xpf.textXP(a))
-                self.uh.saveData()
 
         # Sends BotOwner commands, which are triggering the bot
         if len(a) > 0 and a[0] == self.ch.getFromConfig("command_prefix"):

cogs/commanduser.py

@@ -191,8 +191,7 @@ async def getUserData(self, ctx, userID):
                 + f" VoiceXP: {voice} TextXP: {text} TextCount: {textCount}"
             )
         else:
-            user = self.bot.get_user(int(userID))
-            message = f"User was not found in data. Created user: {user.mention}"
+            message = "User was not found in data."
         await ctx.send(message)
 
     @userSetParent.command(name="voice", brief="Sets users voicexp")

datahandler/userHandle.py

@@ -37,7 +37,7 @@ def __new__(
             else:
                 cls.db_path = db_file_path
 
-            cls.db = SqliteDict(cls.db_path, outer_stack=False, autocommit=True)
+            cls.db = SqliteDict(cls.db_path, outer_stack=True, autocommit=True)
             if (
                 load_from_json_if_not_init
                 and not cls.db
@@ -50,7 +50,7 @@ def __new__(
                         cls.db[key] = value
 
             cls.ch = ConfigHandle()
-        atexit.register(UserHandle._cleanup, cls.db)
+            atexit.register(UserHandle._cleanup, cls.db)
         return cls._instance
 
     """
@@ -357,7 +357,7 @@ def addUserVoice(self, userID: int | str, voice: int | str = 1):
         """
         self.addNewDataEntry(userID)
         entry = self.db[str(userID)]
-        entry["Voice"] += int(voice)
+        entry["Voice"] = int(entry["Voice"]) + int(voice)
         self.db[str(userID)] = entry
 
     def addUserText(self, userID: int | str, text: str | int):
@@ -369,7 +369,7 @@ def addUserText(self, userID: int | str, text: str | int):
         """
         self.addNewDataEntry(userID)
         entry = self.db[str(userID)]
-        entry["Text"] += int(text)
+        entry["Text"] = int(entry["Text"]) + int(text)
         self.db[str(userID)] = entry
 
     def addUserTextCount(self, userID: int | str, count: int | str = 1):
@@ -382,7 +382,7 @@ def addUserTextCount(self, userID: int | str, count: int | str = 1):
         """
         self.addNewDataEntry(userID)
         entry = self.db[str(userID)]
-        entry["TextCount"] += int(count)
+        entry["TextCount"] = int(entry["TextCount"]) + int(count)
         self.db[str(userID)] = entry
 
     @classmethod
@@ -394,5 +394,6 @@ def _cleanup(cls, db):
         Keyword arguments:
         db -- SqliteDict database object, which should be closed.
         """
+        print("[UserHandle] Closing Databank")
         db.commit()
         db.close()