e10f7dd7a7
Remove no longer used google scripts
Remove convenience files from comicataggerlib and import comicapi directly
Add type-hints to facilitate auto-complete tools
Make PyQt5 code more compatible with PyQt6
Implement automatic tooling
isort and black for code formatting
Line length has been set to 120
flake8 for code standards with exceptions:
E203 - Whitespace before ':' - format compatiblity with black
E501 - Line too long - flake8 line limit cannot be set
E722 - Do not use bare except - fixing bare except statements is a
lot of overhead and there are already
many in the codebase
These changes, along with some manual fixes creates much more readable code.
See examples below:
diff --git a/comicapi/comet.py b/comicapi/comet.py
index d1741c5..52dc195 100644
--- a/comicapi/comet.py
+++ b/comicapi/comet.py
@@ -166,7 +166,2 @@ class CoMet:
- if credit['role'].lower() in set(self.editor_synonyms):
- ET.SubElement(
- root,
- 'editor').text = "{0}".format(
- credit['person'])
@@ -174,2 +169,4 @@ class CoMet:
self.indent(root)
+ if credit["role"].lower() in set(self.editor_synonyms):
+ ET.SubElement(root, "editor").text = str(credit["person"])
diff --git a/comictaggerlib/autotagmatchwindow.py b/comictaggerlib/autotagmatchwindow.py
index 4338176..9219f01 100644
--- a/comictaggerlib/autotagmatchwindow.py
+++ b/comictaggerlib/autotagmatchwindow.py
@@ -63,4 +63,3 @@ class AutoTagMatchWindow(QtWidgets.QDialog):
self.skipButton, QtWidgets.QDialogButtonBox.ActionRole)
- self.buttonBox.button(QtWidgets.QDialogButtonBox.Ok).setText(
- "Accept and Write Tags")
+ self.buttonBox.button(QtWidgets.QDialogButtonBox.StandardButton.Ok).setText("Accept and Write Tags")
diff --git a/comictaggerlib/cli.py b/comictaggerlib/cli.py
index 688907d..dbd0c2e 100644
--- a/comictaggerlib/cli.py
+++ b/comictaggerlib/cli.py
@@ -293,7 +293,3 @@ def process_file_cli(filename, opts, settings, match_results):
if opts.raw:
- print((
- "{0}".format(
- str(
- ca.readRawCIX(),
- errors='ignore'))))
+ print(ca.read_raw_cix())
else:
240 lines
8.5 KiB
Python
240 lines
8.5 KiB
Python
"""A PyQT4 dialog to select from automated issue matches"""
|
|
|
|
# Copyright 2012-2014 Anthony Beville
|
|
|
|
# Licensed under the Apache License, Version 2.0 (the "License");
|
|
# you may not use this file except in compliance with the License.
|
|
# You may obtain a copy of the License at
|
|
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
# distributed under the License is distributed on an "AS IS" BASIS,
|
|
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
# See the License for the specific language governing permissions and
|
|
# limitations under the License.
|
|
|
|
import os
|
|
from typing import List, Optional
|
|
|
|
from PyQt5 import QtCore, QtGui, QtWidgets, uic
|
|
|
|
from comicapi.comicarchive import MetaDataStyle
|
|
from comictaggerlib.coverimagewidget import CoverImageWidget
|
|
from comictaggerlib.resulttypes import MultipleMatch
|
|
from comictaggerlib.settings import ComicTaggerSettings
|
|
from comictaggerlib.ui.qtutils import reduce_widget_font_size
|
|
|
|
|
|
class AutoTagMatchWindow(QtWidgets.QDialog):
|
|
volume_id = 0
|
|
|
|
def __init__(self, parent, match_set_list: List[MultipleMatch], style, fetch_func):
|
|
super().__init__(parent)
|
|
|
|
uic.loadUi(ComicTaggerSettings.get_ui_file("matchselectionwindow.ui"), self)
|
|
|
|
self.current_match_set: Optional[MultipleMatch] = None
|
|
|
|
self.altCoverWidget = CoverImageWidget(self.altCoverContainer, CoverImageWidget.AltCoverMode)
|
|
gridlayout = QtWidgets.QGridLayout(self.altCoverContainer)
|
|
gridlayout.addWidget(self.altCoverWidget)
|
|
gridlayout.setContentsMargins(0, 0, 0, 0)
|
|
|
|
self.archiveCoverWidget = CoverImageWidget(self.archiveCoverContainer, CoverImageWidget.ArchiveMode)
|
|
gridlayout = QtWidgets.QGridLayout(self.archiveCoverContainer)
|
|
gridlayout.addWidget(self.archiveCoverWidget)
|
|
gridlayout.setContentsMargins(0, 0, 0, 0)
|
|
|
|
reduce_widget_font_size(self.twList)
|
|
reduce_widget_font_size(self.teDescription, 1)
|
|
|
|
self.setWindowFlags(
|
|
QtCore.Qt.WindowType(
|
|
self.windowFlags()
|
|
| QtCore.Qt.WindowType.WindowSystemMenuHint
|
|
| QtCore.Qt.WindowType.WindowMaximizeButtonHint
|
|
)
|
|
)
|
|
|
|
self.skipButton = QtWidgets.QPushButton("Skip to Next")
|
|
self.buttonBox.addButton(self.skipButton, QtWidgets.QDialogButtonBox.ButtonRole.ActionRole)
|
|
self.buttonBox.button(QtWidgets.QDialogButtonBox.StandardButton.Ok).setText("Accept and Write Tags")
|
|
|
|
self.match_set_list = match_set_list
|
|
self.style = style
|
|
self.fetch_func = fetch_func
|
|
|
|
self.current_match_set_idx = 0
|
|
|
|
self.twList.currentItemChanged.connect(self.current_item_changed)
|
|
self.twList.cellDoubleClicked.connect(self.cell_double_clicked)
|
|
self.skipButton.clicked.connect(self.skip_to_next)
|
|
|
|
self.update_data()
|
|
|
|
def update_data(self):
|
|
|
|
self.current_match_set = self.match_set_list[self.current_match_set_idx]
|
|
|
|
if self.current_match_set_idx + 1 == len(self.match_set_list):
|
|
self.buttonBox.button(QtWidgets.QDialogButtonBox.StandardButton.Cancel).setDisabled(True)
|
|
self.skipButton.setText("Skip")
|
|
|
|
self.set_cover_image()
|
|
self.populate_table()
|
|
self.twList.resizeColumnsToContents()
|
|
self.twList.selectRow(0)
|
|
|
|
path = self.current_match_set.ca.path
|
|
self.setWindowTitle(
|
|
"Select correct match or skip ({0} of {1}): {2}".format(
|
|
self.current_match_set_idx + 1,
|
|
len(self.match_set_list),
|
|
os.path.split(path)[1],
|
|
)
|
|
)
|
|
|
|
def populate_table(self):
|
|
|
|
while self.twList.rowCount() > 0:
|
|
self.twList.removeRow(0)
|
|
|
|
self.twList.setSortingEnabled(False)
|
|
|
|
row = 0
|
|
for match in self.current_match_set.matches:
|
|
self.twList.insertRow(row)
|
|
|
|
item_text = match["series"]
|
|
item = QtWidgets.QTableWidgetItem(item_text)
|
|
item.setData(QtCore.Qt.ItemDataRole.ToolTipRole, item_text)
|
|
item.setData(QtCore.Qt.ItemDataRole.UserRole, (match,))
|
|
item.setFlags(QtCore.Qt.ItemFlag.ItemIsSelectable | QtCore.Qt.ItemFlag.ItemIsEnabled)
|
|
self.twList.setItem(row, 0, item)
|
|
|
|
if match["publisher"] is not None:
|
|
item_text = str(match["publisher"])
|
|
else:
|
|
item_text = "Unknown"
|
|
item = QtWidgets.QTableWidgetItem(item_text)
|
|
item.setData(QtCore.Qt.ItemDataRole.ToolTipRole, item_text)
|
|
item.setFlags(QtCore.Qt.ItemFlag.ItemIsSelectable | QtCore.Qt.ItemFlag.ItemIsEnabled)
|
|
self.twList.setItem(row, 1, item)
|
|
|
|
month_str = ""
|
|
year_str = "????"
|
|
if match["month"] is not None:
|
|
month_str = f"-{int(match['month']):02d}"
|
|
if match["year"] is not None:
|
|
year_str = str(match["year"])
|
|
|
|
item_text = year_str + month_str
|
|
item = QtWidgets.QTableWidgetItem(item_text)
|
|
item.setData(QtCore.Qt.ItemDataRole.ToolTipRole, item_text)
|
|
item.setFlags(QtCore.Qt.ItemFlag.ItemIsSelectable | QtCore.Qt.ItemFlag.ItemIsEnabled)
|
|
self.twList.setItem(row, 2, item)
|
|
|
|
item_text = match["issue_title"]
|
|
if item_text is None:
|
|
item_text = ""
|
|
item = QtWidgets.QTableWidgetItem(item_text)
|
|
item.setData(QtCore.Qt.ItemDataRole.ToolTipRole, item_text)
|
|
item.setFlags(QtCore.Qt.ItemFlag.ItemIsSelectable | QtCore.Qt.ItemFlag.ItemIsEnabled)
|
|
self.twList.setItem(row, 3, item)
|
|
|
|
row += 1
|
|
|
|
self.twList.resizeColumnsToContents()
|
|
self.twList.setSortingEnabled(True)
|
|
self.twList.sortItems(2, QtCore.Qt.SortOrder.AscendingOrder)
|
|
self.twList.selectRow(0)
|
|
self.twList.resizeColumnsToContents()
|
|
self.twList.horizontalHeader().setStretchLastSection(True)
|
|
|
|
def cell_double_clicked(self, r, c):
|
|
self.accept()
|
|
|
|
def current_item_changed(self, curr, prev):
|
|
|
|
if curr is None:
|
|
return
|
|
if prev is not None and prev.row() == curr.row():
|
|
return
|
|
|
|
self.altCoverWidget.set_issue_id(self.current_match()["issue_id"])
|
|
if self.current_match()["description"] is None:
|
|
self.teDescription.setText("")
|
|
else:
|
|
self.teDescription.setText(self.current_match()["description"])
|
|
|
|
def set_cover_image(self):
|
|
ca = self.current_match_set.ca
|
|
self.archiveCoverWidget.set_archive(ca)
|
|
|
|
def current_match(self):
|
|
row = self.twList.currentRow()
|
|
match = self.twList.item(row, 0).data(QtCore.Qt.ItemDataRole.UserRole)[0]
|
|
return match
|
|
|
|
def accept(self):
|
|
|
|
self.save_match()
|
|
self.current_match_set_idx += 1
|
|
|
|
if self.current_match_set_idx == len(self.match_set_list):
|
|
# no more items
|
|
QtWidgets.QDialog.accept(self)
|
|
else:
|
|
self.update_data()
|
|
|
|
def skip_to_next(self):
|
|
self.current_match_set_idx += 1
|
|
|
|
if self.current_match_set_idx == len(self.match_set_list):
|
|
# no more items
|
|
QtWidgets.QDialog.reject(self)
|
|
else:
|
|
self.update_data()
|
|
|
|
def reject(self):
|
|
reply = QtWidgets.QMessageBox.question(
|
|
self,
|
|
"Cancel Matching",
|
|
"Are you sure you wish to cancel the matching process?",
|
|
QtWidgets.QMessageBox.StandardButton.Yes,
|
|
QtWidgets.QMessageBox.StandardButton.No,
|
|
)
|
|
|
|
if reply == QtWidgets.QMessageBox.StandardButton.No:
|
|
return
|
|
|
|
QtWidgets.QDialog.reject(self)
|
|
|
|
def save_match(self):
|
|
|
|
match = self.current_match()
|
|
ca = self.current_match_set.ca
|
|
|
|
md = ca.read_metadata(self.style)
|
|
if md.is_empty:
|
|
md = ca.metadata_from_filename()
|
|
|
|
# now get the particular issue data
|
|
cv_md = self.fetch_func(match)
|
|
if cv_md is None:
|
|
QtWidgets.QMessageBox.critical(
|
|
self, "Network Issue", "Could not connect to Comic Vine to get issue details!"
|
|
)
|
|
return
|
|
|
|
QtWidgets.QApplication.setOverrideCursor(QtGui.QCursor(QtCore.Qt.CursorShape.WaitCursor))
|
|
md.overlay(cv_md)
|
|
success = ca.write_metadata(md, self.style)
|
|
ca.load_cache([MetaDataStyle.CBI, MetaDataStyle.CIX])
|
|
|
|
QtWidgets.QApplication.restoreOverrideCursor()
|
|
|
|
if not success:
|
|
QtWidgets.QMessageBox.warning(self, "Write Error", "Saving the tags to the archive seemed to fail!")
|