bec-widgets 1.16.2__tar.gz → 1.16.3__tar.gz

{bec_widgets-1.16.2 → bec_widgets-1.16.3}/CHANGELOG.md

@@ -1,6 +1,20 @@
 # CHANGELOG
 
 
+## v1.16.3 (2025-01-20)
+
+### Bug Fixes
+
+- **error_popups**: Logger message in SafeSlot for errors; identification in error log from which
+  property or signal errors comes from
+  ([`02a4862`](https://gitlab.psi.ch/bec/bec_widgets/-/commit/02a4862afdbbb5d343f798a086395e1596d1239a))
+
+### Testing
+
+- **error_popups**: Safeslot tests adjusted; tests extended to cover SafeProperty
+  ([`dfa2908`](https://gitlab.psi.ch/bec/bec_widgets/-/commit/dfa2908c3de39802d40a2dee3e77cd5ca2ccad3b))
+
+
 ## v1.16.2 (2025-01-20)
 
 ### Bug Fixes

{bec_widgets-1.16.2 → bec_widgets-1.16.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: bec_widgets
-Version: 1.16.2
+Version: 1.16.3
 Summary: BEC Widgets
 Project-URL: Bug Tracker, https://gitlab.psi.ch/bec/bec_widgets/issues
 Project-URL: Homepage, https://gitlab.psi.ch/bec/bec_widgets

{bec_widgets-1.16.2 → bec_widgets-1.16.3}/bec_widgets/qt_utils/error_popups.py

@@ -33,45 +33,60 @@ def SafeProperty(prop_type, *prop_args, popup_error: bool = False, default=None,
     """
 
     def decorator(py_getter):
+        """Decorator for the user's property getter function."""
+
         @functools.wraps(py_getter)
         def safe_getter(self_):
             try:
                 return py_getter(self_)
             except Exception:
+                # Identify which property function triggered error
+                prop_name = f"{py_getter.__module__}.{py_getter.__qualname__}"
+                error_msg = traceback.format_exc()
+
                 if popup_error:
                     ErrorPopupUtility().custom_exception_hook(*sys.exc_info(), popup_error=True)
-                # Return the user-defined default (which might be anything, including None).
                 else:
-                    error_msg = traceback.format_exc()
-                    logger.error(str(error_msg))
+                    logger.error(f"SafeProperty error in GETTER of '{prop_name}':\n{error_msg}")
                 return default
 
         class PropertyWrapper:
+            """
+            Intermediate wrapper used so that the user can optionally chain .setter(...).
+            """
+
             def __init__(self, getter_func):
                 # We store only our safe_getter in the wrapper
                 self.getter_func = safe_getter
 
             def setter(self, setter_func):
+                """Wraps the user-defined setter to handle errors safely."""
+
                 @functools.wraps(setter_func)
                 def safe_setter(self_, value):
                     try:
                         return setter_func(self_, value)
                     except Exception:
+                        prop_name = f"{setter_func.__module__}.{setter_func.__qualname__}"
+                        error_msg = traceback.format_exc()
+
                         if popup_error:
                             ErrorPopupUtility().custom_exception_hook(
                                 *sys.exc_info(), popup_error=True
                             )
-                        # Swallow the exception; no crash in Designer
                         else:
-                            error_msg = traceback.format_exc()
-                            logger.error(str(error_msg))
+                            logger.error(
+                                f"SafeProperty error in SETTER of '{prop_name}':\n{error_msg}"
+                            )
                         return
 
                 # Return the full read/write Property
                 return Property(prop_type, self.getter_func, safe_setter, *prop_args, **prop_kwargs)
 
             def __call__(self):
-                # If the user never chains a .setter(...) call, we produce a read-only property
+                """
+                If user never calls `.setter(...)`, produce a read-only property.
+                """
                 return Property(prop_type, self.getter_func, None, *prop_args, **prop_kwargs)
 
         return PropertyWrapper(py_getter)
@@ -95,7 +110,14 @@ def SafeSlot(*slot_args, **slot_kwargs):  # pylint: disable=invalid-name
             try:
                 return method(*args, **kwargs)
             except Exception:
-                ErrorPopupUtility().custom_exception_hook(*sys.exc_info(), popup_error=popup_error)
+                slot_name = f"{method.__module__}.{method.__qualname__}"
+                error_msg = traceback.format_exc()
+                if popup_error:
+                    ErrorPopupUtility().custom_exception_hook(
+                        *sys.exc_info(), popup_error=popup_error
+                    )
+                else:
+                    logger.error(f"SafeSlot error in slot '{slot_name}':\n{error_msg}")
 
         return wrapper
 

{bec_widgets-1.16.2 → bec_widgets-1.16.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "bec_widgets"
-version = "1.16.2"
+version = "1.16.3"
 description = "BEC Widgets"
 requires-python = ">=3.10"
 classifiers = [