Apply pre-commit code formatting

src/App/Dialogs.py

@@ -37,10 +37,12 @@
 class MessageDialogHTML(HTML):
     """A special version of HTML which is always published as text/html
     """
+
     def __call__(self, *args, **kw):
         class _HTMLString(str):
             """A special string that will be published as text/html
             """
+
             def asHTML(self):
                 return self
         return _HTMLString(super().__call__(*args, **kw))

src/App/config.py

@@ -55,6 +55,7 @@ class DefaultConfiguration:
     """
     This configuration should be used effectively only during unit tests
     """
+
     def __init__(self):
         from App import FindHomes
         self.clienthome = FindHomes.CLIENT_HOME

src/App/tests/test_ProductContext.py

@@ -134,6 +134,7 @@ def _verify_reg(self, name, obj):
 
 class _Product:
     """Product mockup."""
+
     def __init__(self):
         self.id = "pid"
 

src/OFS/DTMLDocument.py

@@ -48,8 +48,8 @@ class DTMLDocument(PropertyManager, DTMLMethod):
 
     # Replace change_dtml_methods by change_dtml_documents
     __ac_permissions__ = tuple([
-        (perms[0] == change_dtml_methods) and  # NOQA: W504
-        (change_dtml_documents, perms[1]) or perms
+        (perms[0] == change_dtml_methods)  # NOQA: W504
+        and (change_dtml_documents, perms[1]) or perms
         for perms in DTMLMethod.__ac_permissions__])
 
     def __call__(self, client=None, REQUEST={}, RESPONSE=None, **kw):

src/OFS/absoluteurl.py

@@ -124,6 +124,7 @@ def _isVirtualHostRoot(self):
 class RootAbsoluteURL(OFSTraversableAbsoluteURL):
     """An absolute_url adapter for the root object (OFS.Application)
     """
+
     def breadcrumbs(self):
         context = self.context
 

src/OFS/tests/testHistory.py

@@ -166,6 +166,7 @@ def test_manage_historicalComparison(self):
 class HistoryItemWithSetState(HistoryItem):
     """A class with a data migration on __setstate__
     """
+
     def __setstate__(self, state):
         super().__setstate__(state)
         self.changed_something = True

src/OFS/tests/testObjectManager.py

@@ -615,6 +615,7 @@ def test_export_import(self):
 
 class _CallResult:
     """Auxiliary class to provide defined call results."""
+
     def __init__(self, result):
         self.result = result
 

src/OFS/tests/testTraverse.py

@@ -21,6 +21,7 @@ class UnitTestSecurityPolicy:
         Stub out the existing security policy for unit testing purposes.
     """
     #   Standard SecurityPolicy interface
+
     def validate(self, accessed=None, container=None, name=None, value=None,
                  context=None, roles=None, *args, **kw):
         return 1
@@ -33,6 +34,7 @@ class CruelSecurityPolicy:
     """Denies everything
     """
     #   Standard SecurityPolicy interface
+
     def validate(self, accessed, container, name, value, *args):
         from AccessControl import Unauthorized
         raise Unauthorized(name)
@@ -44,6 +46,7 @@ def checkPermission(self, permission, object, context):
 class ProtectedMethodSecurityPolicy:
     """Check security strictly on bound methods.
     """
+
     def validate(self, accessed, container, name, value, *args):
         from AccessControl import Unauthorized
         from Acquisition import aq_base
@@ -135,6 +138,7 @@ class UnitTestUser(Implicit):
             """
                 Stubbed out manager for unit testing purposes.
             """
+
             def getId(self):
                 return 'unit_tester'
             getUserName = getId
@@ -192,10 +196,12 @@ class Restricted(SimpleItem):
             """Instance we'll check with ProtectedMethodSecurityPolicy."""
 
             getId__roles__ = None  # ACCESS_PUBLIC
+
             def getId(self):  # NOQA: E306  # pseudo decorator
                 return self.id
 
             private__roles__ = ()  # ACCESS_PRIVATE
+
             def private(self):  # NOQA: E306  # pseudo decorator
                 return 'private!'
 

src/Products/Five/browser/pagetemplatefile.py

@@ -35,6 +35,7 @@ def getEngine():
 class ViewPageTemplateFile(TrustedAppPT, PageTemplateFile):
     """Page Template used as class variable of views defined as Python classes.
     """
