diff --git a/models/api_enedis_acc.py b/models/api_enedis_acc.py index 46a5c4e6fdc1ae5bf8538668c90520285ae8ead7..aadd61bf611f239ac2cdf46e9945d6cc82a3194b 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 fdcaacebe7d9dc762a240c3bf125e8a4b94230cc..7f48d85d94c60379700dda9b1d50f0f5689b95ee 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 5b272c110f36571cb2694db90a40cae20ff1d9e0..caa0f41ede6813b97bc09f72080be1893a8a82e3 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"))