From bc5f38bc73085e3cb1c749a9003538ff72174010 Mon Sep 17 00:00:00 2001
From: Abdurrahman Saber <a.saber@odex.sa>
Date: Sat, 1 Nov 2025 12:55:17 +0200
Subject: [PATCH] [IMP] exp_official_mission: adjust approval process

---
 odex25_hr/exp_official_mission/models/hr_official_mission.py | 4 ++++
 .../security/official_mission_security.xml                   | 4 ++--
 odex25_hr/exp_official_mission/views/hr_official_mission.xml | 5 ++++-
 3 files changed, 10 insertions(+), 3 deletions(-)

diff --git a/odex25_hr/exp_official_mission/models/hr_official_mission.py b/odex25_hr/exp_official_mission/models/hr_official_mission.py
index 817d00992..bd9ffd281 100644
--- a/odex25_hr/exp_official_mission/models/hr_official_mission.py
+++ b/odex25_hr/exp_official_mission/models/hr_official_mission.py
@@ -571,6 +571,10 @@ class HrOfficialMission(models.Model):
         if self.mission_type.approve_by == 'direct_manager':
             self.approve()
 
+    def accounting_manager_refused(self):
+        for rec in self:
+            rec.refused()
+
     # Refuse For The Direct Manager Only
     def direct_manager_refused(self):
         for rec in self:
diff --git a/odex25_hr/exp_official_mission/security/official_mission_security.xml b/odex25_hr/exp_official_mission/security/official_mission_security.xml
index 820ecc413..8f0eb3c18 100644
--- a/odex25_hr/exp_official_mission/security/official_mission_security.xml
+++ b/odex25_hr/exp_official_mission/security/official_mission_security.xml
@@ -28,7 +28,7 @@
             <field name="name">Allow HR : views missions of all employees</field>
             <field name="model_id" ref="model_hr_official_mission"/>
             <field name="domain_force">[(1 ,'=', 1)]</field>
-            <field name="groups" eval="[(3, ref('hr_base.group_account_manager')),
+            <field name="groups" eval="[(4, ref('hr_base.group_account_manager')),
                                         (4, ref('hr_base.group_executive_manager')),
                                         (4, ref('hr_base.group_general_manager')),
                                         (4, ref('exp_hr_appraisal.group_appraisal_manager')),
@@ -76,7 +76,7 @@
             <field name="name">Allow HR : views missions of all employees</field>
             <field name="model_id" ref="model_hr_official_mission_employee"/>
             <field name="domain_force">[(1 ,'=', 1)]</field>
-            <field name="groups" eval="[(3, ref('hr_base.group_account_manager')),
+            <field name="groups" eval="[(4, ref('hr_base.group_account_manager')),
                                         (4, ref('hr_base.group_executive_manager')),
                                         (4, ref('hr_base.group_general_manager')),
                                         (4, ref('hr.group_hr_manager')),
diff --git a/odex25_hr/exp_official_mission/views/hr_official_mission.xml b/odex25_hr/exp_official_mission/views/hr_official_mission.xml
index 6e5813677..bd208747a 100644
--- a/odex25_hr/exp_official_mission/views/hr_official_mission.xml
+++ b/odex25_hr/exp_official_mission/views/hr_official_mission.xml
@@ -29,6 +29,9 @@
                                 states="send" groups="hr_base.group_division_manager"/>
                         <button name="accounting_manager" string="Accounting Manager Approve" class="oe_highlight" type="object"
                                 states="accounting_manager" groups="hr_base.group_account_manager"/>
+                        <button name="accounting_manager_refused" string="Refused" class="oe_highlight" type="object"
+                                states="accounting_manager"
+                                groups="hr_base.group_account_manager"/>
                         <button name="depart_manager" string="Department Manager" class="oe_highlight" type="object"
                                 states="direct_manager" groups="hr_base.group_department_manager"/>
                         <button name="dep_manager_refused" string="Refused" class="oe_highlight" type="object"
@@ -45,7 +48,7 @@
                                 states="accounting_manager_final" groups="hr_base.group_account_manager"/>
                         <button name="refused" string="Refused" class="oe_highlight" type="object"
                                 states="accounting_manager_final"
-                                groups="hr_base.group_general_manager,hr_base.group_executive_manager,hr_base.group_account_manager"/>
+                                groups="hr_base.group_account_manager"/>
 
                         <button name="draft_state" string="RE-Draft" class="oe_highlight" type="object"
                                 states="approve,refused" groups="hr.group_hr_user"