From b721e215e2be059218cb171c778553d1e70a7843 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Rapha=C3=ABl=20Valyi?= Date: Fri, 18 Oct 2024 19:59:02 +0000 Subject: [PATCH] [REF] spec_driven_model: pre-commit fixes --- spec_driven_model/models/spec_mixin.py | 2 +- spec_driven_model/models/spec_models.py | 8 ++++---- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/spec_driven_model/models/spec_mixin.py b/spec_driven_model/models/spec_mixin.py index 03458402e7a1..533d4ffa5f0d 100644 --- a/spec_driven_model/models/spec_mixin.py +++ b/spec_driven_model/models/spec_mixin.py @@ -59,7 +59,7 @@ def _register_hook(self): if not hasattr(self, "_spec_module"): return res - load_key = "_%s_loaded" % (self._spec_module,) + load_key = f"_{self._spec_module}_loaded" if hasattr(self.env.registry, load_key): # already done for registry return res setattr(self.env.registry, load_key, True) diff --git a/spec_driven_model/models/spec_models.py b/spec_driven_model/models/spec_models.py index 6faae882cb6a..591783ed1847 100644 --- a/spec_driven_model/models/spec_models.py +++ b/spec_driven_model/models/spec_models.py @@ -178,7 +178,7 @@ def _setup_fields(self): def _map_concrete(cls, dbname, key, target, quiet=False): # TODO bookkeep according to a key to allow multiple injection contexts if not quiet: - _logger.debug("%s ---> %s" % (key, target)) + _logger.debug(f"{key} ---> {target}") global SPEC_MIXIN_MAPPINGS SPEC_MIXIN_MAPPINGS[dbname][key] = target @@ -188,7 +188,7 @@ def spec_module_classes(cls, spec_module): Cache the list of spec_module classes to save calls to slow reflection API. """ - spec_module_attr = "_spec_cache_%s" % (spec_module.replace(".", "_"),) + spec_module_attr = f"_spec_cache_{spec_module.replace('.', '_')}" if not hasattr(cls, spec_module_attr): setattr( cls, spec_module_attr, getmembers(sys.modules[spec_module], isclass) @@ -239,7 +239,7 @@ class StackedModel(SpecModel): def _build_model(cls, pool, cr): # inject all stacked m2o as inherited classes if cls._stacked: - _logger.info("building StackedModel %s %s" % (cls._name, cls)) + _logger.info(f"building StackedModel {cls._name} {cls}") node = cls._odoo_name_to_class(cls._stacked, cls._spec_module) env = api.Environment(cr, SUPERUSER_ID, {}) for kind, klass, _path, _field_path, _child_concrete in cls._visit_stack( @@ -318,7 +318,7 @@ def _visit_stack(cls, env, node, path=None): ): # then we will STACK the child in the current class child._stack_path = path - child_path = "%s.%s" % (path, field_path) + child_path = f"{path}.{field-path}" cls._stacking_points[name] = env[node._name]._fields.get(name) yield from cls._visit_stack(env, child, child_path) else: