[18.0][IMP] pre-commit updates

account_ebics/__manifest__.py

@@ -26,7 +26,6 @@
         "wizards/ebics_admin_order.xml",
         "views/menu.xml",
     ],
-    "installable": True,
     "application": True,
     "external_dependencies": {
         "python": [

account_ebics/data/ebics_file_format.xml

@@ -1,4 +1,4 @@
-<?xml version="1.0" encoding="utf-8" ?>
+<?xml version="1.0" encoding="UTF-8" ?>
 <odoo noupdate="1">
     <!-- Download formats -->
 

account_ebics/migrations/13.0.1.1/post-migration.py

@@ -3,7 +3,7 @@
 
 import os
 
-from openupgradelib import openupgrade  # pylint: disable=W7936
+from openupgradelib import openupgrade  # pylint: disable=E0401
 
 
 @openupgrade.migrate()

account_ebics/models/account_journal.py

@@ -11,13 +11,13 @@ class AccountJournal(models.Model):
         comodel_name="ebics.config",
         relation="account_journal_ebics_config_rel",
         readonly=True,
-        string="Ebics Configs",
+        string="EBICS Configs",
     )
     ebics_config_id = fields.Many2one(
         comodel_name="ebics.config",
         compute="_compute_ebics_config_id",
         compute_sudo=True,
-        string="Ebics Config",
+        string="EBICS Config",
     )
 
     @api.depends("ebics_config_ids")

account_ebics/models/ebics_config.py

@@ -8,7 +8,7 @@ import re
 from odoo import api, fields, models
 from odoo.exceptions import UserError
 
-_logger = logging.getLogger(__name__)
+_logger = logging.getLogger(__name__)  # lint-ignore=unused-logger
 
 
 class EbicsConfig(models.Model):
@@ -160,13 +160,12 @@ class EbicsConfig(models.Model):
         for rec in self:
             rec.company_ids = rec.journal_ids.mapped("company_id")
 
-    def unlink(self):
-        for ebics_config in self:
-            if ebics_config.state == "active":
-                raise UserError(
-                    self.env._("You cannot remove active EBICS configurations.")
-                )
-        return super().unlink()
+    @api.ondelete(at_uninstall=False)
+    def _unlink_except_active(self):
+        if any(rec.state == "active" for rec in self):
+            raise UserError(
+                self.env._("You cannot remove active EBICS configurations.")
+            )
 
     def set_to_draft(self):
         return self.write({"state": "draft"})
@@ -199,8 +198,8 @@ class EbicsConfig(models.Model):
         if not os.path.exists(dirname):
             raise UserError(
                 self.env._(
-                    "EBICS Keys Root Directory %s is not available."
-                    "\nPlease contact your system administrator."
+                    "EBICS Keys Root Directory %(dir)s is not available."
+                    "\nPlease contact your system administrator.",
+                    dir=dirname,
                 )
-                % dirname
             )

account_ebics/models/ebics_file.py

@@ -9,12 +9,12 @@ from traceback import format_exception
 
 from lxml import etree
 
-from odoo import fields, models
+from odoo import api, fields, models
 from odoo.exceptions import UserError
 
 from odoo.addons.base.models.res_bank import sanitize_account_number
 
-_logger = logging.getLogger(__name__)
+_logger = logging.getLogger(__name__)  # lint-ignore=unused-logger
 
 DUP_CHECK_FORMATS = ["cfonb120", "camt053"]
 
@@ -63,7 +63,6 @@ class EbicsFile(models.Model):
     )
     user_id = fields.Many2one(
         comodel_name="res.users",
-        string="User",
         default=lambda self: self.env.user,
         readonly=True,
     )
@@ -91,13 +90,16 @@ class EbicsFile(models.Model):
         help="Companies sharing this EBICS file.",
     )
 
+    @api.ondelete(at_uninstall=False)
+    def _unlink_except_done(self):
+        if any(rec.state == "done" for rec in self):
+            raise UserError(
+                self.env._("You can only remove EBICS files in state 'Draft'.")
+            )
+
     def unlink(self):
         ff_methods = self._file_format_methods()
         for ebics_file in self:
