From eefdb87ad6a4540f7b489e4f423585cee6a765c0 Mon Sep 17 00:00:00 2001
From: Mohamed Eltayar <152964073+maltayyar2@users.noreply.github.com>
Date: Thu, 20 Nov 2025 13:04:38 +0300
Subject: [PATCH] Revert "Fix Purchase Order and Request Layout Issues"

---
 .../i18n/ar_001.po                            | 41 +--------
 .../purchase_requisition_custom/i18n/ar_SY.po |  2 +-
 .../views/purchase_request.xml                | 61 +++++++------
 .../views/purchase_requisition_custom.xml     | 85 ++++++++-----------
 4 files changed, 75 insertions(+), 114 deletions(-)

diff --git a/odex25_purchase/purchase_requisition_custom/i18n/ar_001.po b/odex25_purchase/purchase_requisition_custom/i18n/ar_001.po
index d5d0e23a2..f0db7176b 100644
--- a/odex25_purchase/purchase_requisition_custom/i18n/ar_001.po
+++ b/odex25_purchase/purchase_requisition_custom/i18n/ar_001.po
@@ -669,12 +669,12 @@ msgstr "الكمية المطلوبة"
 #: model:ir.model.fields.selection,name:purchase_requisition_custom.selection__purchase_order__purchase_cost__department
 #: model:ir.model.fields.selection,name:purchase_requisition_custom.selection__purchase_requisition__purchase_cost__department
 msgid "Department"
-msgstr "الإدارة"
+msgstr "القسم"
 
 #. module: purchase_requisition_custom
 #: model:ir.model.fields,field_description:purchase_requisition_custom.field_purchase_order_line__department_name
 msgid "Department Name"
-msgstr "اسم الإدارة"
+msgstr "اسم القسم"
 
 #. module: purchase_requisition_custom
 #: model:ir.model.fields,field_description:purchase_requisition_custom.field_purchase_request_line__description
@@ -1956,41 +1956,6 @@ msgstr "الفرع"
 msgid "Requested by Purchase"
 msgstr "الطلب لموظف اخر؟"
 
-#. module: purchase_requisition_custom
-#: model_terms:ir.ui.view,arch_db:purchase_requisition_custom.purchase_request_form
-msgid "Requester Information"
-msgstr "معلومات مقدم الطلب"
-
-#. module: purchase_requisition_custom  
-#: model_terms:ir.ui.view,arch_db:purchase_requisition_custom.purchase_request_form
-msgid "Request Details"
-msgstr "تفاصيل الطلب"
-
-#. module: purchase_requisition_custom
-#: model_terms:ir.ui.view,arch_db:purchase_requisition_custom.purchase_request_form  
-msgid "Vendor & Accounting"
-msgstr "المورد والمحاسبة"
-
-#. module: purchase_requisition_custom
-#: model_terms:ir.ui.view,arch_db:purchase_requisition_custom.purchase_requisition_custom_form_view
-msgid "Purchase Configuration"
-msgstr "إعدادات الشراء"
-
-#. module: purchase_requisition_custom
-#: model_terms:ir.ui.view,arch_db:purchase_requisition_custom.purchase_requisition_custom_form_view
-msgid "Department & Request Details"  
-msgstr "القسم وتفاصيل الطلب"
-
-#. module: purchase_requisition_custom
-#: model_terms:ir.ui.view,arch_db:purchase_requisition_custom.purchase_order_form_inherit
-msgid "Request Information"
-msgstr "معلومات الطلب"
-
-#. module: purchase_requisition_custom
-#: model_terms:ir.ui.view,arch_db:purchase_requisition_custom.purchase_order_form_inherit
-msgid "Status & Tracking"
-msgstr "الحالة والمتابعة"
-
 #. module: purchase_requisition_custom
 #: model:ir.model.fields,field_description:purchase_requisition_custom.field_change_purchase_user_state__requisition_id
 msgid "Requisition"
@@ -2844,7 +2809,7 @@ msgstr "الموظف"
 #. module: purchase_requisition_custom
 #: model_terms:ir.ui.view,arch_db:purchase_requisition_custom.purchase_request_search
 msgid "Department"
-msgstr "الإدارة"
+msgstr "القسم"
 
 #. module: purchase_requisition_custom
 #: model_terms:ir.ui.view,arch_db:purchase_requisition_custom.purchase_request_search
