From 5ca2bb785f2d61ed9aaa3af287d2d02d49cc2cd4 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Thibault=20Delavall=C3=A9e?= <tde@odoo.com>
Date: Thu, 12 Jan 2017 12:33:12 +0100
Subject: [PATCH] [IMP] ir_cron: improve form view usability

 * User renamed to Cron User which is the uid used to run
   the scheduled action
 * Clean 'repeat every' option to make it easier to understand
 * Priority field should not be named sequence
---
 odoo/addons/base/ir/ir_cron.py       |  2 +-
 odoo/addons/base/ir/ir_cron_view.xml | 11 +++++++----
 2 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/odoo/addons/base/ir/ir_cron.py b/odoo/addons/base/ir/ir_cron.py
index a3c48a66826..cb7a61b2b25 100644
--- a/odoo/addons/base/ir/ir_cron.py
+++ b/odoo/addons/base/ir/ir_cron.py
@@ -42,7 +42,7 @@ class ir_cron(models.Model):
         'ir.actions.server', 'Server action',
         delegate=True, ondelete='restrict', required=True)
     cron_name = fields.Char('Name', related='ir_actions_server_id.name', store=True)
-    user_id = fields.Many2one('res.users', string='User', default=lambda self: self.env.user, required=True)
+    user_id = fields.Many2one('res.users', string='Cron User', default=lambda self: self.env.user, required=True)
     active = fields.Boolean(default=True)
     interval_number = fields.Integer(default=1, help="Repeat every x.")
     interval_type = fields.Selection([('minutes', 'Minutes'),
diff --git a/odoo/addons/base/ir/ir_cron_view.xml b/odoo/addons/base/ir/ir_cron_view.xml
index 045fa688ef3..9eb8eb5dca6 100644
--- a/odoo/addons/base/ir/ir_cron_view.xml
+++ b/odoo/addons/base/ir/ir_cron_view.xml
@@ -15,11 +15,14 @@
                 <xpath expr="//group[@name='action_wrapper']" position="inside">
                     <group>
                         <field name="user_id"/>
-                        <field name="priority"/>
-                        <field name="interval_number"/>
-                        <field name="interval_type"/>
+                        <label for="interval_number" string="Execute Every"/>
+                        <div>
+                            <field name="interval_number" class="oe_inline"/>
+                            <field name="interval_type" class="oe_inline"/>
+                        </div>
                         <field name="nextcall"/>
                         <field name="numbercall"/>
+                        <field name="priority"/>
                         <field name="doall"/>
                     </group>
                 </xpath>
@@ -29,7 +32,7 @@
             <field name="model">ir.cron</field>
             <field name="arch" type="xml">
                 <tree string="Scheduled Actions" decoration-muted="(not active)">
-                    <field name="priority" string="Sequence"/>
+                    <field name="priority"/>
                     <field name="name"/>
                     <field name="nextcall"/>
                     <field name="interval_number"/>