[UPD] perimeter to cron

a513ba25 · Julien - Le Filament · Rémi - Le Filament · b1e536ea · a513ba25 · a513ba25
--- a/data/cron_data.xml
+++ b/data/cron_data.xml
@@ -2,7 +2,7 @@
 <odoo>
    <data noupdate="1">
        <record model="ir.cron" forcecreate="True" id="ir_cron_get_perimeter">
-            <field name="name">Enedis recuperation perimetre</field>
+            <field name="name">Enedis recuperation perimetre et courbes</field>
            <field name="model_id" ref="oacc.model_acc_operation" />
            <field name="state">code</field>
            <field name="code">model._cron()</field>

--- a/data/mail_template_data.xml
+++ b/data/mail_template_data.xml
@@ -6,7 +6,7 @@
        <!--Email template -->
        <record id="email_template_new_prm" model="mail.template">
            <field name="name">Paramètres: Nouveau PRM ou pèriode</field>
-            <field name="model_id" ref="oacc.model_acc_operation" />
+            <field name="model_id" ref="oacc.model_acc_logs" />
            <field name="email_from">"Elocoop" &lt;bonjour@elo.coop></field>
            <field name="email_to">"Elocoop" &lt;bonjour@elo.coop></field>
            <field name="reply_to">"Elocoop" &lt;bonjour@elo.coop></field>
@@ -20,15 +20,14 @@
                <div style="margin: 0px; padding: 0px;">
                        Bonjour,<br />
                    Vous recevez ce mail car vous êtes administrateur de l’opération d’autoconsommation collective : <t
-                        t-out="object.name"
+                        t-out="object.acc_operation_id.name"
-                    >operation</t> (<t t-out="object.description">oacc</t>)<br
+                    >operation</t> (<t
-                    />
+                        t-out="object.acc_operation_id.description"
+                    >oacc</t>)<br />
                        D’après les données Enedis, des nouveaux PRM ont été ajoutés et/ou retirés de l’opération ou ont changé de propriétaire.<br
                    />
                        Numéros des PRM concernées :<br />
-                            <t
+                            <t t-out="object.new_prm">operation</t><br />
-                        t-out="object.perimeter_impacted_prm"
-                    >operation</t><br />
                    Si des PRM sont ajoutés, afin qu’ils soient bien pris en compte dans Elocoop, nous vous remercions de renseigner les informations des participants liés à ces PRM dans votre espace d’administration Elocoop, de cette manière :<br
                    />
@@ -54,7 +53,7 @@
        <record id="email_template_api_error" model="mail.template">
            <field name="name">Paramètres: Erreur API Enedis</field>
-            <field name="model_id" ref="oacc.model_acc_operation" />
+            <field name="model_id" ref="oacc.model_acc_logs" />
            <field name="email_from">"Elocoop" &lt;bonjour@elo.coop></field>
            <field name="email_to">"Elocoop" &lt;bonjour@elo.coop></field>
            <field name="reply_to">"Elocoop" &lt;bonjour@elo.coop></field>
@@ -68,19 +67,19 @@
                <div style="margin: 0px; padding: 0px;">
                        Bonjour,<br />
                        Vous recevez ce mail car vous êtes administrateur de l’opération d’autoconsommation collective: <t
-                        t-out="object.name"
+                        t-out="object.acc_operation_id.name"
-                    >operation</t> (<t t-out="object.description">oacc</t>)<br
+                    >operation</t> (<t
-                    />.
+                        t-out="object.acc_operation_id.description"
+                    >oacc</t>)<br />.
                        Lors de la mise à jour automatique du périmètre et des données Enedis, l’API a renvoyé l’erreur suivante :<br
                    />
-                            <t t-out="object.perimeter_api_log">operation</t><br
+                            <t t-out="object.message">operation</t><br />
-                    />
                    Nous vous remercions de vérifier que :<br />
                    <ul>
                        <li
                        >Vos identifiants Enedis sont corrects dans l’onglet Autres Informations de votre espace d’administration Elocoop</li>
                        <li>L’identifiant <t
-                                t-out="object.name"
+                                t-out="object.acc_operation_id.name"
                            >operation</t> est correct sur la page de votre opération.</li>
                        <li
                        >Nous informer en retour de ce mail lorsque cela est fait afin que nous vérifions la bonne configuration de l’API</li>

--- a/models/acc_operation.py
+++ b/models/acc_operation.py
 # Copyright 2021- Le Filament (https://le-filament.com)
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html)
-from datetime import date, datetime, timedelta
 import calendar
+from datetime import date, datetime, timedelta
 from odoo import fields, models
 from odoo.exceptions import ValidationError
@@ -106,21 +107,14 @@ class AccOperation(models.Model):
    def _cron_perimeter(self):
        """
-        get perimeter from cron and log it
+        get perimeter from cron
        """
        try:
            message = self.perimeter(from_cron=True)
        except ValidationError as exc:
            message = str(exc)
-            self.create_log(
+            self.create_log(message=message, type_log="cron")
-                message=message, type_log="cron", new_prm=self.perimeter_impacted_prm
-            )
            raise exc from exc
-        self.create_log(
-            message=message, type_log="cron", new_prm=self.perimeter_impacted_prm
-        )
        return message
    def _cron_curves(self):
@@ -153,24 +147,24 @@ class AccOperation(models.Model):
        """
        for operation in self.search([]):
            try:
-                operation.perimeter_api_log = operation._cron_perimeter()
+                operation._cron_perimeter()
+                operation._cron_curves()
            except ValidationError:
                continue
-            operation._cron_curves()
-    def send_new_prm_email(self):
+    def send_new_prm_email(self, log_id):
        """
        send email for new prm
        """
        template_id = self.env.ref("oacc_perimeter_api.email_template_new_prm")
-        template_id.send_mail(self.id)
+        template_id.send_mail(log_id.id)
-    def send_api_error_mail(self):
+    def send_api_error_mail(self, log_id):
        """
        send api error email
        """
        template_id = self.env.ref("oacc_perimeter_api.email_template_api_error")
-        template_id.send_mail(self.id)
+        template_id.send_mail(log_id.id)
    # ------------------------------------------------------
    # API functions
@@ -196,9 +190,8 @@ class AccOperation(models.Model):
            perimeter_data = self._get_perimeter()
        except ValidationError as e:
            if from_cron:
-                self.perimeter_api_log = str(e)
+                log_id = self.create_log(message=str(e), type_log="cron")
-                self.send_api_error_mail()
+                self.send_api_error_mail(log_id)
-                self.create_log(message=str(e), type_log="cron")
            raise e from e
        message += "<p><strong>Appel API terminé<br/>" "Traitement des données ...<br/>"
@@ -347,11 +340,12 @@ class AccOperation(models.Model):
        message += (
            "<h1>Fin appel API Périmètre: " + str(fields.Datetime.now()) + "</h1>"
        )
-        self.perimeter_api_log = message
-        self.perimeter_impacted_prm = "<br/>".join(counter_used)
        if from_cron:
-            self.send_new_prm_email()
+            log_id = self.create_log(
+                message=message, type_log="cron", new_prm="<br/>".join(counter_used)
+            )
+            self.send_new_prm_email(log_id)
        return message