diff --git a/odex25_purchase/purchase_requisition_custom/i18n/ar_SY.po b/odex25_purchase/purchase_requisition_custom/i18n/ar_SY.po
index e063564d1..ce6081ab4 100644
--- a/odex25_purchase/purchase_requisition_custom/i18n/ar_SY.po
+++ b/odex25_purchase/purchase_requisition_custom/i18n/ar_SY.po
@@ -2481,7 +2481,7 @@ msgstr "الموظف"
 #. module: purchase_requisition_custom
 #: model_terms:ir.ui.view,arch_db:purchase_requisition_custom.purchase_request_search
 msgid "Department"
-msgstr "الإدارة"
+msgstr "القسم"
 
 #. module: purchase_requisition_custom
 #: model_terms:ir.ui.view,arch_db:purchase_requisition_custom.purchase_request_search
diff --git a/odex25_purchase/purchase_requisition_custom/views/purchase_request.xml b/odex25_purchase/purchase_requisition_custom/views/purchase_request.xml
index 42cc61dd5..94b8a3cd5 100644
--- a/odex25_purchase/purchase_requisition_custom/views/purchase_request.xml
+++ b/odex25_purchase/purchase_requisition_custom/views/purchase_request.xml
@@ -30,7 +30,6 @@
                                 attrs="{'invisible' : ['|',('state' , '!=' , 'waiting'),('purchase_create' , '=' , True)]}"/>
 
                         <button name="action_refuse" type="object" string="Refuse" id="prc_request_refuse"
-                                class="btn-secondary"
                                 groups="purchase_requisition_custom.group_refuse_purchase"
                                 attrs="{'invisible' : [('state' , 'not in' , ('direct_manager','waiting'))]}"/>
 
@@ -48,21 +47,13 @@
                             <field name="is_requisition" invisible="1"/>
                             <field name="is_creator" invisible="1"/>
                             <button class="oe_stat_button" name="open_requisition" type="object"
-                                    icon="fa-handshake-o"
+                                    string="Purchase Agreement" icon="fa-list-ol"
                                     attrs="{'invisible' : [('is_requisition' , '=' , False)]}"
-                                    groups="purchase.group_purchase_user,purchase.group_purchase_manager">
-                                <div class="o_field_widget o_stat_info">
-                                    <span class="o_stat_text">Purchase Agreement</span>
-                                </div>
-                            </button>
+                                    groups="purchase.group_purchase_user,purchase.group_purchase_manager"/>
                             <button class="oe_stat_button" name="open_purchase" type="object"
-                                    icon="fa-shopping-cart"
+                                    string="Purchase Orders" icon="fa-list-ol"
                                     attrs="{'invisible' : [('purchase_create' , '=' , False)]}"
-                                    groups="purchase.group_purchase_user,purchase.group_purchase_manager">
-                                <div class="o_field_widget o_stat_info">
-                                    <span class="o_stat_text">Purchase Orders</span>
-                                </div>
-                            </button>
+                                    groups="purchase.group_purchase_user,purchase.group_purchase_manager"/>
                         </div>
                         <div class="oe_title">
                             <!--                            <label for="name" class="oe_edit_only"/>-->
@@ -75,13 +66,13 @@
                             <field name="picking_type_id" invisible="1"/>
                             <field name="department_id" force_save="1"
                                    attrs="{'readonly' : ['|',('state' , '!=' , 'draft'),('by_purchase' , '!=' , True)] , 'required' : [('by_purchase' , '=' , True)]}"
-                                   readonly="1"/>
-                            <field name="branch_id" readonly="1"/>
+                                   readonly="1" context="{'show_department_short': True}"/>
+                            <field name="branch_id" readonly="1" context="{'show_branch_short': True}"/>
                             <field name="employee_id" attrs="{'readonly':[('by_purchase' , '=' , False)]}" required="1"
                                    options="{'no_create' : True , 'no_edit' : True ,'no_open' : True}"/>
                             <field name="product_category_ids" widget="many2many_tags"/>
                             <field name="purchase_purpose" required="1"
-                                   attrs="{'readonly':[('state' , '!=' , 'draft')]}" placeholder="Request Purpose..."/>
+                                   attrs="{'readonly':[('state' , '!=' , 'draft')]}"/>
                             <field name="date" readonly="1"/>
                             <field name="purchase_create" invisible="1"/>
                             <field name="partner_id"
@@ -100,7 +91,6 @@
                             <field name="edit_partner_id" invisible="1"/>
 
                         </group>
-
                         <notebook>
                             <page string="Items" name="purchase_request_info">
                                 <field name="line_ids" attrs="{'readonly':[('state' , '!=' , 'draft')]}">
