From 99fc53d3691b24c6724c1cf3e7281c181b31cf45 Mon Sep 17 00:00:00 2001 From: Matej Tyc Date: Tue, 11 Oct 2022 17:07:28 +0200 Subject: [PATCH 1/2] Remove redundant message The send_ready already performs what the removed call could aim to accomplish. --- org_fedora_oscap/gui/spokes/oscap.py | 1 - 1 file changed, 1 deletion(-) diff --git a/org_fedora_oscap/gui/spokes/oscap.py b/org_fedora_oscap/gui/spokes/oscap.py index 6d0aa5c..37b9681 100644 --- a/org_fedora_oscap/gui/spokes/oscap.py +++ b/org_fedora_oscap/gui/spokes/oscap.py @@ -151,7 +151,6 @@ def decorated(self, *args, **kwargs): self._ready = True # pylint: disable-msg=E1101 hubQ.send_ready(self.__class__.__name__) - hubQ.send_message(self.__class__.__name__, self.status) return ret From 24787f02e80162129256dc57dc3d491f00080370 Mon Sep 17 00:00:00 2001 From: Matej Tyc Date: Thu, 13 Oct 2022 17:19:17 +0200 Subject: [PATCH 2/2] Increase robustness of fetching state detection It is not completely practical to rely on locks alone, and we can elliminate some corner cases by looking whether well-known UI threads exist. --- org_fedora_oscap/gui/spokes/oscap.py | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/org_fedora_oscap/gui/spokes/oscap.py b/org_fedora_oscap/gui/spokes/oscap.py index 37b9681..97c4553 100644 --- a/org_fedora_oscap/gui/spokes/oscap.py +++ b/org_fedora_oscap/gui/spokes/oscap.py @@ -389,11 +389,14 @@ def _render_selected(self, column, renderer, model, itr, user_data=None): else: renderer.set_property("stock-id", None) + def _still_fetching(self): + return self._fetching or threadMgr.get('OSCAPguiWaitForDataFetchThread') + def _fetch_data_and_initialize(self): """Fetch data from a specified URL and initialize everything.""" with self._fetch_flag_lock: - if self._fetching: + if self._still_fetching(): # prevent multiple fetches running simultaneously return self._fetching = True @@ -940,7 +943,7 @@ def _refresh_ui(self): # hide the progress box, no progress now with self._fetch_flag_lock: - if not self._fetching: + if not self._still_fetching(): really_hide(self._progress_box) self._content_url_entry.set_sensitive(True) @@ -1165,7 +1168,7 @@ def on_fetch_button_clicked(self, *args): """Handler for the Fetch button""" with self._fetch_flag_lock: - if self._fetching: + if self._still_fetching(): # some other fetching/pre-processing running, give up log.warn( "OSCAP Addon: "