From 2a0f5b8e3c226a9574b02f6ea967c519525a819e Mon Sep 17 00:00:00 2001
From: younes <Y.Ketfi_cherif@exp-sa.com>
Date: Mon, 15 Sep 2025 12:23:13 +0100
Subject: [PATCH] Update payroll workflow

---
 odex25_hr/exp_payroll_custom/data/data.xml           |  2 +-
 odex25_hr/exp_payroll_custom/security/security.xml   |  2 +-
 odex25_hr/exp_payroll_custom/views/payroll_menus.xml | 10 +++++++---
 odex25_hr/exp_payroll_custom/views/payslip_view.xml  | 12 ++++++------
 4 files changed, 15 insertions(+), 11 deletions(-)

diff --git a/odex25_hr/exp_payroll_custom/data/data.xml b/odex25_hr/exp_payroll_custom/data/data.xml
index 97a85fea1..a2c8e769b 100644
--- a/odex25_hr/exp_payroll_custom/data/data.xml
+++ b/odex25_hr/exp_payroll_custom/data/data.xml
@@ -1,5 +1,5 @@
 <odoo>
-    <data>
+    <data noupdate="1">
         <!--Level sequence-->
         <record id="level_sequence" model="ir.sequence">
             <field name="name">level_sequence_name</field>
diff --git a/odex25_hr/exp_payroll_custom/security/security.xml b/odex25_hr/exp_payroll_custom/security/security.xml
index 13ec259bb..077dbd3d6 100644
--- a/odex25_hr/exp_payroll_custom/security/security.xml
+++ b/odex25_hr/exp_payroll_custom/security/security.xml
@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <odoo>
-    <data>
+    <data noupdate="1">
 
         <record id="group_hr_payroll_contributor" model="res.groups">
             <field name="name">Contributor</field>
diff --git a/odex25_hr/exp_payroll_custom/views/payroll_menus.xml b/odex25_hr/exp_payroll_custom/views/payroll_menus.xml
index e6d2561bb..e47d7b0a7 100644
--- a/odex25_hr/exp_payroll_custom/views/payroll_menus.xml
+++ b/odex25_hr/exp_payroll_custom/views/payroll_menus.xml
@@ -2,9 +2,13 @@
 <odoo>
     <data>
 
-        <menuitem id="exp_hr_payroll.menu_hr_payroll_employees_root" name="Payroll"
-                  parent="hr.menu_hr_root"
-                  groups="exp_payroll_custom.group_hr_payroll_contributor,exp_hr_payroll.group_hr_payroll_user,base.group_user"/>
+        <menuitem id="exp_hr_payroll.menu_hr_payroll_root" name="Payroll"
+                  groups="exp_payroll_custom.group_hr_payroll_contributor,exp_hr_payroll.group_hr_payroll_user,exp_payroll_custom.group_payroll_finance_review,exp_payroll_custom.group_payroll_expense_manger,hr_base.group_account_manager,hr.group_hr_manager,hr_base.group_general_manager"/>
+
+        <menuitem id="exp_hr_payroll.menu_hr_payslip_run" sequence="1"
+                  parent="exp_hr_payroll.menu_hr_payroll_root"
+                  groups="hr_base.group_executive_manager,exp_payroll_custom.group_hr_payroll_contributor,exp_hr_payroll.group_hr_payroll_user,exp_payroll_custom.group_payroll_finance_review,exp_payroll_custom.group_payroll_expense_manger,hr_base.group_account_manager,hr.group_hr_manager,hr_base.group_general_manager"
+        />
 
         <record model="ir.ui.menu" id="exp_hr_payroll.menu_department_tree">
             <field name="groups_id"
diff --git a/odex25_hr/exp_payroll_custom/views/payslip_view.xml b/odex25_hr/exp_payroll_custom/views/payslip_view.xml
index 49f7fdaf9..654bb7f06 100644
--- a/odex25_hr/exp_payroll_custom/views/payslip_view.xml
+++ b/odex25_hr/exp_payroll_custom/views/payslip_view.xml
@@ -291,20 +291,20 @@
                         <field name="employee_id"/>
 <!--                        <field name="name"/>-->
                         <field name="company_id" groups="base.group_multi_company" options="{'no_create': True}"/>
-                        <field name="date_from"/>
-                        <field name="date_to"/>
+                        <field name="date_from" optional="hide"/>
+                        <field name="date_to" optional="hide"/>
                         <field name="payslip_run_id" invisible="1"/>
                         <field name="basic_allowances" sum="Basic Salary"/>
                         <field name="house_allowances" sum="House Allowance"/>
                         <field name="trans_allowances" sum="Transport Allowance"/>
                         <field name="other_allowances" sum="Others Allowance"/>
+                        <field name="total_allowances" sum="Total Allowance" decoration-bf="1"/>
                         <field name="employee_insurnce" sum="Employee Insurnce"/>
-                        <field name="total_allowances" sum="Total Allowance"/>
-                        <field name="total_deductions" sum="Total Deduction"/>
+                        <field name="total_deductions" sum="Total Deduction" decoration-bf="1" decoration-danger="1"/>
                         <field name="total_loans" sum="Total Loans"  groups="hr_loans_salary_advance.group_loan_manager"/>
                         <!--field name="percentage" string="Percentage"/-->
-                        <field name="total_sum" sum="Total Net"/>
-                        <field name="state"/>
+                        <field name="total_sum" sum="Total Net" decoration-bf="1" decoration-success="1"/>
+                        <field name="state" optional="hide"/>
                     </tree>
                 </tree>
             </field>