From e332be07b6323f7e5ea2c3c5b30ac4e6c4838d90 Mon Sep 17 00:00:00 2001
From: Julien Ortet <julien@le-filament.com>
Date: Wed, 21 Feb 2024 17:20:58 +0100
Subject: [PATCH] Ruff
---
models/api_enedis_acc.py | 2 +-
tools/key_file.py | 14 +++++++++-----
wizard/acc_repartition_keys_wizard.py | 13 +++++++------
3 files changed, 17 insertions(+), 12 deletions(-)
diff --git a/models/api_enedis_acc.py b/models/api_enedis_acc.py
index 46a5c4e..aadd61b 100644
--- a/models/api_enedis_acc.py
+++ b/models/api_enedis_acc.py
@@ -1,6 +1,6 @@
# Copyright 2021- Le Filament (https://le-filament.com)
# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html)
-from odoo import _, api, models
+from odoo import api, models
class ApiEnedisAcc(models.AbstractModel):
diff --git a/tools/key_file.py b/tools/key_file.py
index fdcaace..7f48d85 100644
--- a/tools/key_file.py
+++ b/tools/key_file.py
@@ -14,12 +14,15 @@ class RepartitionKeyEntryFile:
def check(self):
"""
- Lorsque le fichier est validé, des vérifications sont effectuées, et une erreur est affichée si le test ne passe pas :
- • Première ligne, colonnes 2 à la fin : les numéros de PRM sont exactement ceux qui sont dans l’onglet Point de soutirage.
+ Lorsque le fichier est validé, des vérifications sont effectuées, et une erreur
+ est affichée si le test ne passe pas :
+ • Première ligne, colonnes 2 à la fin : les numéros de PRM sont
+ exactement ceux qui sont dans l’onglet Point de soutirage.
Aucun numéro de PRM n'apparaît plusieurs fois.
- ◦ Erreur envoyée : “Les numéros de PRM ne correspondent pas à ceux de l’opération.”
+ ◦ Erreur envoyée : “Les numéros de PRM ne correspondent pas à ceux
+ de l’opération.”
- • Lignes 2 à la fin, Horodate : Les dates sont celles d’un seul mois complet.
+ • Lignes 2 à la fin, Horodate : Les dates sont celles d’un seul mois complet
◦ Erreur envoyée “Les dates doivent être celles d’un seul mois complet.”
• La somme sur chaque ligne doit être inférieure à 100.
@@ -109,7 +112,8 @@ class RepartitionKeyEntryFile:
if missing_in_file or missing_in_operation:
return {
"check": False,
- "message": "Les numéros de PRM ne correspondent pas à ceux de l’opération",
+ "message": "Les numéros de PRM ne correspondent "
+ "pas à ceux de l’opération",
}
return {"check": True, "message": ""}
diff --git a/wizard/acc_repartition_keys_wizard.py b/wizard/acc_repartition_keys_wizard.py
index 5b272c1..caa0f41 100644
--- a/wizard/acc_repartition_keys_wizard.py
+++ b/wizard/acc_repartition_keys_wizard.py
@@ -3,7 +3,7 @@
import base64
import datetime
-from odoo import api, fields, models
+from odoo import fields, models
from odoo.exceptions import UserError
from ..tools.key_file import RepartitionKeyEntryFile
@@ -51,7 +51,8 @@ class AccRepartitionKeysWizard(models.TransientModel):
# ------------------------------------------------------
def send_imported_file(self):
"""
- testing a file, check taht all prm id exist, are in operation and all operation prm are in file
+ testing a file, check that all prm id exist, are in operation and all
+ operation prm are in file
:return:
"""
if self.csv_file:
@@ -62,8 +63,8 @@ class AccRepartitionKeysWizard(models.TransientModel):
.replace("\r", "")
.split("\n")
)
- except UnicodeDecodeError:
- raise UserError("Fichier de répartitions au mauvais format")
+ except UnicodeDecodeError as exc:
+ raise UserError("Fichier de répartitions au mauvais format") from exc
counter_list_from_operation = [
counter.name
@@ -76,8 +77,8 @@ class AccRepartitionKeysWizard(models.TransientModel):
data=file, operation_counter_list=counter_list_from_operation
)
file_check_result = entry_file_handler.check()
- except (ValueError, IndexError):
- raise UserError("Fichier de répartitions au mauvais format")
+ except (ValueError, IndexError) as exc:
+ raise UserError("Fichier de répartitions au mauvais format") from exc
if not file_check_result.get("check"):
raise UserError(file_check_result.get("message"))
--
GitLab