fix live buttons

Author: davidhozic

src/daf/core.py

@@ -147,7 +147,7 @@ async def initialize(user_callback: Optional[Union[Callable, Coroutine]] = None,
     # Initialize logging
     # ------------------------------------------------------------
     if logger is None:
-        logger = logging.LoggerJSON(path=str(Path.home().joinpath("daf/History")))
+        logger = logging.LoggerJSON()
 
     await logging.initialize(logger)
     # ------------------------------------------------------------

src/daf/logging/_logging.py

@@ -178,9 +178,9 @@ class LoggerCSV(LoggerBASE):
     Parameters
     ----------------
     path: str
-        Path to the folder where logs will be saved.
+        Path to the folder where logs will be saved. Defaults to /<user-home>/daf/History
     delimiter: str
-        The delimiter between columns to use.
+        The delimiter between columns to use. Defaults to ';'
     fallback: Optional[LoggerBASE]
         The manager to use, in case saving using this manager fails.
 
@@ -190,7 +190,12 @@ class LoggerCSV(LoggerBASE):
         Something went wrong at OS level (insufficient permissions?)
         and fallback failed as well.
     """
-    def __init__(self, path: str, delimiter: str, fallback: Optional[LoggerBASE] = None) -> None:
+    def __init__(
+        self,
+        path: str = str(pathlib.Path.home().joinpath("daf/History")),
+        delimiter: str = ';',
+        fallback: Optional[LoggerBASE] = None
+    ) -> None:
         self.path = path
         self.delimiter = delimiter
         super().__init__(fallback)
@@ -261,8 +266,8 @@ class LoggerJSON(LoggerBASE):
 
     Parameters
     ----------------
-    path: str
-        Path to the folder where logs will be saved.
+    path: Optional[str]
+        Path to the folder where logs will be saved. Defaults to /<user-home>/daf/History
     fallback: Optional[LoggerBASE]
         The manager to use, in case saving using this manager fails.
 
@@ -272,7 +277,11 @@ class LoggerJSON(LoggerBASE):
         Something went wrong at OS level (insufficient permissions?)
         and fallback failed as well.
     """
-    def __init__(self, path: str, fallback: Optional[LoggerBASE] = None) -> None:
+    def __init__(
+        self,
+        path: str = str(pathlib.Path.home().joinpath("daf/History")),
+        fallback: Optional[LoggerBASE] = None
+    ) -> None:
         self.path = path
         super().__init__(fallback)
 

src/daf/logging/sql/mgr.py

@@ -258,7 +258,7 @@ def __init__(self,
         if fallback is Ellipsis:
             # Cannot use None as this can be a legit user value
             # and cannot pass directly due to Sphinx issues
-            fallback = logging.LoggerJSON("History")
+            fallback = logging.LoggerJSON()
 
         if dialect is None:
             dialect = "sqlite"

src/daf_gui/widgets/__init__.py

@@ -6,4 +6,4 @@
 from .object import *
 from .dpi import *
 from .async_util import *
-from .convert import *
+from .convert import *

src/daf_gui/widgets/extra.py

@@ -95,7 +95,11 @@ def _callback(*args):
 
 def setup_additional_live_update(w: ttk.Button, frame):
     # Don't have a bound object instance
-    if frame.old_object_info is None or frame.old_object_info.real_object is None:
+    if (
+        (oi := frame.old_object_info) is None or
+        (ro := oi.real_object) is None or
+        getattr(ro, "parent", False) is None
+    ):
         return
 
     def _callback(*args):
@@ -114,7 +118,11 @@ def _callback(*args):
 
 def setup_additional_live_refresh(w: ttk.Button, frame):
     # Don't have a bound object instance
-    if frame.old_object_info is None or frame.old_object_info.real_object is None:
+    if (
+        (oi := frame.old_object_info) is None or
+        (ro := oi.real_object) is None or
+        getattr(ro, "parent", False) is None
+    ):
         return
 
     def _callback(*args):
@@ -146,6 +154,7 @@ def _callback(*args):
     if hasattr(item, "update"):
         if item not in ADDITIONAL_WIDGETS:
             ADDITIONAL_WIDGETS[item] = []
+
         ADDITIONAL_WIDGETS[item].extend([
             AdditionalWidget(ttk.Button, setup_additional_live_update, text="Live update"),
             AdditionalWidget(ttk.Button, setup_additional_live_refresh, text="Refresh")