mirror of
https://github.com/KeksPirates/SoftwareManager.git
synced 2026-08-03 17:39:42 +02:00
Merge pull request #46 from KeksPirates/feat/improve-code-quality
feat: improve output statements on exception blocks
This commit is contained in:
@@ -1,4 +1,4 @@
|
|||||||
from core.utils.logging.loghandler import consoleLog
|
from core.utils.logging.logs import consoleLog
|
||||||
import requests
|
import requests
|
||||||
import re
|
import re
|
||||||
|
|
||||||
|
|||||||
@@ -86,8 +86,8 @@ class ContextMenu:
|
|||||||
if state.dl_session:
|
if state.dl_session:
|
||||||
try:
|
try:
|
||||||
state.dl_session.remove_torrent(magnetdl)
|
state.dl_session.remove_torrent(magnetdl)
|
||||||
except Exception:
|
except Exception as e:
|
||||||
pass
|
consoleLog(f"Exception while removing download from LibTorrent: {e}")
|
||||||
|
|
||||||
consoleLog(f"Cancelled download: {torrent_name}", True)
|
consoleLog(f"Cancelled download: {torrent_name}", True)
|
||||||
|
|
||||||
@@ -125,8 +125,8 @@ class ContextMenu:
|
|||||||
try:
|
try:
|
||||||
state.dl_session.remove_torrent(magnetdl)
|
state.dl_session.remove_torrent(magnetdl)
|
||||||
time.sleep(0.5)
|
time.sleep(0.5)
|
||||||
except Exception:
|
except Exception as e:
|
||||||
pass
|
consoleLog(f"Exception while removing download from LibTorrent: {e}")
|
||||||
if download_path and os.path.exists(download_path):
|
if download_path and os.path.exists(download_path):
|
||||||
import shutil
|
import shutil
|
||||||
last_error = None
|
last_error = None
|
||||||
|
|||||||
@@ -3,6 +3,7 @@ from core.interface.dialogs.hoverrowdelegate import HoverRowDelegate
|
|||||||
from core.interface.dialogs.downloadmodel import DownloadModel
|
from core.interface.dialogs.downloadmodel import DownloadModel
|
||||||
from core.interface.dialogs.theme import _table_stylesheet
|
from core.interface.dialogs.theme import _table_stylesheet
|
||||||
from PySide6.QtWidgets import QTableView, QHeaderView
|
from PySide6.QtWidgets import QTableView, QHeaderView
|
||||||
|
from core.utils.logging.logs import consoleLog
|
||||||
from PySide6 import QtWidgets
|
from PySide6 import QtWidgets
|
||||||
from PySide6.QtCore import Qt
|
from PySide6.QtCore import Qt
|
||||||
|
|
||||||
@@ -84,5 +85,5 @@ def download_list_update(self):
|
|||||||
if editor and delegate:
|
if editor and delegate:
|
||||||
try:
|
try:
|
||||||
delegate.setEditorData(editor, idx)
|
delegate.setEditorData(editor, idx)
|
||||||
except RuntimeError:
|
except RuntimeError as e:
|
||||||
pass
|
consoleLog(f"Exception while updating download list: {e}")
|
||||||
|
|||||||
@@ -117,8 +117,8 @@ class DownloadModel(QAbstractTableModel):
|
|||||||
subprocess.Popen(["xdg-open", save_path])
|
subprocess.Popen(["xdg-open", save_path])
|
||||||
elif platform.system() == "Darwin":
|
elif platform.system() == "Darwin":
|
||||||
subprocess.Popen(["open", save_path])
|
subprocess.Popen(["open", save_path])
|
||||||
except Exception:
|
except Exception as e:
|
||||||
pass
|
consoleLog(f"Exception while opening file: {e}")
|
||||||
return
|
return
|
||||||
|
|
||||||
is_paused = status.paused
|
is_paused = status.paused
|
||||||
|
|||||||
@@ -1,3 +1,4 @@
|
|||||||
|
from core.utils.logging.logs import consoleLog
|
||||||
from core.utils.data.state import state
|
from core.utils.data.state import state
|
||||||
from PySide6.QtCore import QEvent
|
from PySide6.QtCore import QEvent
|
||||||
|
|
||||||
@@ -30,6 +31,6 @@ def eventFilter(self, obj, event):
|
|||||||
old_row = self._hovered_row
|
old_row = self._hovered_row
|
||||||
self._hovered_row = -1
|
self._hovered_row = -1
|
||||||
self._invalidate_hover_row(old_row)
|
self._invalidate_hover_row(old_row)
|
||||||
except (RuntimeError, AttributeError):
|
except (RuntimeError, AttributeError) as e:
|
||||||
pass
|
consoleLog(f"Exception in EventFilter: {e}")
|
||||||
return super(type(self), self).eventFilter(obj, event)
|
return super(type(self), self).eventFilter(obj, event)
|
||||||
|
|||||||
@@ -86,7 +86,7 @@ class MainWindow(QtWidgets.QMainWindow, QWidget):
|
|||||||
except Exception as e:
|
except Exception as e:
|
||||||
consoleLog(f"Could not set app ID: {e}")
|
consoleLog(f"Could not set app ID: {e}")
|
||||||
|
|
||||||
core.interface.dialogs.update
|
# core.interface.dialogs.update
|
||||||
|
|
||||||
self.setWindowTitle("Software Manager")
|
self.setWindowTitle("Software Manager")
|
||||||
self.setGeometry(100, 100, 800, 600)
|
self.setGeometry(100, 100, 800, 600)
|
||||||
@@ -131,7 +131,7 @@ class MainWindow(QtWidgets.QMainWindow, QWidget):
|
|||||||
containerLayout.addWidget(self.searchbar)
|
containerLayout.addWidget(self.searchbar)
|
||||||
containerLayout.addWidget(state.trackertable)
|
containerLayout.addWidget(state.trackertable)
|
||||||
|
|
||||||
core.interface.dialogs.hoverrowdelegate
|
# core.interface.dialogs.hoverrowdelegate
|
||||||
self.download_model = DownloadModel()
|
self.download_model = DownloadModel()
|
||||||
self.downloadList = _create_download_list(self)
|
self.downloadList = _create_download_list(self)
|
||||||
self.downloadList.viewport().installEventFilter(self)
|
self.downloadList.viewport().installEventFilter(self)
|
||||||
|
|||||||
Reference in New Issue
Block a user