cleanup previous PR

manual/template_lang.rst

@@ -649,7 +649,9 @@ Python Template Mode
 
 Python Template Mode (PTM) lets you write templates using native python and the `calibre API <https://manual.calibre-ebook.com/develop.html#api-documentation-for-various-parts-of-calibre>`_. The database API will be of most use; further discussion is beyond the scope of this manual. PTM templates are faster and can do more complicated operations but you must know how to write code in python using the calibre API.
 
-A PTM template begins with::
+A PTM template begins with:
+
+.. code-block:: python
 
  python:
  def evaluate(book, db, globals, arguments, **kwargs):
@@ -664,7 +666,7 @@ A PTM template begins with::
 
 You can add the above text to your template using the context menu, usually accessed with a right click. The comments are not significant and can be removed. You must use python indenting.
 
-Here is an example of a PTM template that produces a list of all the authors for a series. The list is stored in a `Column built from other columns, behaves like tags`. It shows in :guilabel:`Book details` and has the `on separate lines` checked (in :guilabel:`Preferences->Look & feel->Book details`). That option requires the list to be comma-separated. To satisfy that requirement the template converts commas in author names to semicolons then builds a comma-separated list of authors. The authors are then sorted, which is why the template uses author_sort.::
+Here is an example of a PTM template that produces a list of all the authors for a series. The list is stored in a `Column built from other columns, behaves like tags`. It shows in :guilabel:`Book details` and has the :guilabel:`on separate lines` checked (in :guilabel:`Preferences->Look & feel->Book details`). That option requires the list to be comma-separated. To satisfy that requirement the template converts commas in author names to semicolons then builds a comma-separated list of authors. The authors are then sorted, which is why the template uses author_sort.::
 
     python:
     def evaluate(book, db, globals, arguments, **kwargs):
@@ -672,7 +674,7 @@ Here is an example of a PTM template that produces a list of all the authors for
             return ''
         ans = set()
         for id_ in db.search_getting_ids(f'series:"={book.series}"', ''):
-            ans.update([v.strip() for v in db.new_api.field_for('author_sort', id_).split('&')])
+            ans.update(v.strip() for v in db.new_api.field_for('author_sort', id_).split('&'))
         return ', '.join(v.replace(',', ';') for v in sorted(ans))
 
 The output in :guilabel:`Book details` looks like this:

src/calibre/gui2/actions/booklist_context_menu.py

@@ -1,7 +1,6 @@
 #!/usr/bin/env python
 # License: GPLv3 Copyright: 2022, Charles Haley
 #
-from qt.core import QPoint
 
 from calibre.gui2.actions import InterfaceAction
 

src/calibre/gui2/dialogs/template_dialog.py

@@ -776,7 +776,7 @@ class TemplateDialog(QDialog, Ui_TemplateDialog):
     def textbox_changed(self):
         cur_text = str(self.textbox.toPlainText())
         if cur_text.startswith('python:'):
-            if self.highlighting_gpm == True:
+            if self.highlighting_gpm is True:
                 self.highlighter.initialize_rules(self.builtins, True)
                 self.highlighting_gpm = False
                 self.break_box.setChecked(False)

src/calibre/gui2/preferences/template_functions.py

@@ -315,7 +315,7 @@ class ConfigWidget(ConfigWidgetBase, Ui_Form):
                 return True
         try:
             prog = str(self.program.toPlainText())
-            cls = compile_user_function(name, str(self.documentation.toPlainText()),
+            compile_user_function(name, str(self.documentation.toPlainText()),
                                         self.argument_count.value(), prog)
         except:
             error_dialog(self.gui, _('Template functions'),

src/calibre/utils/formatter.py

@@ -1550,7 +1550,6 @@ class TemplateFormatter(string.Formatter):
             if compiled_text is None:
                 compiled_text = self.compile_python_template(func.program_text[len('python:'):])
                 func.cached_compiled_text = compiled_text
-            print(args)
             return self._run_python_template(compiled_text, args)
 
     def _eval_python_template(self, template, column_name):
@@ -1588,10 +1587,10 @@ class TemplateFormatter(string.Formatter):
             func = locals_['evaluate']
             return func
         except SyntaxError as e:
-            raise(ValueError(
-                _('Syntax error on line {0} column {1}: text {2}').format(e.lineno, e.offset, e.text)))
+            raise ValueError(
+                _('Syntax error on line {0} column {1}: text {2}').format(e.lineno, e.offset, e.text))
         except KeyError:
-            raise(ValueError(_("The {0} function is not defined in the template").format('evaluate')))
+            raise ValueError(_("The {0} function is not defined in the template").format('evaluate'))
 
     # ################# Override parent classes methods #####################
 

src/calibre/utils/formatter_functions.py

@@ -14,7 +14,7 @@ __docformat__ = 'restructuredtext en'
 import inspect, re, traceback, numbers
 from contextlib import suppress
 from datetime import datetime, timedelta
-from enum import Enum
+from enum import Enum, auto
 from functools import partial
 from math import trunc, floor, ceil, modf
 
@@ -30,9 +30,9 @@ from polyglot.builtins import iteritems, itervalues
 
 
 class StoredObjectType(Enum):
-    PythonFunction = 1
-    StoredGPMTemplate = 2
-    StoredPythonTemplate = 3
+    PythonFunction = auto()
+    StoredGPMTemplate = auto()
+    StoredPythonTemplate = auto()
 
 
 class FormatterFunctions: