From cc6c1019a3defece005049b938bd6a90fed7d21e Mon Sep 17 00:00:00 2001
From: Florian da Costa <florian.dacosta@akretion.com>
Date: Thu, 23 Nov 2017 23:39:16 +0100
Subject: [PATCH] styles and decorator fixes

---
 sql_export_mail/__openerp__.py       | 21 ++-----------
 sql_export_mail/i18n/fr.po           |  2 +-
 sql_export_mail/models/sql_export.py | 46 ++++++++++------------------
 3 files changed, 19 insertions(+), 50 deletions(-)

diff --git a/sql_export_mail/__openerp__.py b/sql_export_mail/__openerp__.py
index 58b11ef5a..992b0814b 100644
--- a/sql_export_mail/__openerp__.py
+++ b/sql_export_mail/__openerp__.py
@@ -1,23 +1,6 @@
 # -*- coding: utf-8 -*-
-##############################################################################
-#
-#    OpenERP, Open Source Management Solution
-#    Copyright (C) 2015 Akretion (<http://www.akretion.com>).
-#
-#    This program is free software: you can redistribute it and/or modify
-#    it under the terms of the GNU Affero General Public License as
-#    published by the Free Software Foundation, either version 3 of the
-#    License, or (at your option) any later version.
-#
-#    This program is distributed in the hope that it will be useful,
-#    but WITHOUT ANY WARRANTY; without even the implied warranty of
-#    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-#    GNU Affero General Public License for more details.
-#
-#    You should have received a copy of the GNU Affero General Public License
-#    along with this program.  If not, see <http://www.gnu.org/licenses/>.
-#
-##############################################################################
+# Copyright 2017 Akretion
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl.html).
 
 {
     'name': 'SQL Export Mail',
diff --git a/sql_export_mail/i18n/fr.po b/sql_export_mail/i18n/fr.po
index 9bbd26c72..483f13599 100644
--- a/sql_export_mail/i18n/fr.po
+++ b/sql_export_mail/i18n/fr.po
@@ -62,7 +62,7 @@ msgstr "Fichier non vide."
 #. module: sql_export_mail
 #: code:addons/sql_export_mail/models/sql_export.py:126
 #, python-format
-msgid "It is not possible to execute and send a query automatically by e-mail if there are parameters to fill"
+msgid "It is not possible to execute and send a query automatically by mail if there are parameters to fill"
 msgstr "Il n'est pas possible d'exécuter en envoyer le résultat d'une requête par mail si celle-ci contient des paramètres."
 
 #. module: sql_export_mail
diff --git a/sql_export_mail/models/sql_export.py b/sql_export_mail/models/sql_export.py
index ca84a47ab..6ee4d04bf 100644
--- a/sql_export_mail/models/sql_export.py
+++ b/sql_export_mail/models/sql_export.py
@@ -1,23 +1,6 @@
 # -*- coding: utf-8 -*-
-##############################################################################
-#
-#    OpenERP, Open Source Management Solution
-#    Copyright (C) 2017 Akretion (<http://www.akretion.com>).
-#
-#    This program is free software: you can redistribute it and/or modify
-#    it under the terms of the GNU Affero General Public License as
-#    published by the Free Software Foundation, either version 3 of the
-#    License, or (at your option) any later version.
-#
-#    This program is distributed in the hope that it will be useful,
-#    but WITHOUT ANY WARRANTY; without even the implied warranty of
-#    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-#    GNU Affero General Public License for more details.
-#
-#    You should have received a copy of the GNU Affero General Public License
-#    along with this program.  If not, see <http://www.gnu.org/licenses/>.
-#
-##############################################################################
+# Copyright 2017 Akretion
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl.html).
 
 from openerp import models, fields, api, _
 from openerp.tools import DEFAULT_SERVER_DATETIME_FORMAT
@@ -69,8 +52,9 @@ class SqlExport(models.Model):
 
         self.write({'cron_ids': [(4, cron.id)]})
 
-    @api.one
+    @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)
@@ -119,21 +103,23 @@ class SqlExport(models.Model):
             else:
                 export.send_mail()
 
-    @api.one
+    @api.multi
     @api.constrains('field_ids', 'mail_user_ids')
     def check_no_parameter_if_sent_by_mail(self):
-        if self.field_ids and self.mail_user_ids:
-            raise UserError(_(
-                "It is not possible to execute and send a query automatically"
-                " by e-mail if there are parameters to fill"))
+        for export in self:
+            if export.field_ids and export.mail_user_ids:
+                raise UserError(_(
+                    "It is not possible to execute and send a query "
+                    "automatically by mail if there are parameters to fill"))
 
-    @api.one
+    @api.multi
     @api.constrains('mail_user_ids')
     def check_mail_user(self):
-        for user in self.mail_user_ids:
-            if not user.email:
-                raise UserError(_(
-                    "The user does not have any e-mail address."))
+        for export in self:
+            for user in export.mail_user_ids:
+                if not user.email:
+                    raise UserError(_(
+                        "The user does not have any e-mail address."))
 
     @api.multi
     def get_email_address_for_template(self):