Bug fixes

Author: davidhozic

docs/source/changelog.rst

@@ -31,6 +31,15 @@ Glossary
 ----------------------
 Releases
 ----------------------
+
+v2.8.3
+=================
+- Fixed new guilds being added whenever :class:`daf.client.ACCOUNT`'s update method failed.
+- Fixed error if passing ``None`` inside update method of account for the ``servers`` parameter.
+- Removed unneded check in object serialization (for remote) which slightly increases performance.
+- Fixed Enum values being converted to objects when viewing live items / importing schema from live view.
+
+
 v2.8.2
 =================
 - Fixed auto installation of ttkboostrap not opening the main window at the end.

src/daf/__init__.py

@@ -19,4 +19,4 @@
     from .misc import *
 
 
-VERSION = "2.8.2"
+VERSION = "2.8.3"

src/daf/client.py

@@ -491,6 +491,8 @@ async def update(self, **kwargs):
                 kwargs["intents"] = None
 
         servers = kwargs.pop("servers", self.servers + self._uiservers)
+        if servers is None:
+            servers = []
 
         @misc._async_safe("_update_sem")
         async def update_servers(self_):
@@ -514,5 +516,6 @@ async def update_servers(self_):
             await update_servers(self)
         except Exception:
             await self.initialize()  # re-login
+            servers = self.servers  # Only return initialized servers
             await update_servers(self)
             raise

src/daf/convert.py

@@ -353,10 +353,8 @@ def __convert_to_slotted():
                 if isinstance(v, LAMBDA_TYPE):
                     v = v(_return)
 
-                if not isinstance(v, discord.Client):  # For some reason it fails to logging when copied.
-                    v = copy.copy(v)  # Prevent external modifications since it's passed by reference
-
-                setattr(_return, k, v)
+                # copy.copy prevents external modifications since it's passed by reference
+                setattr(_return, k, copy.copy(v))
 
         return _return
 

src/daf/guild.py

@@ -1119,5 +1119,7 @@ async def update(self, init_options = None, **kwargs):
             return await misc._update(self, init_options=init_options, **kwargs)
         except Exception:
             self.cache.clear()
-            await self.initialize(self.parent)  # Reopen any async related connections
+            if self.parent is not None:  # Only if it were previously initialized
+                await self.initialize(self.parent)  # Reopen any async related connections
+
             raise

src/daf_gui/convert.py

@@ -287,7 +287,7 @@ def get_conversion_map(object_type):
 
     object_type = type(object_)
 
-    if object_type in {int, float, str, bool, decimal.Decimal, type(None)}:
+    if object_type in {int, float, str, bool, decimal.Decimal, type(None)} or isinstance(object_, Enum):
         if object_type is decimal.Decimal:
             object_ = float(object_)