From 647d954584689e63701b9511dfba2c018f78d4aa Mon Sep 17 00:00:00 2001
From: hkapatel <helly.kapatel@initos.com>
Date: Tue, 15 Jun 2021 17:07:19 +0530
Subject: [PATCH] [MIG] sql_export_mail: Migration to 14.0

---
 sql_export_mail/__manifest__.py           |  7 ++-----
 sql_export_mail/i18n/fr.po                | 12 ------------
 sql_export_mail/models/sql_export.py      | 10 +---------
 sql_export_mail/readme/CONTRIBUTORS.rst   |  1 +
 sql_export_mail/views/sql_export_view.xml |  2 +-
 5 files changed, 5 insertions(+), 27 deletions(-)

diff --git a/sql_export_mail/__manifest__.py b/sql_export_mail/__manifest__.py
index 3edc802aa..278d42817 100644
--- a/sql_export_mail/__manifest__.py
+++ b/sql_export_mail/__manifest__.py
@@ -1,15 +1,12 @@
 # License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl.html).
 {
     "name": "SQL Export Mail",
-    "version": "12.0.1.0.0",
+    "version": "14.0.1.0.0",
     "category": "Generic Modules",
     "summary": "Send csv file generated by sql query by mail.",
     "author": "Akretion, Odoo Community Association (OCA)",
     "website": "https://github.com/OCA/server-tools",
-    "depends": [
-        "sql_export",
-        "mail",
-    ],
+    "depends": ["sql_export", "mail"],
     "license": "AGPL-3",
     "data": [
         "views/sql_export_view.xml",
diff --git a/sql_export_mail/i18n/fr.po b/sql_export_mail/i18n/fr.po
index b308c401e..a9c3a6941 100644
--- a/sql_export_mail/i18n/fr.po
+++ b/sql_export_mail/i18n/fr.po
@@ -19,17 +19,6 @@ msgstr ""
 #. module: sql_export_mail
 #: model:mail.template,body_html:sql_export_mail.sql_export_mailer
 #, fuzzy
-#| msgid ""
-#| "\n"
-#| "<div style=\"font-family: 'Lucida Grande', Ubuntu, Arial, Verdana, sans-"
-#| "serif; font-size: 12px; color: rgb(34, 34, 34); background-color: #FFF; "
-#| "\">\n"
-#| "\n"
-#| "<p>You will find the report ${object.name or ''} as an attachment of the "
-#| "mail.</p>\n"
-#| "\n"
-#| "</div>\n"
-#| "    "
 msgid ""
 "\n"
 "<div style=\"font-family: 'Lucida Grande', Ubuntu, Arial, Verdana, sans-"
@@ -94,7 +83,6 @@ msgstr ""
 #. module: sql_export_mail
 #: model:ir.model.fields,field_description:sql_export_mail.field_sql_export__mail_condition
 #, fuzzy
-#| msgid "Mail condition"
 msgid "Mail Condition"
 msgstr "Condition d'envoi par mail"
 
diff --git a/sql_export_mail/models/sql_export.py b/sql_export_mail/models/sql_export.py
index f2a4aae9a..d3529c638 100644
--- a/sql_export_mail/models/sql_export.py
+++ b/sql_export_mail/models/sql_export.py
@@ -5,7 +5,6 @@ from datetime import datetime, timedelta
 
 from odoo import SUPERUSER_ID, _, api, fields, models
 from odoo.exceptions import UserError
-from odoo.tools import DEFAULT_SERVER_DATETIME_FORMAT
 
 
 class SqlExport(models.Model):
@@ -28,7 +27,6 @@ class SqlExport(models.Model):
         [("not_empty", "File Not Empty")], default="not_empty"
     )
 
-    @api.multi
     def create_cron(self):
         self.ensure_one()
         nextcall = datetime.now() + timedelta(hours=2)
@@ -51,11 +49,9 @@ class SqlExport(models.Model):
         cron.write(write_vals)
         self.write({"cron_ids": [(4, cron.id)]})
 
-    @api.multi
     def send_mail(self, params=None):
         self.ensure_one()
         mail_template = self.env.ref("sql_export_mail.sql_export_mailer")
-        now_time = datetime.strftime(datetime.now(), DEFAULT_SERVER_DATETIME_FORMAT)
         attach_obj = self.env["ir.attachment"]
         if self.mail_condition == "not_empty":
             res = self._execute_sql_request(params=params, mode="fetchone")
@@ -78,8 +74,7 @@ class SqlExport(models.Model):
         msg_id = mail_template.send_mail(self.id, force_send=False)
         mail = self.env["mail.mail"].browse(msg_id)
         attach_vals = {
-            "name": now_time + " - " + self.name,
-            "datas_fname": filename,
+            "name": filename,
             "datas": binary,
             "res_model": "mail.mail",
             "res_id": mail.id,
@@ -115,7 +110,6 @@ class SqlExport(models.Model):
             else:
                 export.send_mail()
 
-    @api.multi
     @api.constrains("field_ids", "mail_user_ids")
     def check_no_parameter_if_sent_by_mail(self):
         for export in self:
@@ -127,7 +121,6 @@ class SqlExport(models.Model):
                     )
                 )
 
-    @api.multi
     @api.constrains("mail_user_ids")
     def check_mail_user(self):
         for export in self:
@@ -135,7 +128,6 @@ class SqlExport(models.Model):
                 if not user.email:
                     raise UserError(_("The user does not have any e-mail address."))
 
-    @api.multi
     def get_email_address_for_template(self):
         """
         Called from mail template
diff --git a/sql_export_mail/readme/CONTRIBUTORS.rst b/sql_export_mail/readme/CONTRIBUTORS.rst
index 0bddb053a..c2183a83d 100644
--- a/sql_export_mail/readme/CONTRIBUTORS.rst
+++ b/sql_export_mail/readme/CONTRIBUTORS.rst
@@ -1 +1,2 @@
 * Florian da Costa <florian.dacosta@akretion.com>
+* Helly kapatel <helly.kapatel@initos.com>
diff --git a/sql_export_mail/views/sql_export_view.xml b/sql_export_mail/views/sql_export_view.xml
index 19b5402cf..7645c5c44 100644
--- a/sql_export_mail/views/sql_export_view.xml
+++ b/sql_export_mail/views/sql_export_view.xml
@@ -30,7 +30,7 @@
                         <field
                             name="cron_ids"
                             nolabel="1"
-                            domain="[('function', '=', '_run_all_sql_export_for_cron')]"
+                            domain="[('model_id', '=', 'sql.export')]"
                             groups="base.group_system"
                         />
                     </group>