-            if ebics_file.state == "done":
-                raise UserError(
-                    self.env._("You can only remove EBICS files in state 'Draft'.")
-                )
             # execute format specific actions
             ff = ebics_file.format_id.download_process_method
             if ff in ff_methods:
@@ -385,8 +387,7 @@ class EbicsFile(models.Model):
         file_data = base64.b64decode(self.data).replace(b"\n", b"").replace(b"\r", b"")
         if len(file_data) % 120:
             message = self.env._(
-                "Incorrect CFONB120 file:\n"
-                "the file is not divisible in 120 char lines"
+                "Incorrect CFONB120 file:\nthe file is not divisible in 120 char lines"
             )
             res["notifications"].append({"type": "error", "message": message})
             return datas
@@ -650,7 +651,7 @@ class EbicsFile(models.Model):
             self.env._(
                 "The current version of the 'account_ebics' module "
                 "has no support to automatically process EBICS files "
-                "with format %s."
+                "with format %s.",
+                self.format_id.name,
             )
-            % self.format_id.name
         )

account_ebics/models/ebics_userid.py

@@ -69,8 +69,7 @@ class EbicsUserID(models.Model):
         string="Users",
         required=True,
         domain=[("share", "=", False)],
-        help="Users who are allowed to use this EBICS UserID for "
-        " bank transactions.",
+        help="Users who are allowed to use this EBICS UserID for bank transactions.",
     )
     signature_class = fields.Selection(
         selection=[("E", "Single signature"), ("T", "Transport signature")],
@@ -149,7 +148,7 @@ class EbicsUserID(models.Model):
     # create self-signed X.509 certificates
     ebics_key_x509 = fields.Boolean(
         string="X509 support",
-        help="Set this flag in order to work with " "self-signed X.509 certificates",
+        help="Set this flag in order to work with self-signed X.509 certificates",
     )
     ebics_key_x509_dn_cn = fields.Char(
         string="Common Name [CN]",
@@ -417,7 +416,7 @@ class EbicsUserID(models.Model):
         if not ebics_config_bank:
             raise UserError(
                 self.env._(
-                    "No bank defined for the financial journal " "of the EBICS Config"
+                    "No bank defined for the financial journal of the EBICS Config"
                 )
             )
         try:
@@ -476,8 +475,7 @@ class EbicsUserID(models.Model):
         if cc in ["FR", "DE"]:
             lang = cc
         else:
-            lang = self.env.user.lang or self.env["res.lang"].search([])[0].code
-            lang = lang[:2]
+            lang = self.env.user.lang[:2]
         lang = lang.lower()
         lang = lang if lang in INI_LETTER_LANGS else "en"
         fn_date = fields.Date.today().isoformat()

account_ebics/security/ebics_security.xml

@@ -1,6 +1,6 @@
-<?xml version="1.0" encoding="utf-8" ?>
+<?xml version="1.0" encoding="UTF-8" ?>
 <odoo>
-    <record model="res.groups" id="group_ebics_manager">
+    <record id="group_ebics_manager" model="res.groups">
         <field name="name">EBICS Manager</field>
         <field name="category_id" ref="base.module_category_hidden" />
     </record>

account_ebics/views/ebics_config_views.xml

@@ -1,4 +1,4 @@
-<?xml version="1.0" encoding="utf-8" ?>
+<?xml version="1.0" encoding="UTF-8" ?>
 <odoo>
     <record id="ebics_config_view_list" model="ir.ui.view">
         <field name="name">ebics.config.list</field>

account_ebics/views/ebics_file_format_views.xml

@@ -1,4 +1,4 @@
-<?xml version="1.0" encoding="utf-8" ?>
+<?xml version="1.0" encoding="UTF-8" ?>
 <odoo>
     <record id="ebics_file_format_view_search" model="ir.ui.view">
         <field name="name">ebics.file.format.search</field>

account_ebics/views/ebics_file_views.xml

@@ -1,4 +1,4 @@
-<?xml version="1.0" encoding="utf-8" ?>
+<?xml version="1.0" encoding="UTF-8" ?>
 <odoo>
     <record id="ebics_file_view_search" model="ir.ui.view">
         <field name="name">ebics.file.search</field>
@@ -81,7 +81,7 @@
     <record id="ebics_file_view_form" model="ir.ui.view">
         <field name="name">ebics.file.form</field>
         <field name="model">ebics.file</field>
-        <field name="priority">1</field>
+        <field name="priority" eval="1" />
         <field name="arch" type="xml">
             <form create="false">
                 <header>
@@ -177,7 +177,7 @@
     <record id="ebics_file_view_form_result" model="ir.ui.view">
         <field name="name">ebics.file.process.result</field>
         <field name="model">ebics.file</field>
-        <field name="priority">100</field>
+        <field name="priority" eval="100" />
         <field name="arch" type="xml">
             <form string="Process EBICS File">
                 <separator colspan="4" string="Results :" />

account_ebics/views/ebics_userid_views.xml

@@ -1,4 +1,4 @@
-<?xml version="1.0" encoding="utf-8" ?>
+<?xml version="1.0" encoding="UTF-8" ?>
 <odoo>
     <record id="ebics_userid_view_list" model="ir.ui.view">
         <field name="name">ebics.userid.list</field>

account_ebics/views/menu.xml

@@ -1,4 +1,4 @@
-<?xml version="1.0" encoding="utf-8" ?>
+<?xml version="1.0" encoding="UTF-8" ?>
 <odoo>
     <menuitem
         id="ebics_processing_menu"

account_ebics/wizards/ebics_admin_order.py

@@ -37,9 +37,9 @@ class EbicsAdminOrder(models.TransientModel):
         self.ensure_one()
         client = self._setup_client()
         if not client:
-            self.note += (
-                self.env._("EBICS client setup failed for connection '%s'")
-                % self.ebics_config_id.name
+            self.note += self.env._(
+                "EBICS client setup failed for connection '%s'",
+                self.ebics_config_id.name,
             )
         else:
             try:

account_ebics/wizards/ebics_admin_order.xml

@@ -1,10 +1,10 @@
-<?xml version="1.0" encoding="utf-8" ?>
+<?xml version="1.0" encoding="UTF-8" ?>
 <odoo>
     <record id="ebics_admin_order_view_form" model="ir.ui.view">
         <field name="name">EBICS Administrative Order</field>
         <field name="model">ebics.admin.order</field>
         <field name="inherit_id" ref="ebics_xfer_view_form_download" />
-        <field name="priority">1</field>
+        <field name="priority" eval="1" />
         <field name="arch" type="xml">
             <field name="date_from" position="attributes">
                 <attribute name="invisible">1</attribute>
@@ -31,7 +31,7 @@
     <record id="ebics_admin_order_view_form_result" model="ir.ui.view">
         <field name="name">EBICS Administrative Order result</field>
         <field name="model">ebics.admin.order</field>
-        <field name="priority">2</field>
+        <field name="priority" eval="2" />
         <field name="arch" type="xml">
             <form string="EBICS Administrative Order result">
                 <separator colspan="4" string="Results :" />

account_ebics/wizards/ebics_change_passphrase.xml

@@ -1,9 +1,9 @@
-<?xml version="1.0" encoding="utf-8" ?>
+<?xml version="1.0" encoding="UTF-8" ?>
 <odoo>
     <record id="ebics_change_passphrase_view_form" model="ir.ui.view">
         <field name="name">EBICS Keys Change Passphrase</field>
         <field name="model">ebics.change.passphrase</field>
-        <field name="priority">1</field>
+        <field name="priority" eval="1" />
         <field name="arch" type="xml">
             <form string="EBICS Keys Change Passphrase">
                 <group>
@@ -40,7 +40,7 @@
     <record id="ebics_change_passphrase_view_form_result" model="ir.ui.view">
         <field name="name">EBICS Keys Change Passphrase</field>
         <field name="model">ebics.change.passphrase</field>
-        <field name="priority">2</field>
+        <field name="priority" eval="2" />
         <field name="arch" type="xml">
             <form string="EBICS Keys Change Passphrase">
                 <separator colspan="4" string="Results :" />

account_ebics/wizards/ebics_xfer.py

@@ -224,9 +224,9 @@ class EbicsXfer(models.TransientModel):
         client = self._setup_client()
         if not client:
             err_cnt += 1
-            self.note += (
-                self.env._("EBICS client setup failed for connection '%s'")
-                % self.ebics_config_id.name
+            self.note += self.env._(
+                "EBICS client setup failed for connection '%s'",
+                self.ebics_config_id.name,
             )
         else:
             download_formats = (
@@ -336,11 +336,8 @@ class EbicsXfer(models.TransientModel):
             if ebics_files:
                 self.note += "\n"
                 for f in ebics_files:
-                    self.note += (
-                        self.env._(
-                            "EBICS File '%s' is available for further processing."
-                        )
-                        % f.name
+                    self.note += self.env._(
+                        "EBICS File '%s' is available for further processing.", f.name
                     )
                     self.note += "\n"
 
@@ -406,9 +403,8 @@ class EbicsXfer(models.TransientModel):
                     OrderID = client.upload(order_type, upload_data)
                 if OrderID:
                     self.note += "\n"
-                    self.note += (
-                        self.env._("EBICS File has been uploaded (OrderID %s).")
-                        % OrderID
+                    self.note += self.env._(
+                        "EBICS File has been uploaded (OrderID %s).", OrderID
                     )
                     ef_note = ""
                     if self.env.context.get("origin"):
@@ -619,9 +615,9 @@ class EbicsXfer(models.TransientModel):
                 self.env._(
                     "EBICS File with name '%s' has already been downloaded."
                     "\nPlease check this file and rename in case there is "
-                    "no risk on duplicate transactions."
+                    "no risk on duplicate transactions.",
+                    fn,
                 )
-                % fn
             )
         data = base64.encodebytes(data)
         ef_vals = {

account_ebics/wizards/ebics_xfer.xml

@@ -1,9 +1,9 @@
-<?xml version="1.0" encoding="utf-8" ?>
+<?xml version="1.0" encoding="UTF-8" ?>
 <odoo>
     <record id="ebics_xfer_view_form_download" model="ir.ui.view">
         <field name="name">EBICS File Download</field>
         <field name="model">ebics.xfer</field>
-        <field name="priority">1</field>
+        <field name="priority" eval="1" />
         <field name="arch" type="xml">
             <form string="EBICS File Download">
                 <group>
@@ -55,7 +55,7 @@
     <record id="ebics_xfer_view_form_upload" model="ir.ui.view">
         <field name="name">EBICS File Upload</field>
         <field name="model">ebics.xfer</field>
-        <field name="priority">1</field>
+        <field name="priority" eval="1" />
         <field name="arch" type="xml">
             <form string="EBICS File Upload">
                 <group>
@@ -120,7 +120,7 @@
     <record id="ebics_xfer_view_form_result" model="ir.ui.view">
         <field name="name">EBICS File Transfer</field>
         <field name="model">ebics.xfer</field>
-        <field name="priority">2</field>
+        <field name="priority" eval="2" />
         <field name="arch" type="xml">
             <form string="EBICS File Transfer">
                 <separator colspan="4" string="Results :" />
@@ -148,7 +148,7 @@
     <record id="ebics_xfer_view_form_mark_as_downloaded" model="ir.ui.view">
         <field name="name">Mark EBICS files as Downloaded</field>
         <field name="model">ebics.xfer</field>
-        <field name="priority">1</field>
+        <field name="priority" eval="1" />
         <field name="arch" type="xml">
             <form string="EBICS Mark as Downloaded">
                 <div class="alert alert-warning" role="alert">