Boria138/PortProtonQt
3
3
Fork 3
Code Issues 1 Pull Requests Actions Releases 3 Activity

fix: Allow context menu for PortProton games without valid exe

Browse Source 
Signed-off-by: Boris Yumankulov <boria138@altlinux.org>
This commit is contained in:
Boris Yumankulov
2025-07-20 12:31:36 +05:00
parent e91037708a
commit 04aaf68e36
1 changed files with 17 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files

36
portprotonqt/context_menu_manager.py
View File

@ -148,10 +148,7 @@ class ContextMenuManager:
return False return False
current_exe = os.path.basename(exe_path) current_exe = os.path.basename(exe_path)
# Check if the current_exe matches the target_exe in MainWindow return hasattr(self.parent, 'target_exe') and self.parent.target_exe == current_exe
if hasattr(self.parent, 'target_exe') and self.parent.target_exe == current_exe:
return True
return False
def show_context_menu(self, game_card, pos: QPoint): def show_context_menu(self, game_card, pos: QPoint):
""" """
@ -161,7 +158,6 @@ class ContextMenuManager:
game_card: The GameCard instance requesting the context menu. game_card: The GameCard instance requesting the context menu.
pos: The position (in widget coordinates) where the menu should appear. pos: The position (in widget coordinates) where the menu should appear.
""" """
def get_safe_icon(icon_name: str) -> QIcon: def get_safe_icon(icon_name: str) -> QIcon:
icon = self.theme_manager.get_icon(icon_name) icon = self.theme_manager.get_icon(icon_name)
if isinstance(icon, QIcon): if isinstance(icon, QIcon):
@ -173,7 +169,18 @@ class ContextMenuManager:
menu = QMenu(self.parent) menu = QMenu(self.parent)
menu.setStyleSheet(self.theme.CONTEXT_MENU_STYLE) menu.setStyleSheet(self.theme.CONTEXT_MENU_STYLE)
# Check if the game is running # For non-Steam and non-Epic games, check if exe exists
if game_card.game_source not in ("steam", "epic"):
exec_line = self._get_exec_line(game_card.name, game_card.exec_line)
exe_path = self._parse_exe_path(exec_line, game_card.name) if exec_line else None
if not exe_path:
# Show only "Delete from PortProton" if no valid exe
delete_action = menu.addAction(get_safe_icon("delete"), _("Delete from PortProton"))
delete_action.triggered.connect(lambda: self.delete_game(game_card.name, game_card.exec_line))
menu.exec(game_card.mapToGlobal(pos))
return
# Normal menu for games with valid exe or from Steam/Epic
is_running = self._is_game_running(game_card) is_running = self._is_game_running(game_card)
action_text = _("Stop Game") if is_running else _("Launch Game") action_text = _("Stop Game") if is_running else _("Launch Game")
action_icon = "stop" if is_running else "play" action_icon = "stop" if is_running else "play"
@ -697,15 +704,12 @@ Icon={icon_path}
return None return None
return exec_line return exec_line
def _parse_exe_path(self, exec_line, game_name): def _parse_exe_path(self, exec_line: str, game_name: str) -> str | None:
"""Parse the executable path from exec_line.""" """Parse the executable path from exec_line."""
try: try:
entry_exec_split = shlex.split(exec_line) entry_exec_split = shlex.split(exec_line)
if not entry_exec_split: if not entry_exec_split:
self.signals.show_warning_dialog.emit( logger.debug("Invalid executable command for '%s': %s", game_name, exec_line)
_("Error"),
_("Invalid executable command: {exec_line}").format(exec_line=exec_line)
)
return None return None
if entry_exec_split[0] == "env" and len(entry_exec_split) >= 3: if entry_exec_split[0] == "env" and len(entry_exec_split) >= 3:
exe_path = entry_exec_split[2] exe_path = entry_exec_split[2]
@ -714,17 +718,11 @@ Icon={icon_path}
else: else:
exe_path = entry_exec_split[-1] exe_path = entry_exec_split[-1]
if not exe_path or not os.path.exists(exe_path): if not exe_path or not os.path.exists(exe_path):
self.signals.show_warning_dialog.emit( logger.debug("Executable not found for '%s': %s", game_name, exe_path or "None")
_("Error"),
_("Executable not found: {path}").format(path=exe_path or "None")
)
return None return None
return exe_path return exe_path
except Exception as e: except Exception as e:
self.signals.show_warning_dialog.emit( logger.debug("Failed to parse executable for '%s': %s", game_name, e)
_("Error"),
_("Failed to parse executable: {error}").format(error=str(e))
)
return None return None
def _remove_file(self, file_path, error_message, success_message, game_name, location=""): def _remove_file(self, file_path, error_message, success_message, game_name, location=""):