[IMP] export_async_schedule: Migration to 15.0

Author: vincent-cowboy

export_async_schedule/__manifest__.py

@@ -4,7 +4,7 @@
 {
     "name": "Scheduled Asynchronous Export",
     "summary": "Generate and send exports by emails on a schedule",
-    "version": "12.0.1.0.0",
+    "version": "15.0.1.0.0",
     "author": "Camptocamp, Odoo Community Association (OCA)",
     "license": "AGPL-3",
     "website": "https://github.com/OCA/queue",

export_async_schedule/models/export_async_schedule.py

@@ -77,12 +77,15 @@ def run_schedule(self):
             record.next_execution = record._compute_next_date()
 
     def _compute_next_date(self):
+        next_execution = self.next_execution
+        if next_execution < datetime.now():
+            next_execution = datetime.now()
         args = {self.interval_unit: self.interval}
         if self.interval_unit == "months" and self.end_of_month:
             # dateutil knows how to deal with variable days of months,
             # it will put the latest possible day
             args.update({"day": 31, "hour": 23, "minute": 59, "second": 59})
-        return self.next_execution + relativedelta(**args)
+        return next_execution + relativedelta(**args)
 
     @api.onchange("end_of_month")
     def onchange_end_of_month(self):
@@ -131,8 +134,9 @@ def _prepare_export_params(self):
                 [export_field for export_field in export_fields],
             )
 
+        export_format = self.export_format == "excel" and "xlsx" or self.export_format
         return {
-            "format": self.export_format,
+            "format": export_format,
             "model": self.model_name,
             "fields": export_fields,
             "ids": False,

export_async_schedule/tests/test_export_async_schedule.py

@@ -10,7 +10,7 @@
 from odoo.addons.queue_job.tests.common import mock_with_delay
 
 
-class TestExportAsyncSchedule(common.SavepointCase):
+class TestExportAsyncSchedule(common.TransactionCase):
     @classmethod
     def setUpClass(cls):
         super().setUpClass()
@@ -51,7 +51,6 @@ def test_fields_with_labels(self):
             "email",
             "phone",
             "title/shortcut",
-            "parent_id/customer",
             "parent_id/company_id/name",
         ]
         result = self.env["export.async.schedule"]._get_fields_with_labels(
@@ -62,10 +61,6 @@ def test_fields_with_labels(self):
             {"label": "Email", "name": "email"},
             {"label": "Phone", "name": "phone"},
             {"label": "Title/Abbreviation", "name": "title/shortcut"},
-            {
-                "label": "Related Company/Is a Customer",
-                "name": "parent_id/customer",
-            },
             {
                 "label": "Related Company/Company/Company Name",
                 "name": "parent_id/company_id/name",

export_async_schedule/views/export_async_schedule_views.xml

@@ -5,7 +5,7 @@
         <field name="name">export.async.schedule.tree</field>
         <field name="model">export.async.schedule</field>
         <field name="arch" type="xml">
-            <tree string="Scheduled Exports">
+            <tree>
                 <field name="model_id" />
                 <field name="ir_export_id" />
                 <field name="interval" />
@@ -115,7 +115,6 @@
         <field name="name">Scheduled Exports</field>
         <field name="type">ir.actions.act_window</field>
         <field name="res_model">export.async.schedule</field>
-        <field name="view_type">form</field>
         <field name="help">Schedule Exports to send by email</field>
     </record>