From f152381a16b18abfd6bfaf2cdb36efdcdd9379e3 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?C=C3=A9dric=20Krier?= <ced@b2ck.com>
Date: Sat, 27 Feb 2021 18:16:29 +0100
Subject: [PATCH] Follow common rule to organize menu entries

issue10118
review347611002
---
 line.xml      | 35 +++++++++++++++++++++++++----------
 timesheet.xml | 20 ++++++++++++++++----
 work.xml      | 14 ++++++++++----
 3 files changed, 51 insertions(+), 18 deletions(-)

diff --git a/line.xml b/line.xml
index 692026d64b..afba1ab7f0 100644
--- a/line.xml
+++ b/line.xml
@@ -29,8 +29,11 @@ this repository contains the full copyright notices and license terms. -->
             <field name="view" ref="line_view_form"/>
             <field name="act_window" ref="act_line_form"/>
         </record>
-        <menuitem parent="menu_timesheet" action="act_line_form"
-            id="menu_line_form" sequence="30"/>
+        <menuitem
+            parent="menu_timesheet"
+            action="act_line_form"
+            sequence="20"
+            id="menu_line_form"/>
 
         <record model="ir.action.act_window" id="act_line_form_work">
             <field name="name">Timesheet Lines</field>
@@ -92,8 +95,11 @@ this repository contains the full copyright notices and license terms. -->
             <field name="name">Enter Timesheet</field>
             <field name="wiz_name">timesheet.line.enter</field>
         </record>
-        <menuitem parent="menu_timesheet" action="act_line_enter"
-            id="menu_line_enter" sequence="20"/>
+        <menuitem
+            parent="menu_timesheet"
+            sequence="10"
+            action="act_line_enter"
+            id="menu_line_enter"/>
 
         <record model="ir.ui.view" id="hours_employee_view_tree">
             <field name="model">timesheet.hours_employee</field>
@@ -122,8 +128,11 @@ this repository contains the full copyright notices and license terms. -->
             <field name="view" ref="hours_employee_view_graph"/>
             <field name="act_window" ref="act_hours_employee_form"/>
         </record>
-        <menuitem parent="menu_reporting" action="act_hours_employee_form"
-            id="menu_hours_employee" sequence="20"/>
+        <menuitem
+            parent="menu_reporting"
+            action="act_hours_employee_form"
+            sequence="50"
+            id="menu_hours_employee"/>
 
         <record model="ir.rule.group" id="rule_group_hours_employee">
             <field name="name">Own employee hours</field>
@@ -170,8 +179,11 @@ this repository contains the full copyright notices and license terms. -->
             <field name="view" ref="hours_employee_weekly_view_tree"/>
             <field name="act_window" ref="act_hours_employee_weekly_form"/>
         </record>
-        <menuitem parent="menu_reporting" action="act_hours_employee_weekly_form"
-            id="menu_hours_employee_open_weekly" sequence="30"/>
+        <menuitem
+            parent="menu_reporting"
+            action="act_hours_employee_weekly_form"
+            sequence="50"
+            id="menu_hours_employee_open_weekly"/>
 
         <record model="ir.ui.view" id="hours_employee_monthly_view_tree">
             <field name="model">timesheet.hours_employee_monthly</field>
@@ -188,7 +200,10 @@ this repository contains the full copyright notices and license terms. -->
             <field name="view" ref="hours_employee_monthly_view_tree"/>
             <field name="act_window" ref="act_hours_employee_monthly_form"/>
         </record>
-        <menuitem parent="menu_reporting" action="act_hours_employee_monthly_form"
-            id="menu_hours_employee_open_monthly" sequence="40"/>
+        <menuitem
+            parent="menu_reporting"
+            action="act_hours_employee_monthly_form"
+            sequence="50"
+            id="menu_hours_employee_open_monthly"/>
     </data>
 </tryton>
diff --git a/timesheet.xml b/timesheet.xml
index 6052ce1d2c..9fc2b9f481 100644
--- a/timesheet.xml
+++ b/timesheet.xml
@@ -17,14 +17,26 @@ this repository contains the full copyright notices and license terms. -->
             <field name="path">icons/tryton-timesheet.svg</field>
         </record>
 
-        <menuitem name="Timesheet" sequence="6" id="menu_timesheet" icon="tryton-timesheet"/>
-        <menuitem name="Configuration" parent="menu_timesheet" id="menu_configuration" sequence="10" icon="tryton-settings"/>
+        <menuitem
+            name="Timesheet"
+            sequence="100"
+            id="menu_timesheet"
+            icon="tryton-timesheet"/>
+        <menuitem
+            name="Configuration"
+            parent="menu_timesheet"
+            sequence="0"
+            id="menu_configuration"
+            icon="tryton-settings"/>
         <record model="ir.ui.menu-res.group"
             id="menu_configuration_group_timesheet_admin">
             <field name="menu" ref="menu_configuration"/>
             <field name="group" ref="group_timesheet_admin"/>
         </record>
-        <menuitem name="Reporting" parent="menu_timesheet"
-            id="menu_reporting" sequence="100"/>
+        <menuitem
+            name="Reporting"
+            parent="menu_timesheet"
+            sequence="100"
+            id="menu_reporting"/>
     </data>
 </tryton>
diff --git a/work.xml b/work.xml
index f664097aac..180b6f8827 100644
--- a/work.xml
+++ b/work.xml
@@ -42,8 +42,11 @@ this repository contains the full copyright notices and license terms. -->
             <field name="view" ref="work_view_form"/>
             <field name="act_window" ref="act_work_list"/>
         </record>
-        <menuitem parent="menu_configuration"
-            action="act_work_list" id="menu_work_list"/>
+        <menuitem
+            parent="menu_configuration"
+            action="act_work_list"
+            sequence="10"
+            id="menu_work_list"/>
 
         <record model="ir.action.act_window" id="act_work_report">
             <field name="name">Works</field>
@@ -62,8 +65,11 @@ this repository contains the full copyright notices and license terms. -->
             <field name="view" ref="work_view_graph"/>
             <field name="act_window" ref="act_work_report"/>
         </record>
-        <menuitem parent="menu_reporting" action="act_work_report"
-            id="menu_work_report" sequence="10"/>
+        <menuitem
+            parent="menu_reporting"
+            action="act_work_report"
+            sequence="10"
+            id="menu_work_report"/>
 
         <record model="ir.model.access" id="access_work">
             <field name="model" search="[('model', '=', 'timesheet.work')]"/>