From 79bb0dc1bdec8c2f7a2ea1c8f31572be42611d70 Mon Sep 17 00:00:00 2001 From: DevTekVE Date: Sat, 3 Aug 2024 14:10:00 +0200 Subject: [PATCH] Refactor param assignment to use instance variable Changed param assignments in signal handlers to utilize the `self.params` instance variable. This ensures consistency and improves code readability by avoiding reliance on the global `params` object. --- system/updated/updated.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/system/updated/updated.py b/system/updated/updated.py index a417d69160..677f0d2f4c 100755 --- a/system/updated/updated.py +++ b/system/updated/updated.py @@ -44,17 +44,18 @@ class WaitTimeHelper: def __init__(self): self.ready_event = threading.Event() self.user_request = UserRequest.NONE + self.params = Params() signal.signal(signal.SIGHUP, self.update_now) signal.signal(signal.SIGUSR1, self.check_now) def update_now(self, signum: int, frame) -> None: cloudlog.info("caught SIGHUP, attempting to downloading update") - params.put("UpdaterState", "Update request received... Wait...") + self.params.put("UpdaterState", "Update request received... Wait...") self.user_request = UserRequest.FETCH self.ready_event.set() def check_now(self, signum: int, frame) -> None: - params.put("UpdaterState", "Initializing updater to start check...") + self.params.put("UpdaterState", "Initializing updater to start check...") cloudlog.info("caught SIGUSR1, checking for updates") self.user_request = UserRequest.CHECK self.ready_event.set()