From 824b5c789aeb7ba105e3ad48b3f34dcaadcfd8a5 Mon Sep 17 00:00:00 2001
From: e3dio <85405955+e3dio@users.noreply.github.com>
Date: Fri, 3 Jan 2025 22:19:01 -0700
Subject: [PATCH] fix #1245

---
 ui/opensnitch/dialogs/stats.py | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/ui/opensnitch/dialogs/stats.py b/ui/opensnitch/dialogs/stats.py
index 85712383..54821667 100644
--- a/ui/opensnitch/dialogs/stats.py
+++ b/ui/opensnitch/dialogs/stats.py
@@ -1057,9 +1057,6 @@ class StatsDialog(QtWidgets.QDialog, uic.loadUiType(DIALOG_UI_PATH)[0]):
         if nid == None:
             return
         self._notifications_sent[nid] = noti
-        # FIXME: we shouldn't refresh the view here. We should wait for a
-        # notification reply on self._cb_notification_callback
-        self._refresh_active_table()
 
     # https://stackoverflow.com/questions/40225270/copy-paste-multiple-items-from-qtableview-in-pyqt4
     def _copy_selected_rows(self):
@@ -1570,6 +1567,7 @@ class StatsDialog(QtWidgets.QDialog, uic.loadUiType(DIALOG_UI_PATH)[0]):
                 name = model.index(idx.row(), self.COL_R_NAME).data()
                 node = model.index(idx.row(), self.COL_R_NODE).data()
                 self._del_rule(name, node)
+            self._refresh_active_table()
 
         elif cur_idx == self.TAB_RULES and self.alertsTable.isVisible():
             for idx in selection: