diff --git a/src/calibre/gui2/dialogs/quickview.py b/src/calibre/gui2/dialogs/quickview.py index ba5efa7427..c72c65af2d 100644 --- a/src/calibre/gui2/dialogs/quickview.py +++ b/src/calibre/gui2/dialogs/quickview.py @@ -274,7 +274,7 @@ class Quickview(QDialog, Ui_Quickview): index = self.books_table.indexAt(point) item = self.books_table.item(index.row(), 0) if item is None: - return False; + return False book_id = int(item.data(Qt.UserRole)) self.context_menu = QMenu(self) self.context_menu.addAction(self.view_icon, _('View'), @@ -425,8 +425,7 @@ class Quickview(QDialog, Ui_Quickview): try: self.current_column = ( - self.view.column_map.index('authors') if - self.current_column is None and self.view.column_map[idx.column()] == 'title' + self.view.column_map.index('authors') if self.current_column is None and self.view.column_map[idx.column()] == 'title' else idx.column()) key = self.view.column_map[self.current_column] book_id = self.view.model().id(idx.row()) diff --git a/src/calibre/gui2/preferences/coloring.py b/src/calibre/gui2/preferences/coloring.py index 6656542c61..162070d051 100644 --- a/src/calibre/gui2/preferences/coloring.py +++ b/src/calibre/gui2/preferences/coloring.py @@ -210,7 +210,7 @@ class ConditionEditor(QWidget): # {{{ if dt == 'bool': from calibre.gui2.ui import get_gui if not get_gui().current_db.prefs.get('bools_are_tristate'): - dt = 'bool2' + dt = 'bool2' if dt in self.action_map: actions = self.action_map[dt] else: diff --git a/src/calibre/utils/formatter_functions.py b/src/calibre/utils/formatter_functions.py index 072c332f88..fdd969a918 100644 --- a/src/calibre/utils/formatter_functions.py +++ b/src/calibre/utils/formatter_functions.py @@ -1602,6 +1602,7 @@ class BuiltinAuthorSorts(BuiltinFormatterFunction): names = [sort_data.get(n) for n in mi.authors if n.strip()] return val_sep.join(n for n in names) + class BuiltinCheckYesNo(BuiltinFormatterFunction): name = 'check_yes_no' arg_count = 4 @@ -1633,6 +1634,7 @@ class BuiltinCheckYesNo(BuiltinFormatterFunction): return 'yes' return "" + _formatter_builtins = [ BuiltinAdd(), BuiltinAnd(), BuiltinApproximateFormats(), BuiltinAssign(), BuiltinAuthorLinks(), BuiltinAuthorSorts(), BuiltinBooksize(),