@@ -108,16 +98,22 @@
                                         <field name="product_id"
                                                domain="[('purchase_ok', '=', True)]"
                                                options="{'no_create' : True , 'no_edit' : True ,'no_open' : True}"
-                                               required="1"/>
-                                        <field name="description"/>
-                                        <field name="qty" sum="Total Quantity"
-                                               attrs="{'column_required':[('parent.state' , '!=' , 'draft')]}"/>
+                                               optional="show"
+                                                required="1"/>
+
                                         <field name="uom_id"/>
+                                        <field name="description"/>
+
+                                        <field name="qty"
+                                               attrs="{'column_required':[('parent.state' , '!=' , 'draft')]}"
+                                               optional="show"/>
+                                        <!--                                        <field name="price_unit" attrs="{'column_required':[('parent.state' , '!=', 'draft')]}" optional="show"/>-->
+                                        <!--                                        <field name="sum_total" force_save="1" sum="Total amount" optional="show"/>-->
                                         <field name="account_id" invisible="1"/>
                                     </tree>
                                 </field>
                             </page>
-                            <page string="Note" name="notes">
+                            <page string="Note">
                                 <field name="note"/>
                             </page>
                         </notebook>
@@ -139,8 +135,8 @@
                     <field name="name"/>
                     <field name="date"/>
                     <field name="employee_id"/>
-                    <field name="department_id"/>
-                    <field name="branch_id"/>
+                    <field name="department_id" context="{'show_department_short': True}"/>
+                    <field name="branch_id" context="{'show_branch_short': True}"/>
                     <field name="product_category_ids" widget="many2many_tags"/>
                     <field name="state"/>
                     <field name="purchase_state_summary" optional="show"/>
@@ -156,8 +152,8 @@
                 <search string="Purchase Request">
                     <field name="name" string="Request Number"/>
                     <field name="employee_id" string="Employee"/>
-                    <field name="department_id" string="Department"/>
-                    <field name="branch_id" string="Branch"/>
+                    <field name="department_id" string="Department" context="{'show_department_short': True}"/>
+                    <field name="branch_id" string="Branch" context="{'show_branch_short': True}"/>
                     <field name="purchase_purpose" string="Purpose"/>
                     <field name="partner_id" string="Vendor"/>
                     
@@ -180,7 +176,16 @@
                             domain="[('employee_id.user_id','=',uid)]"/>
                     <filter string="My Department" name="my_department" 
                             domain="[('department_id.manager_id.user_id','=',uid)]"/>
+                    <filter string="Pending Approval" name="pending_approval" 
+                            domain="[('state','in',['draft','direct_manager'])]"/>
+                    <filter string="In Progress" name="in_progress" 
+                            domain="[('state','in',['waiting'])]"/>
                     
+                    <separator/>
+                    <filter string="Project Requests" name="project_type" 
+                            domain="[('type','=','project')]"/>
+                    <filter string="Operational Requests" name="operational_type" 
+                            domain="[('type','=','operational')]"/>
                     
                     <separator/>
                     <filter string="With Vendor" name="with_vendor" 
@@ -192,9 +197,11 @@
                     <group expand="0" string="Group By">
                         <filter string="State" name="group_state" context="{'group_by':'state'}"/>
                         <filter string="Employee" name="group_employee" context="{'group_by':'employee_id'}"/>
-                        <filter string="Department" name="group_department" context="{'group_by':'department_id'}"/>
+                        <filter string="Department" name="group_department" context="{'group_by':'department_id', 'show_department_short': True}"/>
                         <filter string="Branch" name="group_branch" context="{'group_by':'branch_id'}"/>
                         <filter string="Vendor" name="group_vendor" context="{'group_by':'partner_id'}"/>
+                        <filter string="Request Type" name="group_type" context="{'group_by':'type'}"/>
+                        <filter string="Product Categories" name="group_categories" context="{'group_by':'product_category_ids'}"/>
                     </group>
                 </search>
             </field>
diff --git a/odex25_purchase/purchase_requisition_custom/views/purchase_requisition_custom.xml b/odex25_purchase/purchase_requisition_custom/views/purchase_requisition_custom.xml
index f8f8d0010..71d3e555f 100644
--- a/odex25_purchase/purchase_requisition_custom/views/purchase_requisition_custom.xml
+++ b/odex25_purchase/purchase_requisition_custom/views/purchase_requisition_custom.xml
@@ -42,7 +42,7 @@
             <field name="arch" type="xml">
                 <form string="committe type">
                     <sheet>
