From 578d3e4b24bed09feca1e8d86afbb99201fe138c Mon Sep 17 00:00:00 2001 From: Felix Moessbauer Date: Fri, 25 Oct 2024 15:53:27 +0200 Subject: [PATCH] do not double-print value when setting a repeated value When clearning or appending to a repeated value, both the "Clearing..." / "Adding..." line and the "Set..." line were shown. However, this is misleading as the only performed operation is the clearing / appending. We fix this by directly returning from the function in case of clearing / appending. --- meshtastic/__main__.py | 1 + 1 file changed, 1 insertion(+) diff --git a/meshtastic/__main__.py b/meshtastic/__main__.py index 7aa9f5e..4603822 100644 --- a/meshtastic/__main__.py +++ b/meshtastic/__main__.py @@ -256,6 +256,7 @@ def setPref(config, comp_name, raw_val) -> bool: cur_vals = [x for x in getattr(config_values, pref.name) if x not in [0, "", b""]] cur_vals.append(val) getattr(config_values, pref.name)[:] = cur_vals + return True prefix = f"{'.'.join(name[0:-1])}." if config_type.message_type is not None else "" print(f"Set {prefix}{uni_name} to {raw_val}")