+
     def __init__(self, filename, _prefix=None, content_type=None):
         _prefix = self.get_path_from_prefix(_prefix)
         super().__init__(filename, _prefix)

src/Products/Five/browser/resource.py

@@ -43,6 +43,7 @@ class Resource:
     thus making this mixin (and probably the other classes in this
     module) obsolete.
     """
+
     def __call__(self):
         name = self.__name__
         container = self.__parent__

src/Products/Five/tests/testing/folder.py

@@ -23,6 +23,7 @@ class NoVerifyPasteFolder(Folder):
     """Folder that does not perform paste verification.
     Used by test_events
     """
+
     def _verifyObjectPaste(self, object, validate_src=1):
         pass
 

src/Products/Five/viewlet/tests.py

@@ -33,6 +33,7 @@ class UnitTestSecurityPolicy:
         Stub out the existing security policy for unit testing purposes.
     """
     # Standard SecurityPolicy interface
+
     def validate(self,
                  accessed=None,
                  container=None,

src/Products/PageTemplates/Expressions.py

@@ -163,6 +163,7 @@ class _CombinedMapping:
 
     Earlier mappings take precedence.
     """
+
     def __init__(self, *ms):
         self.mappings = ms
 

src/Products/PageTemplates/engine.py

@@ -86,6 +86,7 @@ def __call__(self, key, iterable):
 
 class RepeatItem(PathIterator):
     """Iterator compatible with ``chameleon`` and ``zope.tales``."""
+
     def __iter__(self):
         return self
 
@@ -241,6 +242,7 @@ def _C2ZContextWrapper(c_context, attrs):
 
 class MappedExpr:
     """map expression: ``zope.tales`` --> ``chameleon.tales``."""
+
     def __init__(self, type, expression, zt_engine):
         self.type = type
         # At this place, *expression* is a `chameleon.tokenize.Token`
@@ -315,6 +317,7 @@ def __call__(self, target, c_engine):
 
 class MappedExprType:
     """map expression type: ``zope.tales`` --> ``chameleon.tales``."""
+
     def __init__(self, engine, type):
         self.engine = engine
         self.type = type

src/Products/PageTemplates/expression.py

@@ -216,6 +216,7 @@ class ChameleonEngine(ExpressionEngine):
 
     Only partially implemented: its ``compile`` is currently unusable
     """
+
     def compile(self, expression):
         raise NotImplementedError()
 

src/Products/PageTemplates/tests/testDTMLTests.py

@@ -40,6 +40,7 @@ class UnitTestSecurityPolicy:
     #
     #   Standard SecurityPolicy interface
     #
+
     def validate(self,
                  accessed=None,
                  container=None,

src/Products/PageTemplates/tests/testHTMLTests.py

@@ -52,6 +52,7 @@ class UnitTestSecurityPolicy:
         Stub out the existing security policy for unit testing purposes.
     """
     # Standard SecurityPolicy interface
+
     def validate(self,
                  accessed=None,
                  container=None,

src/Products/PageTemplates/tests/testZopeTalesExpressions.py

@@ -9,6 +9,7 @@ class ZopeTalesExpressionTests(testHTMLTests.HTMLTests):
     The tests in this class ensure that the ``_zt_expr_registry``
     is cleared at the start and end of each test.
     """
+
     def setUp(self):
         super().setUp()
         _zt_expr_registry.clear()

src/Shared/DC/Scripts/Signature.py

@@ -32,14 +32,14 @@ def __init__(self, varnames=(), argcount=-1):
     def __eq__(self, other):
         if not isinstance(other, FuncCode):
             return False
-        return ((self.co_argcount, self.co_varnames) ==  # NOQA: W504
-                (other.co_argcount, other.co_varnames))
+        return ((self.co_argcount, self.co_varnames)  # NOQA: W504
+                == (other.co_argcount, other.co_varnames))
 
     def __lt__(self, other):
         if not isinstance(other, FuncCode):
             return False
-        return ((self.co_argcount, self.co_varnames) <  # NOQA: W504
-                (other.co_argcount, other.co_varnames))
+        return ((self.co_argcount, self.co_varnames)  # NOQA: W504
+                < (other.co_argcount, other.co_varnames))
 
 
 def _setFuncSignature(self, defaults=None, varnames=(), argcount=-1):

src/ZPublisher/BaseRequest.py

@@ -849,11 +849,11 @@ def lineno(o, m=False):
             f = i.unwrap(obj.__func__)
             c = obj.__self__.__class__
             desc = f"'{c.__module__}.{c.__qualname__}' " \
-                   f"method '{obj.__qualname__}'{lineno(f, 1)}"
+                f"method '{obj.__qualname__}'{lineno(f, 1)}"
         elif i.isfunction(obj):
             f = i.unwrap(obj)
             desc = f"function '{f.__module__}.{f.__qualname__}'" \
-                   f"{lineno(f)}"
+                f"{lineno(f)}"
         else:
             try:
                 cls_doc = "__doc__" not in obj.__dict__

src/ZPublisher/BeforeTraverse.py

@@ -88,6 +88,7 @@ class MultiHook:
     MultiHook calls the named hook from the class of the container, then
     the prior hook, then all the hooks in its list.
     """
+
     def __init__(self, hookname='<undefined hookname>', prior=None,
                  defined_in_class=False):
         # The default values are needed for unpickling instances of this class

src/ZPublisher/HTTPRequest.py

@@ -611,7 +611,7 @@ def processInputs(
                         elif has_codec(type_name):
                             # recode:
                             assert not isinstance(item, FileUpload), \
-                                   "cannot recode files"
+                                "cannot recode files"
                             item = item.encode(
                                 character_encoding, "surrogateescape")
                             character_encoding = type_name
@@ -1354,6 +1354,7 @@ def sane_environment(env):
 
 class ValueDescriptor:
     """(non data) descriptor to compute `value` from `file`."""
+
     def __get__(self, inst, owner=None):
         if inst is None:
             return self
@@ -1381,6 +1382,7 @@ def __get__(self, inst, owner=None):
 
 class Global:
     """(non data) descriptor to access a (modul) global attribute."""
+
     def __init__(self, name):
         """access global *name*."""
         self.name = name

src/ZPublisher/cookie.py

@@ -107,6 +107,7 @@ class CookieParameterRegistry:
     It maintains 2 maps: one to normalize parameter names
     and one to check and convert parameter values.
     """
+
     def __init__(self):
         self._normalize = {}
         self._convert = {}

src/ZPublisher/tests/testBaseRequest.py

@@ -141,6 +141,7 @@ def __before_publishing_traverse__(self, object, REQUEST):
     def _makeObjectWithBD(self):
         class DummyObjectWithBD(self._makeBasicObjectClass()):
             """Dummy class with __browser_default__."""
+
             def __browser_default__(self, REQUEST):
                 if REQUEST['_test_counter'] < 100:
                     REQUEST['_test_counter'] += 1
@@ -155,6 +156,7 @@ def _makeObjectWithBBT(self):
         @zpublish
         class _DummyResult:
             ''' '''
+
             def __init__(self, tag):
                 self.tag = tag
 

src/ZPublisher/tests/testHTTPRequest.py

@@ -1565,6 +1565,7 @@ class TestSearchType(unittest.TestCase):
 
     see "https://github.com/zopefoundation/Zope/pull/512"
     """
+
     def check(self, val, expect):
         mo = search_type(val)
         if expect is None:
@@ -1590,6 +1591,7 @@ def test_special(self):
 
 class _Unseekable:
     """Auxiliary class emulating an unseekable file like object"""
+
     def __init__(self, file):
         for m in ("read", "readline", "close", "__del__"):
             setattr(self, m, getattr(file, m))

src/webdav/xmltools.py

@@ -189,6 +189,7 @@ def toxml(self):
 class ProtectedExpatParser(ExpatParser):
     """ See https://bugs.launchpad.net/zope2/+bug/1114688
     """
+
     def __init__(self, forbid_dtd=True, forbid_entities=True,
                  *args, **kwargs):
         # Python 2.x old style class

src/zmi/styles/tests.py

@@ -35,6 +35,7 @@ def afterSetUp(self):
 
     def call_manage_main(self):
         """Call /folder/manage_main and return the HTML text."""
+
         def _call_manage_main(self):
             self.setRoles(['Manager'])
             # temporaryPlacelessSetUp insists in creating an interaction

util.py

@@ -1,7 +1,9 @@
 import os
-from typing import Optional, Tuple, List
 from configparser import NoSectionError
 from configparser import RawConfigParser
+from typing import List
+from typing import Optional
+from typing import Tuple
 
 
 HERE = os.path.abspath(os.path.dirname(__file__))