From 04c955fed0fe082b87f628c1a33063011105b599 Mon Sep 17 00:00:00 2001
From: mohammed-alkhazrji <M.Alkhazrji@exp-sa.com>
Date: Sun, 7 Dec 2025 22:41:30 +0300
Subject: [PATCH] fix petty cash

---
 .../purchase_petty_invoice/models/purchase_order.py    | 10 ++++++++++
 .../views/purchase_order_view.xml                      |  5 +++--
 .../models/purchase_order.py                           |  2 +-
 .../views/purchase_requisition_custom.xml              |  6 +++---
 4 files changed, 17 insertions(+), 6 deletions(-)

diff --git a/odex25_purchase/purchase_petty_invoice/models/purchase_order.py b/odex25_purchase/purchase_petty_invoice/models/purchase_order.py
index db65af6a0..1a42aa1e3 100644
--- a/odex25_purchase/purchase_petty_invoice/models/purchase_order.py
+++ b/odex25_purchase/purchase_petty_invoice/models/purchase_order.py
@@ -9,6 +9,16 @@ class PurchaseOrder(models.Model):
     petty_employee_id = fields.Many2one('hr.employee', string='Petty Cashier', copy=False)
 
 
+    def _prepare_invoice(self):
+        res = super(PurchaseOrder, self)._prepare_invoice()
+        res.update({
+            'purchase_id': self.id,
+            'res_id': self.id,
+            'res_model': 'purchase.order',
+            'is_petty_paid': self.is_petty_paid,
+            'petty_employee_id': self.petty_employee_id.id
+        })
+        return res
 
 class AccountMove(models.Model):
     _inherit = 'account.move'
diff --git a/odex25_purchase/purchase_petty_invoice/views/purchase_order_view.xml b/odex25_purchase/purchase_petty_invoice/views/purchase_order_view.xml
index b979d6aac..56a29e9cf 100644
--- a/odex25_purchase/purchase_petty_invoice/views/purchase_order_view.xml
+++ b/odex25_purchase/purchase_petty_invoice/views/purchase_order_view.xml
@@ -23,9 +23,10 @@
         <field name="inherit_id" ref="purchase.purchase_order_form"/>
         <field name="arch" type="xml">
             <xpath expr="//field[@name='partner_id']" position="after">
-                <field name="is_petty_paid"/>
+                <field name="is_petty_paid"  attrs="{'readonly': [('state', 'in', ['sign', 'wait'])]}"/>
                 <field name="petty_employee_id" options='{"no_open": True, "no_create": True}'
-                       attrs="{'invisible':[('is_petty_paid','=',False)],'required':[('is_petty_paid','=',True)]}"/>
+                                           attrs="{'invisible': [('is_petty_paid', '=', False)],
+                               'readonly': [('state', 'in', ['sign', 'wait'])]}"/>
             </xpath>
         </field>
     </record>
diff --git a/odex25_purchase/purchase_requisition_custom/models/purchase_order.py b/odex25_purchase/purchase_requisition_custom/models/purchase_order.py
index f6341d00e..b45a8bd57 100644
--- a/odex25_purchase/purchase_requisition_custom/models/purchase_order.py
+++ b/odex25_purchase/purchase_requisition_custom/models/purchase_order.py
@@ -373,7 +373,7 @@ class PurchaseOrderCustom(models.Model):
 
     def _prepare_invoice(self):
         res = super(PurchaseOrderCustom, self)._prepare_invoice()
-        res.update({'purchase_id': self.id, 'res_id': self.id, 'res_model': 'purchase.order','is_petty_paid':self.is_cash_advance_spent,'petty_employee_id':self.cash_advance_employee_id})
+        res.update({'purchase_id': self.id, 'res_id': self.id, 'res_model': 'purchase.order'})
         return res
 
     @api.onchange('type')
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 06a9fb584..0142c5773 100644
--- a/odex25_purchase/purchase_requisition_custom/views/purchase_requisition_custom.xml
+++ b/odex25_purchase/purchase_requisition_custom/views/purchase_requisition_custom.xml
@@ -123,10 +123,10 @@
                 </xpath>
                 <xpath expr="//field[@name='requisition_id']" position="after">
                   <field name="is_cash_advance_spent"
-                       attrs="{'readonly': [('state', 'in', ['sign', 'wait'])]}"/>
+                         invisible="1"
+                                    />
                 <field name="cash_advance_employee_id"
-                       attrs="{'invisible': [('is_cash_advance_spent', '=', False)],
-                               'readonly': [('state', 'in', ['sign', 'wait'])]}"/>
+                       invisible="1"/>
 
                 </xpath>
                 <xpath expr="//field[@name='payment_term_id']" position="after">