[IMP] chained_swapper: black, isort, prettier

Author: Ernesto Tejeda

chained_swapper/__manifest__.py

@@ -1,21 +1,19 @@
 # Copyright 2020 Tecnativa - Ernesto Tejeda
 # License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
 {
-    'name': 'Chained Swapper',
-    'summary': 'Chained Swapper',
-    'version': '12.0.1.0.1',
-    'author': 'Tecnativa, Odoo Community Association (OCA)',
-    'category': 'Tools',
-    'website': 'https://github.com/OCA/server-ux',
-    'license': 'AGPL-3',
-    'depends': ['base'],
-    'data': [
-        'security/ir.model.access.csv',
-        'views/chained_swapper_views.xml',
-        'wizard/chained_swapper_wizard_views.xml',
+    "name": "Chained Swapper",
+    "summary": "Chained Swapper",
+    "version": "12.0.1.0.1",
+    "author": "Tecnativa, Odoo Community Association (OCA)",
+    "category": "Tools",
+    "website": "https://github.com/OCA/server-ux",
+    "license": "AGPL-3",
+    "depends": ["base"],
+    "data": [
+        "security/ir.model.access.csv",
+        "views/chained_swapper_views.xml",
+        "wizard/chained_swapper_wizard_views.xml",
     ],
-    "demo": [
-        "demo/chained_swapper_demo.xml",
-    ],
-    'uninstall_hook': 'uninstall_hook',
+    "demo": ["demo/chained_swapper_demo.xml",],
+    "uninstall_hook": "uninstall_hook",
 }

chained_swapper/demo/chained_swapper_demo.xml

@@ -1,33 +1,38 @@
-<?xml version="1.0" encoding="UTF-8"?>
+<?xml version="1.0" encoding="UTF-8" ?>
 <!-- Copyright 2020 Tecnativa - Ernesto Tejeda
      Copyright 2020 Tecnativa - Pedro M. Baeza
      License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl). -->
 <odoo noupdate="1">
     <!--Add a chained swapper-->
     <record id="chained_swapper_demo" model="chained.swapper">
         <field name="name">Language</field>
-        <field name="model_id" ref="base.model_res_partner"/>
-        <field name="field_id" ref="base.field_res_partner__lang"/>
+        <field name="model_id" ref="base.model_res_partner" />
+        <field name="field_id" ref="base.field_res_partner__lang" />
     </record>
     <record id="chained_swapper_sub_field_demo" model="chained.swapper.sub.field">
-        <field name="chained_swapper_id" ref="chained_swapper_demo"/>
+        <field name="chained_swapper_id" ref="chained_swapper_demo" />
         <field name="sub_field_chain">child_ids.lang</field>
     </record>
     <record id="chained_swapper_constraint_demo" model="chained.swapper.constraint">
-        <field name="chained_swapper_id" ref="chained_swapper_demo"/>
+        <field name="chained_swapper_id" ref="chained_swapper_demo" />
         <field name="name">Only parent company</field>
         <field name="expression">bool(records.mapped('parent_id'))</field>
     </record>
     <!--Add an action-->
-    <act_window id="partner_chained_swap_lang_action_demo"
-                name="Chained swap: Language"
-                res_model="chained.swapper.wizard"
-                src_model="res.partner"
-                view_mode="form"
-                context="{'chained_swapper_id': ref('chained_swapper_demo')}"
-                key2="client_action_multi"
-                target="new"/>
+    <act_window
+        id="partner_chained_swap_lang_action_demo"
+        name="Chained swap: Language"
+        res_model="chained.swapper.wizard"
+        src_model="res.partner"
+        view_mode="form"
+        context="{'chained_swapper_id': ref('chained_swapper_demo')}"
+        key2="client_action_multi"
+        target="new"
+    />
     <record id="chained_swapper_demo" model="chained.swapper">
-        <field name="ref_ir_act_window_id" ref="partner_chained_swap_lang_action_demo"/>
+        <field
+            name="ref_ir_act_window_id"
+            ref="partner_chained_swap_lang_action_demo"
+        />
     </record>
 </odoo>

chained_swapper/hooks.py

@@ -1,14 +1,14 @@
 # Copyright 2020 Tecnativa - Ernesto Tejeda
 # License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
 
-from odoo.api import Environment, SUPERUSER_ID
+from odoo.api import SUPERUSER_ID, Environment
 
 
 def uninstall_hook(cr, registry):
     """Delete the actions that were created with chained_swapper when
     the module is uninstalled"""
     env = Environment(cr, SUPERUSER_ID, {})
-    env['ir.actions.act_window'].search([
-        ('res_model', '=', 'chained.swapper.wizard')
-    ]).unlink()
+    env["ir.actions.act_window"].search(
+        [("res_model", "=", "chained.swapper.wizard")]
+    ).unlink()
     return True

chained_swapper/models/chained_swapper.py