-                        <group col="4" colspan="2">
+                        <group col="4" colsapn="2">
                             <field name="name" required="1"/>
                             <field name="committe_head"/>
                             <field name="type_cat" widget="many2many_tags"/>
@@ -98,7 +98,7 @@
 
                 <xpath expr="//field[@name='partner_id']" position="after">
                     <field name="recommendation_order" string="Recommend Order"/>
-                    <field name="branch_id"/>
+                    <field name="branch_id" context="{'from_branch_field': True}"/>
                 </xpath>
 
 
@@ -187,7 +187,7 @@
 
                     <button type="object" name="action_refuse" groups="purchase_requisition_custom.committe_member"
                             attrs="{'invisible':['|',('state','in',['done','purchase','waiting','cancel']),('can_committee_vote', '=', False)]}"
-                            string="Refuse" class="btn-secondary"/>
+                            string="Refuse"/>
                     <button type="object" name="action_recommend"
                             groups="purchase_requisition_custom.group_select_recommended_offer"
                             confirm="Are you sure you want to process ?"
@@ -282,10 +282,12 @@
                 <xpath expr="//field[@name='origin']" position="replace">
                     <field name="send_to_budget" invisible="1"/>
                     <field name="email_to_vendor" invisible="1"/>
+                    <field name="request_id" readonly="1"/>
                     <field name="project_id" readonly="1"/>
                     <field name="purchase_cost" invisible="1"/>
                     <field name="budget_amount" invisible="1"/>
                     <field name="is_signed" invisible="1"/>
+
                 </xpath>
                 <xpath expr="//field[@name='order_line']" position="attributes">
                     <attribute name="attrs">{'readonly':[('state','in',('done','cancel','waiting'))]}</attribute>
@@ -297,20 +299,15 @@
                     <attribute name="attrs">{'readonly':[('state','in',('done','cancel','waiting'))]}</attribute>
                 </xpath>
                 <xpath expr="//field[@name='currency_id']" position="after">
-                    <separator string="Request Information"/>
-                    <group col="2" colspan="2">
-                        <field name="request_id" readonly="1"/>
-                        <field name="purchase_request_employee_id"/>
-                        <field name="department_id"/>
-                        <field name="branch_id" readonly="1"/>
-                        <field name="purpose" attrs="{'readonly': [('state','in',['done','cancel'])]}" colspan="2"/>
-                    </group>
-                    <separator string="Status &amp; Tracking"/>
-                    <group col="2" colspan="2">
-                        <field name="state_of_delivery"/>
-                        <field name="no_of_approve" readonly="1"/>
-                    </group>
+                    <field name="state_of_delivery"/>
                     <field name="is_purchase_budget" invisible="1"/>
+                    <field name="no_of_approve" readonly="1"/>
+                    <!--here -->
+                    <field name="purchase_cost" invisible="1"/>
+                    <field name="department_id" invisible="0" context="{'show_department_short': True}"/>
+                    <field name="branch_id" readonly="1" context="{'show_branch_short': True}"/>
+                    <field name="purchase_request_employee_id"/>
+                    <field name="purpose" attrs="{'readonly': [('state','in',['done','cancel'])]}"/>
                 </xpath>
                 <xpath expr="//field[@name='order_line']" position="before">
                     <button name="action_select_all" class="oe_highlight" type="object"
@@ -415,7 +412,7 @@
                 </xpath>
                 <xpath expr="//field[@name='line_ids']/tree[1]/field[@name='product_qty']" position="before">
                     <field name="department_id" attrs="{'column_invisible': [('parent.purchase_cost', '=', 'default')],
-                            'readonly': ['|', ('parent.state', '=', 'accept'), ('parent.order_count', '>', 0)]}"/>
+                            'readonly': ['|', ('parent.state', '=', 'accept'), ('parent.order_count', '>', 0)]}" context="{'show_department_short': True}"/>
                 </xpath>
 
                 <xpath expr="/form/header/button[@name='action_done']" position="replace">
@@ -455,7 +452,7 @@
                 <xpath expr="/form/header/button[@name='action_cancel']" position="replace">
                     <button name="action_cancel" groups="purchase_requisition_custom.group_cancel_purchase_requisition"
                             states="draft,in_progress,ongoing,checked,purchase_manager,rejected_by_committee"
-                            string="Cancel" type="object" class="btn-secondary"/>
+                            string="Cancel" type="object"/>
                 </xpath>
                 <xpath expr="/form/header/button[@name='action_draft']" position="attributes">
                     <attribute name="attrs">{'invisible':[('state','!=','cancel')]}</attribute>
