From bbda48231cdfcb0a24af4febc7162fc57a1fc3b0 Mon Sep 17 00:00:00 2001
From: dip-odoo <dip@odoo.com>
Date: Wed, 14 Mar 2018 18:59:14 +0530
Subject: [PATCH 1/3] [IMP] mass_mailing: add schedule button that opens a
 wizard for scheduling date

Purpose of this commit is to re-arrange mass mailing buttons and to add
a button for scheduling mass mailing in future :

 * add schedule date in debug mode;
 * re-arrange all button;
 * add schedule in future button and on click open wizard to schedule future
   date;
 * on 'send now' button remove the useless '|' in domain;
 * add a small wizard to select the send date;

This commit is related to task ID 46932
---
 addons/mass_mailing/__manifest__.py           |  1 +
 addons/mass_mailing/models/mass_mailing.py    |  7 +++++
 .../mass_mailing/views/mass_mailing_views.xml |  9 +++---
 addons/mass_mailing/wizard/__init__.py        |  1 +
 .../wizard/mass_mailing_schedule_date.py      | 22 +++++++++++++++
 .../mass_mailing_schedule_date_views.xml      | 28 +++++++++++++++++++
 6 files changed, 64 insertions(+), 4 deletions(-)
 create mode 100644 addons/mass_mailing/wizard/mass_mailing_schedule_date.py
 create mode 100644 addons/mass_mailing/wizard/mass_mailing_schedule_date_views.xml

diff --git a/addons/mass_mailing/__manifest__.py b/addons/mass_mailing/__manifest__.py
index 0478ca3a8c6..5f820bbf30f 100644
--- a/addons/mass_mailing/__manifest__.py
+++ b/addons/mass_mailing/__manifest__.py
@@ -23,6 +23,7 @@
         'wizard/mail_compose_message_views.xml',
         'wizard/mass_mailing_list_merge_views.xml',
         'wizard/test_mailing_views.xml',
+        'wizard/mass_mailing_schedule_date_views.xml',
         'views/mass_mailing_report_views.xml',
         'views/mass_mailing_stats_views.xml',
         'views/link_tracker_views.xml',
diff --git a/addons/mass_mailing/models/mass_mailing.py b/addons/mass_mailing/models/mass_mailing.py
index 83c85a7b0ea..6b921a526ef 100644
--- a/addons/mass_mailing/models/mass_mailing.py
+++ b/addons/mass_mailing/models/mass_mailing.py
@@ -637,6 +637,13 @@ class MassMailing(models.Model):
             'context': ctx,
         }
 
+    @api.multi
+    def action_schedule_date(self):
+        self.ensure_one()
+        action = self.env.ref('mass_mailing.mass_mailing_schedule_date_action').read()[0]
+        action['context'] = dict(self.env.context, default_mass_mailing_id=self.id)
+        return action
+
     @api.multi
     def put_in_queue(self):
         self.write({'sent_date': fields.Datetime.now(), 'state': 'in_queue'})
diff --git a/addons/mass_mailing/views/mass_mailing_views.xml b/addons/mass_mailing/views/mass_mailing_views.xml
index fc7ca90e12d..b0faff60be3 100644
--- a/addons/mass_mailing/views/mass_mailing_views.xml
+++ b/addons/mass_mailing/views/mass_mailing_views.xml
@@ -411,10 +411,11 @@
             <field name="arch" type="xml">
                 <form string="Mailing">
                     <header style="min-height:31px;">
-                        <button name="action_test_mailing" type="object" class="oe_highlight" string="Test Mailing"/>
-                        <button name="put_in_queue" type="object" attrs="{'invisible': ['|', ('state', 'in', ('in_queue', 'done'))]}" class="oe_highlight" string="Send to All"
+                        <button name="put_in_queue" type="object" attrs="{'invisible': [('state', 'in', ('in_queue', 'done'))]}" class="oe_highlight" string="Send Now"
                             confirm="This will send the email to all recipients. Do you still want to proceed ?"/>