@@ -2,50 +2,45 @@
 # Copyright 2020 Tecnativa - Pedro M. Baeza
 # License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
 
-from odoo import api, exceptions, fields, models, _
+from odoo import _, api, exceptions, fields, models
 from odoo.tools.safe_eval import safe_eval
 
 
 class ChainedSwapper(models.Model):
     _name = "chained.swapper"
     _description = "Chained Swapper"
 
-    name = fields.Char(
-        required=True,
-        translate=True,
-        index=1,
-    )
+    name = fields.Char(required=True, translate=True, index=1,)
     model_id = fields.Many2one(
-        comodel_name='ir.model',
+        comodel_name="ir.model",
         required=True,
         help="Model is used for Selecting Field. This is editable "
-             "until Contextual Action is not created.",
+        "until Contextual Action is not created.",
     )
     allowed_field_ids = fields.Many2many(
-        comodel_name='ir.model.fields',
-        compute='_compute_allowed_field_ids'
+        comodel_name="ir.model.fields", compute="_compute_allowed_field_ids"
     )
     field_id = fields.Many2one(
-        comodel_name='ir.model.fields',
+        comodel_name="ir.model.fields",
         required=True,
         domain="[('id', 'in', allowed_field_ids)]",
     )
     sub_field_ids = fields.One2many(
-        comodel_name='chained.swapper.sub.field',
-        inverse_name='chained_swapper_id',
-        string='Sub-fields',
+        comodel_name="chained.swapper.sub.field",
+        inverse_name="chained_swapper_id",
+        string="Sub-fields",
     )
     constraint_ids = fields.One2many(
-        comodel_name='chained.swapper.constraint',
-        inverse_name='chained_swapper_id',
-        string='Constraints',
+        comodel_name="chained.swapper.constraint",
+        inverse_name="chained_swapper_id",
+        string="Constraints",
     )
     ref_ir_act_window_id = fields.Many2one(
-        comodel_name='ir.actions.act_window',
-        string='Action',
+        comodel_name="ir.actions.act_window",
+        string="Action",
         readonly=True,
         help="Action to make this template available on records "
-             "of the related document model.",
+        "of the related document model.",
     )
     group_ids = fields.Many2many(
         comodel_name="res.groups",
@@ -56,39 +51,44 @@ class ChainedSwapper(models.Model):
     )
 
     _sql_constraints = [
-        ('model_id_field_id_unique', 'unique (model_id, field_id)',
-         'Model and Field must be unique!'),
+        (
+            "model_id_field_id_unique",
+            "unique (model_id, field_id)",
+            "Model and Field must be unique!",
+        ),
     ]
 
-    @api.depends('model_id')
+    @api.depends("model_id")
     def _compute_allowed_field_ids(self):
-        model_obj = self.env['ir.model']
-        field_obj = self.env['ir.model.fields']
+        model_obj = self.env["ir.model"]
+        field_obj = self.env["ir.model.fields"]
         for record in self:
             allowed_field_ids = False
             if record.model_id:
                 all_models = record.model_id
                 active_model_obj = self.env[record.model_id.model]
                 if active_model_obj._inherits:
                     keys = list(active_model_obj._inherits.keys())
-                    all_models |= model_obj.search([('model', 'in', keys)])
-                allowed_field_ids = field_obj.search([
-                    ('ttype', 'not in', ['reference', 'function', 'one2many']),
-                    ('model_id', 'in', all_models.ids),
-                ])
+                    all_models |= model_obj.search([("model", "in", keys)])
+                allowed_field_ids = field_obj.search(
+                    [
+                        ("ttype", "not in", ["reference", "function", "one2many"]),
+                        ("model_id", "in", all_models.ids),
+                    ]
+                )
             record.allowed_field_ids = allowed_field_ids
 
-    @api.constrains('model_id', 'field_id')
+    @api.constrains("model_id", "field_id")
     def _check_sub_field_ids(self):
-        self.mapped('sub_field_ids')._check_sub_field_chain()
+        self.mapped("sub_field_ids")._check_sub_field_chain()
 
-    @api.onchange('model_id')
+    @api.onchange("model_id")
     def _onchange_model_id(self):
         self.field_id = False
 
     def write(self, vals):
         res = super().write(vals)
-        if 'name' in vals:
+        if "name" in vals:
             self.mapped("ref_ir_act_window_id").write({"name": vals["name"]})
         return res
 
@@ -100,25 +100,27 @@ def unlink(self):
     @api.multi
     def add_action(self):
         self.ensure_one()