@@ -477,33 +474,25 @@
                     <!--                    <field name="city" attrs="{'readonly': [('state','!=', 'draft')]}"/>-->
                 </xpath>
                 <xpath expr="//field[@name='user_id']" position="after">
-                    <separator string="Purchase Configuration"/>
-                    <group col="4">
-                        <field name="category_ids" required="1"
-                               attrs="{'readonly':['|',('state','in',('cancel','checked','done','waiting')),('check_request','=',True)]}"
-                               widget="many2many_tags" colspan="4"/>
-                        
-                        <field name="purchase_cost" required="1" widget="radio" options="{'horizontal': true}"
-                               attrs="{'readonly':[('state','in',('cancel','checked','done','waiting'))]}" colspan="4"/>
-                        
-                        <field name="type" required="1" widget="radio" options="{'horizontal': true}"
-                               attrs="{'readonly':[('state','in',('cancel','checked','done','waiting'))]}" colspan="2"/>
-                        <field name="project_id" colspan="2"
-                               attrs="{'invisible':[('type','!=', 'project')], 'required':[('type','=', 'project')], 'readonly':[('state' , '!=' , 'draft')]}"/>
-                    </group>
-                    
-                    <separator string="Department &amp; Request Details"/>
-                    <group col="4">
-                        <field name="department_id" attrs="{'required':[('purchase_cost','=', 'department')], 'invisible':[('purchase_cost','=', 'product_line')],
-                               'readonly':['|',('state','in',('cancel','checked','done','waiting')),('check_request','=',True)]}" colspan="2"/>
-                        <field name="request_id" readonly="1" colspan="2"/>
-                        
-                        <field name="purpose" required="1" colspan="4"
-                               attrs="{'readonly':['|',('state','in',('cancel','checked','done','waiting')),('check_request','=',True)]}"/>
-                    </group>
-                    
-                    <!-- Hidden fields -->
                     <field name="check_request" invisible="1"/>
+                    <field name="category_ids" required="1"
+                           attrs="{'readonly':['|',('state','in',('cancel','checked','done','waiting')),('check_request','=',True)]}"
+                           widget="many2many_tags"/>
+
+                    <field name="purchase_cost" required="1"
+                           attrs="{'readonly':[('state','in',('cancel','checked','done','waiting'))]}"/>
+
+                    <field name="department_id" attrs="{'required':[('purchase_cost','=', 'department')], 'invisible':[('purchase_cost','=', 'product_line')],
+                           'readonly':['|',('state','in',('cancel','checked','done','waiting')),('check_request','=',True)]}" context="{'show_department_short': True}"/>
+                    <field name="purpose" required="1"
+                           attrs="{'readonly':['|',('state','in',('cancel','checked','done','waiting')),('check_request','=',True)]}"/>
+
+                    <field name="type" required="1"
+                           attrs="{'readonly':[('state','in',('cancel','checked','done','waiting'))]}"/>
+                    <field name="project_id"
+                           attrs="{'invisible':[('type','!=', 'project')], 'required':[('type','=', 'project')], 'readonly':[('state' , '!=' , 'draft')]}"/>
+
+                    <field name="request_id" readonly="1"/>
                     <field name="type_exclusive" invisible="1"/>
                 </xpath>
                 <xpath expr="//field[@name='vendor_id']" position="attributes">
@@ -630,8 +619,8 @@
                     <field name="origin" invisible="1"/>
                     <field name="reject_reason" required="1"/>
                     <footer>
-                        <button name="action_reject" string="Confirm" type="object" class="btn-secondary"/>
-                        <button string="Cancel" class="btn-secondary" special="cancel"/>
+                        <button name="action_reject" string="Confirm" type="object" class="oe_highlight"/>
+                        <button string="Cancel" class="btn btn-default" special="cancel"/>
                     </footer>
                 </form>
             </field>
@@ -662,8 +651,8 @@
                     <field name="order_id" invisible="1"/>
                     <field name="refuse_reason" required="1"/>
                     <footer>
-                        <button name="action_refuse" string="Confirm" type="object" class="btn-secondary"/>
-                        <button string="Cancel" class="btn-secondary" special="cancel"/>
+                        <button name="action_refuse" string="Confirm" type="object" class="oe_highlight"/>
+                        <button string="Cancel" class="btn btn-default" special="cancel"/>
                     </footer>
                 </form>
             </field>