From cf87ed2142e6c1c1038e0a925eec056031c4b9f5 Mon Sep 17 00:00:00 2001 From: Daniel Hatton Date: Wed, 19 Mar 2025 10:27:10 +0000 Subject: [PATCH] Do not try and record notification parameters that are not properly set --- src/murfey/workflows/notifications/__init__.py | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/src/murfey/workflows/notifications/__init__.py b/src/murfey/workflows/notifications/__init__.py index 7c9709f21..f7314e8e8 100644 --- a/src/murfey/workflows/notifications/__init__.py +++ b/src/murfey/workflows/notifications/__init__.py @@ -35,15 +35,16 @@ def notification_setup( enp.max_value = v[1] break else: - new_notification_parameters.append( - NotificationParameter( - dcg_id=dcgid, - name=k, - min_value=v[0], - max_value=v[1], - num_instances_since_triggered=num_instances_between_triggers, + if v[0] is not None and v[1] is not None: + new_notification_parameters.append( + NotificationParameter( + dcg_id=dcgid, + name=k, + min_value=v[0], + max_value=v[1], + num_instances_since_triggered=num_instances_between_triggers, + ) ) - ) murfey_db.add_all(existing_notification_parameters + new_notification_parameters) murfey_db.commit() murfey_db.close()