Merge pull request #31 from oleksis/master

Fix issue #27

.github/workflows/release.yml

@@ -30,6 +30,7 @@ jobs:
       id: build_x64_release
       run: |
         C:\Miniconda\condabin\conda.bat activate base
+        python -m pip install --upgrade pip setuptools wheel
         python setup.py pyinstaller
     - name: Create Release
       id: create_release

.gitignore

@@ -1,14 +1,29 @@
 *.pyc
-*.mp3
-*.mp4
+*.srt
+*.ttml
+*.sbv
+*.vtt
 *.flv
+*.mp4
+*.m4a
+*.m4v
+*.mp3
+*.3gp
+*.wav
+*.ape
+*.mkv
+*.swf
+*.part
+.idea
 .vscode
 env
-build
-dist
+env/
+dist/
+build/
+*venv/
+*venv.back/
 *.spec
 *.egg-info
 old/
 *.part
-.idea
-.DS_Store
+.DS_Store

GUI/AboutDialog.py

@@ -4,6 +4,6 @@
 
 class AboutDialog(QtWidgets.QDialog):
     def __init__(self, parent=None):
-        super(AboutDialog, self).__init__(parent, QtCore.Qt.WindowSystemMenuHint | QtCore.Qt.WindowTitleHint)
+        super(AboutDialog, self).__init__(parent, QtCore.Qt.WindowSystemMenuHint | QtCore.Qt.WindowTitleHint)  # type: ignore
         self.ui = Ui_Dialog()
         self.ui.setupUi(self)

GUI/BatchAddUrls.py

@@ -4,7 +4,9 @@
 
 class BatchAddDialogue(QtWidgets.QDialog):
     def __init__(self, parent=None):
-        super(BatchAddDialogue, self).__init__(parent, QtCore.Qt.WindowSystemMenuHint | QtCore.Qt.WindowTitleHint)
+        super(BatchAddDialogue, self).__init__(
+            parent, QtCore.Qt.WindowSystemMenuHint | QtCore.Qt.WindowTitleHint
+        )  # type: ignore
         self.ui = Ui_BatchAdd()
         self.ui.setupUi(self)
         self.download = False
@@ -14,24 +16,24 @@ def __init__(self, parent=None):
 
     def browse_clicked(self):
         file_name, _ = QtWidgets.QFileDialog.getOpenFileName(
-            self, "Select txt file", filter='*.txt'
+            self, "Select txt file", filter="*.txt"
         )
 
-        if file_name == '':
+        if not file_name:
             return
-        with open(file_name, 'r') as file_data:
+
+        with open(file_name, "r") as file_data:
             for line in file_data.readlines():
                 self.ui.UrlList.append(line.strip())
 
     def add_clicked(self):
-        if str(self.ui.UrlList.toPlainText()).strip() == '':
+        _have_urls = str(self.ui.UrlList.toPlainText()).strip()  # != ''
+        if not _have_urls:
             QtWidgets.QMessageBox.information(self, "Error!", "No urls given!")
-            return
         else:
             self.download = True
             self.close()
-    
+
     def close_clicked(self):
         self.download = False
         self.close()
-

GUI/LicenseDialog.py

@@ -4,7 +4,7 @@
 
 class LicenseDialogue(QtWidgets.QDialog):
     def __init__(self, parent=None):
-        super(LicenseDialogue, self).__init__(parent, QtCore.Qt.WindowSystemMenuHint | QtCore.Qt.WindowTitleHint)
+        super(LicenseDialogue, self).__init__(parent, QtCore.Qt.WindowSystemMenuHint | QtCore.Qt.WindowTitleHint)  # type: ignore
         self.ui = Ui_Dialog()
         self.ui.setupUi(self)
-        self.ui.ExitButton.clicked.connect(self.close)
+        self.ui.ExitButton.clicked.connect(self.close)