diff --git a/src/calibre/gui2/dialogs/check_library.py b/src/calibre/gui2/dialogs/check_library.py index d4265aaad1..1328d541fc 100644 --- a/src/calibre/gui2/dialogs/check_library.py +++ b/src/calibre/gui2/dialogs/check_library.py @@ -266,25 +266,25 @@ class CheckLibraryDialog(QDialog): tl.setData(1, Qt.ItemDataRole.UserRole, self.is_fixable) tl.setText(1, _('(fixable)')) tl.setFlags(Qt.ItemFlag.ItemIsEnabled | Qt.ItemFlag.ItemIsUserCheckable) - tl.setCheckState(1, False) + tl.setCheckState(1, Qt.CheckState.Unchecked) else: tl.setData(1, Qt.ItemDataRole.UserRole, self.is_deletable) tl.setData(2, Qt.ItemDataRole.UserRole, self.is_deletable) tl.setText(1, _('(deletable)')) tl.setFlags(Qt.ItemFlag.ItemIsEnabled | Qt.ItemFlag.ItemIsUserCheckable) - tl.setCheckState(1, False) + tl.setCheckState(1, Qt.CheckState.Unchecked) if attr == 'extra_covers': tl.setData(2, Qt.ItemDataRole.UserRole, self.is_deletable) tl.setText(2, _('(deletable)')) tl.setFlags(Qt.ItemFlag.ItemIsEnabled | Qt.ItemFlag.ItemIsUserCheckable) - tl.setCheckState(2, False) + tl.setCheckState(2, Qt.CheckState.Unchecked) self.top_level_items[attr] = tl for problem in list_: it = Item() if checkable: it.setFlags(Qt.ItemFlag.ItemIsEnabled | Qt.ItemFlag.ItemIsUserCheckable) - it.setCheckState(2, False) + it.setCheckState(2, Qt.CheckState.Unchecked) it.setData(2, Qt.ItemDataRole.UserRole, self.is_deletable) else: it.setFlags(Qt.ItemFlag.ItemIsEnabled) @@ -318,6 +318,8 @@ class CheckLibraryDialog(QDialog): def item_changed(self, item, column): def set_delete_boxes(node, col, to_what): + if isinstance(to_what, bool): + to_what = Qt.CheckState.Checked if to_what else Qt.CheckState.Unchecked self.log.blockSignals(True) if col: node.setCheckState(col, to_what) @@ -357,7 +359,7 @@ class CheckLibraryDialog(QDialog): set_delete_boxes(item, column, item.checkState(column)) if column == 2: self.log.blockSignals(True) - item.setCheckState(1, False) + item.setCheckState(1, Qt.CheckState.Unchecked) self.log.blockSignals(False) else: item.setCheckState(column, Qt.CheckState.Unchecked) diff --git a/src/calibre/gui2/library/views.py b/src/calibre/gui2/library/views.py index 75b4fbacbd..c37b34d78e 100644 --- a/src/calibre/gui2/library/views.py +++ b/src/calibre/gui2/library/views.py @@ -1520,6 +1520,7 @@ class DeviceBooksView(BooksView): # {{{ def reverse_sort(self): h = self.horizontalHeader() - h.setSortIndicator(h.sortIndicatorSection(), 1 - int(h.sortIndicatorOrder())) + h.setSortIndicator( + h.sortIndicatorSection(), Qt.SortOrder.AscendingOrder if h.sortIndicatorOrder() == Qt.SortOrder.DescendingOrder else Qt.SortOrder.DescendingOrder) # }}} diff --git a/src/calibre/gui2/store/search/search.py b/src/calibre/gui2/store/search/search.py index b200ba2a0c..e5ccbd0090 100644 --- a/src/calibre/gui2/store/search/search.py +++ b/src/calibre/gui2/store/search/search.py @@ -253,7 +253,7 @@ class SearchDialog(QDialog, Ui_Dialog): self.config['store_splitter_state'] = bytearray(self.store_splitter.saveState()) self.config['results_view_column_width'] = [self.results_view.columnWidth(i) for i in range(self.results_view.model().columnCount())] self.config['sort_col'] = self.results_view.model().sort_col - self.config['sort_order'] = self.results_view.model().sort_order + self.config['sort_order'] = self.results_view.model().sort_order.value self.config['open_external'] = self.open_external.isChecked() store_check = {} @@ -288,8 +288,11 @@ class SearchDialog(QDialog, Ui_Dialog): self.store_checks[n].setChecked(store_check[n]) self.results_view.model().sort_col = self.config.get('sort_col', 2) - self.results_view.model().sort_order = self.config.get('sort_order', Qt.SortOrder.AscendingOrder) - self.results_view.header().setSortIndicator(self.results_view.model().sort_col, self.results_view.model().sort_order) + so = self.config.get('sort_order', Qt.SortOrder.AscendingOrder) + if isinstance(so, int): + so = Qt.SortOrder(so) + self.results_view.model().sort_order = so + self.results_view.header().setSortIndicator(self.results_view.model().sort_col, so) def load_settings(self): # Seconds