Added multi-selection in rules list for context menu, open folder after build, reduced the opening time of settings and fixed focus when opening settings

Author: SystemXFiles

build_portable.py

@@ -7,6 +7,7 @@
 
 from constants.app_info import APP_NAME, APP_VERSION, APP_AUTHOR, APP_NAME_WITH_VERSION
 from constants.files import CONFIG_FILE_NAME
+from util.files import explore
 
 # Setting paths and configuration parameters
 VERSION_FILE = "versionfile.txt"
@@ -52,8 +53,10 @@
 ])
 
 # Creating an archive of the built application
-shutil.make_archive(APP_DIST_WITH_VERSION, 'zip', APP_DIST)
+archive_file_path = shutil.make_archive(APP_DIST_WITH_VERSION, 'zip', APP_DIST)
 
 # Copying the configuration file for tests into the built application
 if os.path.isfile(CONFIG_FILE_FOR_TESTS):
     shutil.copyfile(CONFIG_FILE_FOR_TESTS, CONFIG_FILE_IN_APP_DIST)
+
+explore(archive_file_path)

src/model/service.py

@@ -1,14 +1,12 @@
 from dataclasses import dataclass
-from typing import Optional
 
 
 @dataclass
 class Service:
     """
     The Service class represents information about a Windows service.
 
-    It includes attributes such as service process ID (pid), service name, display name, description, current status,
-    and binary path.
+    It includes attributes such as service process ID (pid), name and current status.
     """
 
     pid: int
@@ -21,22 +19,7 @@ class Service:
     The name of the Windows service.
     """
 
-    display_name: str
-    """
-    The display name of the Windows service.
-    """
-
-    description: Optional[str]
-    """
-    A description of the Windows service. Default is None (no description available).
-    """
-
     status: str
     """
     The current status of the Windows service.
     """
-
-    bin_path: str
-    """
-    The binary path of the Windows service, specifying the location of the service executable.
-    """

src/service/processes_info_service.py

@@ -1,5 +1,6 @@
 import subprocess
 from abc import ABC
+from typing import Optional
 
 import psutil
 from psutil import NoSuchProcess
@@ -12,7 +13,6 @@
 class ProcessesInfoService(ABC):
     """
     The ProcessesInfoService class provides methods for retrieving information about running processes.
-    It is an abstract base class (ABC) to be subclassed by specific implementation classes.
     """
 
     _cache: dict[int, Process] = {}
@@ -27,7 +27,7 @@ def get_processes(cls) -> dict[int, Process]:
         """
 
         cache = cls._cache
-        services = ServicesInfoService.get_services()
+        services: Optional[dict] = None
         pids = set(psutil.pids())
 
         for pid in pids:
@@ -48,6 +48,9 @@ def get_processes(cls) -> dict[int, Process]:
                         process.is_new = False
                         continue
 
+                if services is None:
+                    services = ServicesInfoService.get_running_services()
+
                 service = services.get(pid)
                 info = process_info.as_dict(attrs=[
                     'name', 'exe', 'cmdline',

src/service/rules_service.py

@@ -24,7 +24,6 @@
 class RulesService(ABC):
     """
     The RulesService class provides methods for applying rules to processes and services.
-    It is an abstract base class (ABC) to be subclassed by specific implementation classes.
     """
 
     __ignore_pids: set[int] = {0, os.getpid()}

src/service/services_info_service.py

@@ -4,7 +4,6 @@
 from psutil import STATUS_STOPPED, NoSuchProcess, ZombieProcess, AccessDenied
 from psutil._pswindows import WindowsService
 
-from constants.log import LOG
 from model.service import Service
 from util.decorators import suppress_exception
 
@@ -23,37 +22,48 @@
 
 class ServicesInfoService(ABC):
     """
-    The ServicesInfoService class provides methods for retrieving information about running services.
-    It is an abstract base class (ABC) to be subclassed by specific implementation classes.
+    The ServicesInfoService class provides methods for retrieving information about Windows services.
     """
 
     @staticmethod
-    def get_services() -> dict[int, Service]:
-        """
-        Get a dictionary of running services and their information.
-
-        Returns:
-            dict[int, Service]: A dictionary where keys are process IDs (pids) and values are Service objects
-            representing the running services.
-        """
+    def get_running_services() -> dict[int, Service]:
         result: dict[int, Service] = {}
 
         for service in psutil.win_service_iter():
             try:
-                info = service.as_dict()
+                # noinspection PyUnresolvedReferences
+                info = service._query_status()
+                status = info['status']
+                pid = info['pid']
 
-                if info['status'] == STATUS_STOPPED:
+                if pid == STATUS_STOPPED:
                     continue
 
-                result[info['pid']] = Service(
-                    info['pid'],
-                    info['name'],
-                    info['display_name'],
-                    info['description'],
-                    info['status'],
-                    info['binpath']
+                result[pid] = Service(
+                    pid,
+                    service.name(),
+                    status
                 )
             except NoSuchProcess:
-                LOG.warning(f"No such service: {service.name}")
+                pass
+
+        return result
+
+    @staticmethod
+    def get_services() -> list[Service]:
+        result: list[Service] = []
+
+        for service in psutil.win_service_iter():
+            try:
+                # noinspection PyUnresolvedReferences
+                info = service._query_status()
+
+                result.append(Service(
+                    info['pid'],
+                    service.name(),
+                    info['status']
+                ))
+            except NoSuchProcess:
+                pass
 
         return result

src/ui/settings.py

@@ -41,6 +41,7 @@ def __init__(self):
     def _after_init(self):
         self._setup_tooltips()
         self._setup_binds()
+
         self._tabs.load_data()
 
     def _setup_window(self):
@@ -355,6 +356,7 @@ def settings():
 
             try:
                 __app = Settings()
+                __app.after_idle(__app.to_front)
                 __app.mainloop()
             finally:
                 __app = None

src/ui/widget/settings/tabs/processes/process_tab.py

@@ -82,18 +82,16 @@ def _update_process_list(self):
                 LOG.exception("Update process list error")
 
     def _refresh(self):
-        LOG.info("Refreshing process list...")
-
         def update_process_list():
-            LOG.info("Updating process list...")
+            # LOG.info("Updating process list...")
 
             try:
                 self._update_process_list()
             finally:
                 self._refresh_state()
 
         def load_data():
-            LOG.info("Loading data...")
+            # LOG.info("Loading data...")
 
             try:
                 self.process_list.set_data(ProcessesInfoService.get_processes())

src/ui/widget/settings/tabs/rules/rules_list.py

@@ -88,9 +88,11 @@ def _show_context_menu(self, event):
         context_menu = self._context_menu
         process_list = self
         row_id = process_list.identify_row(event.y)
+        selected_items = process_list.selection()
 
         if row_id:
-            process_list.selection_set(row_id)
+            if selected_items and len(selected_items) == 1:
+                process_list.selection_set(row_id)
             context_menu.post(event.x_root, event.y_root)
 
     def _errors(self) -> dict[Any, Any]:

src/util/utils.py

@@ -161,7 +161,8 @@ def get_values_from_enum(annotation):
     return values
 
 
-def get_icon_from_exe(exe_path, icon_index=0, large=False, try_load_another_if_absent=True):
+@cache
+def get_icon_from_exe(exe_path, icon_index=0, large=False):
     if large:
         icon_size = (
             win32api.GetSystemMetrics(win32con.SM_CXICON),