From 6298b598e25cc5541691f38ad453fcd320607cff Mon Sep 17 00:00:00 2001
From: Florian da Costa <florian.dacosta@akretion.com>
Date: Fri, 3 Feb 2023 11:36:59 +0100
Subject: [PATCH] [IMP] Add back the variable feature in sql_export Refactore
 to use the new properties field

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

diff --git a/sql_export_mail/models/sql_export.py b/sql_export_mail/models/sql_export.py
index c30d2d4d2..be8d806d6 100644
--- a/sql_export_mail/models/sql_export.py
+++ b/sql_export_mail/models/sql_export.py
@@ -113,10 +113,10 @@ class SqlExport(models.Model):
             else:
                 export.send_mail()
 
-    @api.constrains("field_ids", "mail_user_ids")
+    @api.constrains("query_properties_definition", "mail_user_ids")
     def check_no_parameter_if_sent_by_mail(self):
         for export in self:
-            if export.field_ids and export.mail_user_ids:
+            if export.query_properties_definition and export.mail_user_ids:
                 raise UserError(
                     _(
                         "It is not possible to execute and send a query "