In Windows, disable opening of safe PDF, make saving PDF required, and after safe PDF has been created, open Windows explorer with the safe PDF selected

dangerzone/settings_widget.py

@@ -1,5 +1,6 @@
 import os
 import subprocess
+import platform
 from PyQt5 import QtCore, QtGui, QtWidgets
 
 
@@ -20,16 +21,26 @@ class SettingsWidget(QtWidgets.QWidget):
         # Save safe version
         self.save_checkbox = QtWidgets.QCheckBox("Save safe PDF")
         self.save_checkbox.clicked.connect(self.update_ui)
+        self.save_label = QtWidgets.QLabel("Save safe PDF")  # For Windows
+        self.save_label.hide()
+        if platform.system() == "Windows":
+            # In Windows, users must save the PDF, since they can't open it
+            self.save_checkbox.setCheckState(QtCore.Qt.Checked)
+            self.save_checkbox.setEnabled(False)
+            self.save_checkbox.hide()
+            self.save_label.show()
         self.save_lineedit = QtWidgets.QLineEdit()
         self.save_lineedit.setReadOnly(True)
         self.save_browse_button = QtWidgets.QPushButton("Save as...")
         self.save_browse_button.clicked.connect(self.save_browse_button_clicked)
         save_layout = QtWidgets.QHBoxLayout()
         save_layout.addWidget(self.save_checkbox)
+        save_layout.addWidget(self.save_label)
         save_layout.addWidget(self.save_lineedit)
         save_layout.addWidget(self.save_browse_button)
         save_layout.addStretch()
 
+        if platform.system() != "Windows":
             # Open safe document
             self.open_checkbox = QtWidgets.QCheckBox(
                 "Open safe document after converting, using"
@@ -37,7 +48,9 @@ class SettingsWidget(QtWidgets.QWidget):
             self.open_checkbox.clicked.connect(self.update_ui)
             self.open_combobox = QtWidgets.QComboBox()
             for k in self.common.pdf_viewers:
-            self.open_combobox.addItem(k, QtCore.QVariant(self.common.pdf_viewers[k]))
+                self.open_combobox.addItem(
+                    k, QtCore.QVariant(self.common.pdf_viewers[k])
+                )
             open_layout = QtWidgets.QHBoxLayout()
             open_layout.addWidget(self.open_checkbox)
             open_layout.addWidget(self.open_combobox)
@@ -75,6 +88,7 @@ class SettingsWidget(QtWidgets.QWidget):
         layout.addWidget(self.dangerous_doc_label)
         layout.addSpacing(20)
         layout.addLayout(save_layout)
+        if platform.system() != "Windows":
             layout.addLayout(open_layout)
         layout.addLayout(ocr_layout)
         layout.addLayout(update_layout)
@@ -98,6 +112,7 @@ class SettingsWidget(QtWidgets.QWidget):
         if index != -1:
             self.ocr_combobox.setCurrentIndex(index)
 
+        if platform.system() != "Windows":
             if self.common.settings.get("open"):
                 self.open_checkbox.setCheckState(QtCore.Qt.Checked)
             else:
@@ -122,6 +137,11 @@ class SettingsWidget(QtWidgets.QWidget):
             self.update_checkbox.setEnabled(False)
 
     def update_ui(self):
+        if platform.system() == "Windows":
+            # Because the save checkbox is always checked in Windows, the
+            # start button can be enabled
+            self.start_button.setEnabled(True)
+        else:
             # Either save or open must be checked
             if (
                 self.save_checkbox.checkState() == QtCore.Qt.Checked
@@ -163,6 +183,7 @@ class SettingsWidget(QtWidgets.QWidget):
             "ocr", self.ocr_checkbox.checkState() == QtCore.Qt.Checked
         )
         self.common.settings.set("ocr_language", self.ocr_combobox.currentText())
+        if platform.system() != "Windows":
             self.common.settings.set(
                 "open", self.open_checkbox.checkState() == QtCore.Qt.Checked
             )

dangerzone/tasks_widget.py

@@ -1,6 +1,8 @@
 import shutil
 import tempfile
 import os
+import platform
+import subprocess
 from PyQt5 import QtCore, QtGui, QtWidgets
 
 from .tasks import PullImageTask, BuildContainerTask, ConvertToPixels, ConvertToPDF
@@ -97,6 +99,13 @@ class TasksWidget(QtWidgets.QWidget):
             dest_filename = tmp[1]
         shutil.move(source_filename, dest_filename)
 
+        # In Windows, open Explorer with the safe PDF in focus
+        if platform.system() == "Windows":
+            dest_filename_windows = dest_filename.replace("/", "\\")
+            subprocess.Popen(
+                f'explorer.exe /select,"{dest_filename_windows}"', shell=True
+            )
+
         # Open
         if self.common.settings.get("open"):
             self.common.open_pdf_viewer(dest_filename)

poetry.lock

@@ -204,7 +204,7 @@ version = "12.7.1"
 [[package]]
 category = "main"
 description = "Python for Window Extensions"
-marker = "sys_platform == \"win32\""
+marker = "sys_platform == \"windows\""
 name = "pywin32"
 optional = false
 python-versions = "*"
@@ -284,7 +284,7 @@ python-versions = "*"
 version = "1.4.9"
 
 [metadata]
-content-hash = "28c51f60ace8c8578c21279f8c46ba095d0ec1f1f86bb155c24cba67ac3291ee"
+content-hash = "8a375835b3ed53e46bb4aaf31cb5cca43d33ed03ac217cf247ed65724d3ee3a8"
 python-versions = "^3.7"
 
 [metadata.files]

pyproject.toml

@@ -15,13 +15,13 @@ pyxdg = {version = "^0.26", platform = "linux"}
 pyobjc-core = {version = "^6.1", platform = "darwin"}
 pyobjc-framework-launchservices = {version = "^6.1", platform = "darwin"}
 macholib = "^1.14"
-pywin32 = {version = "^227", platform = "win32"}
+pywin32 = {version = "^227", platform = "windows"}
 wmi = {version = "^1.4.9", platform = "windows"}
 
 [tool.poetry.dev-dependencies]
 black = "^19.10b0"
 pyinstaller = {version = "^3.6", platform = "darwin"}
-setuptools = {version = "^45.2.0", platform = "win32"}
+setuptools = {version = "^45.2.0", platform = "windows"}
 
 [build-system]
 requires = ["poetry>=0.12"]