-                        <button name="cancel_mass_mailing" type="object" attrs="{'invisible': [('state', '!=', 'in_queue')]}" class="oe_highlight" string="Cancel"/>
+                        <button name="action_schedule_date" type="object" attrs="{'invisible': [('state', 'in', ('in_queue', 'done'))]}" class="btn-default" string="Schedule"/>
+                        <button name="action_test_mailing" type="object" class="btn-default" string="Test"/>
+                        <button name="cancel_mass_mailing" type="object" attrs="{'invisible': [('state', '!=', 'in_queue')]}" class="btn-default" string="Cancel"/>
                         <button name="retry_failed_mail" type="object" attrs="{'invisible': ['|', ('state', '!=', 'done'), ('failed', '=', 0)]}" class="oe_highlight" string="Retry"/>
 
                         <field name="state" readonly="1" widget="statusbar"/>
@@ -532,7 +533,7 @@
                                             <field name="attachment_ids"  widget="many2many_binary" string="Attach a file" class="oe_inline"/>
                                         </div>
                                         <field name="mail_server_id" groups="base.group_no_one" options="{'no_create': True, 'no_open': True}"/>
-                                        <field name="schedule_date" />
+                                        <field name="schedule_date" groups='base.group_no_one'/>
                                         <field name="keep_archives" groups="base.group_no_one"/>
                                         <field name="create_date" readonly="1" attrs="{'invisible': [('create_date', '=', False)]}"/>
                                         <field name="sent_date" readonly="1" attrs="{'invisible': [('sent_date', '=', False)]}"/>
diff --git a/addons/mass_mailing/wizard/__init__.py b/addons/mass_mailing/wizard/__init__.py
index 41ed9cc32f3..d501dfb411a 100644
--- a/addons/mass_mailing/wizard/__init__.py
+++ b/addons/mass_mailing/wizard/__init__.py
@@ -4,3 +4,4 @@
 from . import test_mailing
 from . import mail_compose_message
 from . import mass_mailing_list_merge
+from . import mass_mailing_schedule_date
diff --git a/addons/mass_mailing/wizard/mass_mailing_schedule_date.py b/addons/mass_mailing/wizard/mass_mailing_schedule_date.py
new file mode 100644
index 00000000000..428ee9c54e5
--- /dev/null
+++ b/addons/mass_mailing/wizard/mass_mailing_schedule_date.py
@@ -0,0 +1,22 @@
+# -*- coding: utf-8 -*-
+# Part of Odoo. See LICENSE file for full copyright and licensing details.
+
+from odoo import api, fields, models, _
+from odoo.exceptions import ValidationError
+
+
+class MassMailingScheduleDate(models.TransientModel):
+    _name = 'mass.mailing.schedule.date'
+    _description = 'Schedule in the Future'
+
+    schedule_date = fields.Datetime(string='Schedule in the Future')
+    mass_mailing_id = fields.Many2one('mail.mass_mailing', required=True)
+
+    @api.constrains('schedule_date')
+    def _check_schedule_date(self):
+        for scheduler in self:
+            if scheduler.schedule_date < fields.Datetime.now():
+                raise ValidationError(_('Please select a date equal/or greater than the current date.'))
+
+    def set_schedule_date(self):
+        self.mass_mailing_id.write({'schedule_date': self.schedule_date, 'state': 'in_queue'})
diff --git a/addons/mass_mailing/wizard/mass_mailing_schedule_date_views.xml b/addons/mass_mailing/wizard/mass_mailing_schedule_date_views.xml
new file mode 100644
index 00000000000..b0e1aabe884
--- /dev/null
+++ b/addons/mass_mailing/wizard/mass_mailing_schedule_date_views.xml
@@ -0,0 +1,28 @@
+<?xml version="1.0"?>
+<odoo>
+    <record id="mass_mailing_schedule_date_view_form"  model="ir.ui.view">
+        <field name="name">mass.mailing.schedule.date.view.form</field>
+        <field name="model">mass.mailing.schedule.date</field>
+        <field name="arch" type="xml">
+            <form string="Take Future Schedule Date">
+                <group>
+                    <group>
+                        <field name="schedule_date" required="1"/>
+                    </group>
+                </group>
+                <footer>
+                    <button string="Schedule" name="set_schedule_date" type="object" class="btn-primary"/>
+                    <button string="Discard " class="btn-default" special="cancel" />
+                </footer>
+            </form>
+        </field>
+    </record>
+
+    <record id="mass_mailing_schedule_date_action" model="ir.actions.act_window">
+        <field name="name">When do you want to send your mailing?</field>
+        <field name="res_model">mass.mailing.schedule.date</field>
+        <field name="type">ir.actions.act_window</field>
+        <field name="view_mode">form</field>
+        <field name="target">new</field>
+    </record>
+</odoo>

