From 2d8147ec4d68ba9fbc277fc17722e6f9f82d7465 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?R=C3=A9mi=20-=20Le=20Filament?= <remi@le-filament.com>
Date: Thu, 2 May 2024 21:14:57 +0200
Subject: [PATCH] [FIX] pre-commit

---
 wizard/ebics_statement_rop_import_wizard.py  | 59 ++++++++++----------
 wizard/ebics_statement_rop_import_wizard.xml |  1 -
 2 files changed, 28 insertions(+), 32 deletions(-)

diff --git a/wizard/ebics_statement_rop_import_wizard.py b/wizard/ebics_statement_rop_import_wizard.py
index 6f183b7..4350ac0 100644
--- a/wizard/ebics_statement_rop_import_wizard.py
+++ b/wizard/ebics_statement_rop_import_wizard.py
@@ -12,7 +12,7 @@ class EbicsStatementRopImport(models.TransientModel):
 
     date_from = fields.Date()
     date_to = fields.Date()
-    log = fields.Html("Log")
+    log = fields.Html()
 
     def load_ebics(self):
         self.log = ""
@@ -33,11 +33,11 @@ class EbicsStatementRopImport(models.TransientModel):
         )
         ebics_conf_id = ebics_mvt_obj.ebics_config_id
         ebics_wizard_act = ebics_mvt_obj.ebics_download()
-        self.log += """
+        self.log += f"""
             <strong><u>Téléchargement des mouvements enrichis<u></strong><br/>
-            %s
+            {ebics_mvt_obj.note}
             <hr/>
-        """ % (ebics_mvt_obj.note,)
+        """
         ebics_mvt_file_id = ebics_wizard_act.get("context").get("ebics_file_ids")
         ebics_mvt_file = self.env["ebics.file"].search(
             [("id", "in", ebics_mvt_file_id)]
@@ -54,11 +54,11 @@ class EbicsStatementRopImport(models.TransientModel):
             }
         )
         ebics_rop_wizard_act = ebics_rop_obj.ebics_download()
-        self.log += """
+        self.log += f"""
             <strong><u>Téléchargement des mouvements ROP<u></strong><br/>
-            %s
+            {ebics_rop_obj.note}
             <hr/>
-        """ % (ebics_rop_obj.note,)
+        """
         ebics_rop_file_id = ebics_rop_wizard_act.get("context").get("ebics_file_ids")
         ebics_rop_file = self.env["ebics.file"].search(
             [("id", "in", ebics_rop_file_id)]
@@ -78,46 +78,43 @@ class EbicsStatementRopImport(models.TransientModel):
                 }
             )
             import_mvt_act = import_mvt.with_context(
-                {"journal_id": journal_id.id}
+                journal_id=journal_id.id
             ).import_file()
             bank_statement_id = import_mvt_act.get("context").get("statement_ids")
             bank_statement = self.env["account.bank.statement"].browse(
                 bank_statement_id
             )
             bank_statement.ensure_one()
-            self.log += """
+            self.log += f"""
                 <strong><u>Import des mouvements enrichis<u></strong><br/>
-                Référence : %s - Journal : %s - Date : %s <br/>
-                Solde Initial : %s €<br/>
-                Solde Final : %s €
+                Référence : {bank_statement.name}
+                 - Journal : {bank_statement.journal_id.name}
+                 - Date : {bank_statement.date} <br/>
+                Solde Initial : {bank_statement.balance_start} €<br/>
+                Solde Final : {bank_statement.balance_end} €
                 <hr/>
-            """ % (
-                bank_statement.name,
-                bank_statement.journal_id.name,
-                bank_statement.date,
-                bank_statement.balance_start,
-                bank_statement.balance_end,
-            )
+            """
 
         else:
             self.log += """
                 <strong><u>Import des mouvements enrichis<u></strong><br/>
-                Aucune donnée n'a pu être téléchargée, il n'y a donc pas de données à importer.
+                Aucune donnée n'a pu être téléchargée,
+                 il n'y a donc pas de données à importer.
                 <hr/>
             """
 
         if ebics_rop_file and ebics_mvt_file:
             # Suppression de lignes CORPORATE
             line_to_delete = bank_statement.line_ids.filtered(
-                lambda l: "CORPORATE CARD" in l.name
+                lambda line: "CORPORATE CARD" in line.name
             )
             line_nb = len(line_to_delete)
             line_to_delete.unlink()
-            self.log += """
+            self.log += f"""
                 <strong><u>Suppression de lignes CORPORATE<u></strong><br/>
-                Suppression de %s lignes.
+                Suppression de {line_nb} lignes.
                 <hr/>
-            """ % (line_nb,)
+            """
 
             # Ajout des lignes ROP
             rop_lines = self._get_rop_lines(ebics_rop_file.data)
@@ -125,15 +122,17 @@ class EbicsStatementRopImport(models.TransientModel):
                 ropl.update({"statement_id": bank_statement.id})
                 self.env["account.bank.statement.line"].create(ropl)
 
-            self.log += """
+            self.log += f"""
                         <strong><u>Création des lignes ROP<u></strong><br/>
-                        Création de %s lignes.
+                        Création de {len(rop_lines)} lignes.
                         <hr/>
-                    """ % (len(rop_lines),)
+                    """
         else:
             self.log += """
-                <strong><u>Suppression de lignes CORPORATE & Intégration du fichier ROP<u></strong><br/>
-                Il n'y a pas de fichier ROP ou Mouvement disponible donc les lignes <i>CORPORATE</i>
+                <strong><u>Suppression de lignes CORPORATE &
+                 Intégration du fichier ROP<u></strong><br/>
+                Il n'y a pas de fichier ROP ou Mouvement disponible
+                 donc les lignes <i>CORPORATE</i>
                 ne peuvent être supprimées et le fichier ROP ne peut être intégré.
                 <hr/>
             """
@@ -144,7 +143,6 @@ class EbicsStatementRopImport(models.TransientModel):
         return {
             "name": "Résultat du chargement automatique",
             "res_id": self.id,
-            "view_type": "form",
             "view_mode": "form",
             "res_model": "ebics.statement.import.wizard",
             "view_id": result_view.id,
@@ -157,7 +155,6 @@ class EbicsStatementRopImport(models.TransientModel):
         statement_id = self.env.context.get("statement_id")
         return {
             "res_id": statement_id,
-            "view_type": "form",
             "view_mode": "form",
             "res_model": "account.bank.statement",
             "type": "ir.actions.act_window",
diff --git a/wizard/ebics_statement_rop_import_wizard.xml b/wizard/ebics_statement_rop_import_wizard.xml
index 9b50517..de86585 100644
--- a/wizard/ebics_statement_rop_import_wizard.xml
+++ b/wizard/ebics_statement_rop_import_wizard.xml
@@ -70,7 +70,6 @@
     <field name="name">Chargement automatique des fichiers</field>
     <field name="type">ir.actions.act_window</field>
     <field name="res_model">ebics.statement.import.wizard</field>
-    <field name="view_type">form</field>
     <field name="view_mode">form</field>
     <field name="target">new</field>
     <field name="view_id" ref="ebics_statement_import_wizard_form_load" />
-- 
GitLab