[REF] ir_cron: make scheduled actions inherits from server actions

The purpose of this commit is to handle code execution only in server
action and delegate schedule management to ir_cron model.

ir.cron model now inherits from ir.actions.server. Fields model, function
and args are removed as well as the logic to handle them. There is no
more code manipulation and evaluation in ir_cron, only a call to the run
method of ir.actions.server.

Cron form view use server action form view as primary view. This way
automated actions use the same base form as server actions with
cron details added.

Thanks to @fpodoo for the original idea and preliminary work. Thanks to
@jpr-odoo for first developments. Thanks to @jem-odoo for reviewing.

1 files changed, 4 insertions, 5 deletions

diff --git a/addons/event/data/event_data.xml b/addons/event/data/event_data.xml
index 3232b5514a4..b4c1070bf0d 100644
--- a/addons/event/data/event_data.xml
+++ b/addons/event/data/event_data.xml
@@ -13,16 +13,15 @@
 
         <!-- Event Mail Scheduler-->
         <record model="ir.cron" forcecreate="True" id="event_mail_scheduler">
-            <field name="name">[event] mail scheduler</field>
-            <field name="active">True</field>
+            <field name="name">Event: Mail Scheduler</field>
+            <field name="model_id" ref="model_event_mail"/>
+            <field name="state">code</field>
+            <field name="code">model.run(True)</field>
             <field name="user_id" ref="base.user_root"/>
             <field name="interval_number">1</field>
             <field name="interval_type">hours</field>
             <field name="numbercall">-1</field>
             <field name="doall" eval="False" />
-            <field eval="'(True,)'" name="args"/>
-            <field name="model">event.mail</field>
-            <field name="function">run</field>
         </record>
 
     </data>