From 419b4659ca0cb73a2e1fa5fd6144af49db2f82e6 Mon Sep 17 00:00:00 2001
From: Barad Mahendra <mba@openerp.com>
Date: Wed, 2 May 2018 18:59:04 +0530
Subject: [PATCH 2/3] [FIX] mass_mailing: fix mass mailing sent date
 computation

Currently when you click on send now button or schedule button it will enter
current date as sent_date while the mailing list is still in queue. Actual
sent date will be the date when the mail is sent.

Related task ID : 46932
---
 addons/mass_mailing/models/mass_mailing.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/addons/mass_mailing/models/mass_mailing.py b/addons/mass_mailing/models/mass_mailing.py
index 6b921a526ef..a47e2840470 100644
--- a/addons/mass_mailing/models/mass_mailing.py
+++ b/addons/mass_mailing/models/mass_mailing.py
@@ -646,7 +646,7 @@ class MassMailing(models.Model):
 
     @api.multi
     def put_in_queue(self):
-        self.write({'sent_date': fields.Datetime.now(), 'state': 'in_queue'})
+        self.write({'state': 'in_queue'})
 
     @api.multi
     def cancel_mass_mailing(self):
@@ -791,7 +791,7 @@ class MassMailing(models.Model):
             # auto-commit except in testing mode
             auto_commit = not getattr(threading.currentThread(), 'testing', False)
             composer.send_mail(auto_commit=auto_commit)
-            mailing.state = 'done'
+            mailing.write({'state': 'done', 'sent_date': fields.Datetime.now()})
         return True
 
     def convert_links(self):
@@ -825,4 +825,4 @@ class MassMailing(models.Model):
                 mass_mailing.state = 'sending'
                 mass_mailing.send_mail()
             else:
-                mass_mailing.state = 'done'
+                mass_mailing.write({'state': 'done', 'sent_date': fields.Datetime.now()})

From bd4daccb9b42518c28bece8ee8e3000011de8568 Mon Sep 17 00:00:00 2001
From: Barad Mahendra <mba@openerp.com>
Date: Fri, 25 May 2018 19:07:28 +0530
Subject: [PATCH 3/3] [IMP] mass_mailing: reset schedule_date when canceling
 mass mailing

Otherwise if you make a mistake, cancel and reschedule a mass mailing
schedule date is not correctly set.
---
 addons/mass_mailing/models/mass_mailing.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/addons/mass_mailing/models/mass_mailing.py b/addons/mass_mailing/models/mass_mailing.py
index a47e2840470..76b02e71226 100644
--- a/addons/mass_mailing/models/mass_mailing.py
+++ b/addons/mass_mailing/models/mass_mailing.py
@@ -650,7 +650,7 @@ class MassMailing(models.Model):
 
     @api.multi
     def cancel_mass_mailing(self):
-        self.write({'state': 'draft'})
+        self.write({'state': 'draft', 'schedule_date': False})
 
     @api.multi
     def retry_failed_mail(self):