From 54247119c18a704b56e5074387d1d3bff60532de Mon Sep 17 00:00:00 2001
From: Florian da Costa <florian.dacosta@akretion.com>
Date: Mon, 17 Jun 2024 11:18:19 +0200
Subject: [PATCH] [FIX] sql_export_mail : avoid trying to send mail for a not
 valid query. Odoo would raise an error anyway

---
 sql_export_mail/models/sql_export.py | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/sql_export_mail/models/sql_export.py b/sql_export_mail/models/sql_export.py
index 488b9381d..0cdbc2ab9 100644
--- a/sql_export_mail/models/sql_export.py
+++ b/sql_export_mail/models/sql_export.py
@@ -92,7 +92,9 @@ class SqlExport(models.Model):
 
     @api.model
     def _run_all_sql_export_for_cron(self, cron_ids):
-        exports = self.search([("cron_ids", "in", cron_ids)])
+        exports = self.search(
+            [("cron_ids", "in", cron_ids), ("state", "=", "sql_valid")]
+        )
         for export in exports:
             if "%(company_id)s" in export.query and "%(user_id)s" not in export.query:
                 variable_dict = {}