-        action = self.env['ir.actions.act_window'].create({
-            'name': _("Chained swap") + ": " + self.name,
-            'type': 'ir.actions.act_window',
-            'res_model': 'chained.swapper.wizard',
-            'src_model': self.model_id.model,
-            'groups_id': [(4, x.id) for x in self.group_ids],
-            'view_type': 'form',
-            'context': "{'chained_swapper_id': %d}" % (self.id),
-            'view_mode': 'form',
-            'target': 'new',
-            'binding_model_id': self.model_id.id,
-            'binding_type': 'action',
-        })
-        self.write({'ref_ir_act_window_id': action.id})
+        action = self.env["ir.actions.act_window"].create(
+            {
+                "name": _("Chained swap") + ": " + self.name,
+                "type": "ir.actions.act_window",
+                "res_model": "chained.swapper.wizard",
+                "src_model": self.model_id.model,
+                "groups_id": [(4, x.id) for x in self.group_ids],
+                "view_type": "form",
+                "context": "{'chained_swapper_id': %d}" % (self.id),
+                "view_mode": "form",
+                "target": "new",
+                "binding_model_id": self.model_id.id,
+                "binding_type": "action",
+            }
+        )
+        self.write({"ref_ir_act_window_id": action.id})
         return True
 
     @api.multi
     def unlink_action(self):
-        self.mapped('ref_ir_act_window_id').unlink()
+        self.mapped("ref_ir_act_window_id").unlink()
         return True
 
 
@@ -127,59 +129,61 @@ class ChainedSwapperSubField(models.Model):
     _description = "Chained Swapper Sub-field"
 
     chained_swapper_id = fields.Many2one(
-        comodel_name='chained.swapper', ondelete="cascade"
+        comodel_name="chained.swapper", ondelete="cascade"
     )
     sub_field_chain = fields.Char(
         required=True,
         help="You can specify here a field of related fields as "
-             "dotted names. Ex.: 'child_ids.lang'."
+        "dotted names. Ex.: 'child_ids.lang'.",
     )
 
-    @api.constrains('chained_swapper_id', 'sub_field_chain')
+    @api.constrains("chained_swapper_id", "sub_field_chain")
     def _check_sub_field_chain(self):
         for rec in self:
             # Check sub-field exist
             try:
-                chain_list = rec.sub_field_chain.split('.')
+                chain_list = rec.sub_field_chain.split(".")
                 chain_field_name = chain_list.pop()
                 chain_model = self.env[rec.chained_swapper_id.model_id.model]
                 for name in chain_list:
                     chain_model = chain_model[name]
                 chain_model[chain_field_name]  # pylint: disable=W0104
             except KeyError:
                 raise exceptions.ValidationError(
-                    _("Incorrect sub-field expression:") + " " +
-                    rec.sub_field_chain)
+                    _("Incorrect sub-field expression:") + " " + rec.sub_field_chain
+                )
             # Check sub-field and original field are the same type
             swap_field = rec.chained_swapper_id.field_id
-            chain_field = self.env['ir.model.fields'].search([
-                ('model_id', '=', rec.chained_swapper_id.model_id.id),
-                ('name', '=', chain_field_name),
-            ])
-            if (chain_field.ttype != swap_field.ttype or
-                    chain_field.relation != swap_field.relation):
+            chain_field = self.env["ir.model.fields"].search(
+                [
+                    ("model_id", "=", rec.chained_swapper_id.model_id.id),
+                    ("name", "=", chain_field_name),
+                ]
+            )
+            if (
+                chain_field.ttype != swap_field.ttype
+                or chain_field.relation != swap_field.relation
+            ):
                 raise exceptions.ValidationError(
-                    _("The sub-field '%s' is not compatible with the main"
-                      " field.") % rec.sub_field_chain)
+                    _("The sub-field '%s' is not compatible with the main" " field.")
+                    % rec.sub_field_chain
+                )
 
 
 class ChainedSwapperConstraint(models.Model):
     _name = "chained.swapper.constraint"
     _description = "Chained Swapper Constraint"
 
     chained_swapper_id = fields.Many2one(
-        comodel_name='chained.swapper', ondelete="cascade"
-    )
-    name = fields.Char(
-        required=True,
-        translate=True,
+        comodel_name="chained.swapper", ondelete="cascade"
     )
+    name = fields.Char(required=True, translate=True,)
     expression = fields.Text(
-        string='Constraint expression',
+        string="Constraint expression",
         required=True,
         help="Boolean python expression. You can use the keyword "
-             "'records' as the records selected to execute the "
-             "contextual action. Ex.: bool(records.mapped('parent_id'))",
+        "'records' as the records selected to execute the "
+        "contextual action. Ex.: bool(records.mapped('parent_id'))",
         default="True",
     )
 
@@ -188,8 +192,8 @@ def _check_expression(self):
         for record in self:
             model = self.env[record.chained_swapper_id.model_id.model]
             try:
-                safe_eval(record.expression, {'records': model})
+                safe_eval(record.expression, {"records": model})
             except Exception:
-                raise exceptions.ValidationError(_(
-                    "Invalid constraint expression:" + "  " + record.expression
-                ))
+                raise exceptions.ValidationError(
+                    _("Invalid constraint expression:" + "  " + record.expression)
+                )