From 873537b89aab48ce9f1e606b3221865fa7795d82 Mon Sep 17 00:00:00 2001
From: Saran440 <saranl@ecosoft.co.th>
Date: Fri, 11 Aug 2023 17:45:44 +0700
Subject: [PATCH] [FIX] activity_id not required when move is not affect budget

---
 budget_activity/views/account_move_views.xml | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/budget_activity/views/account_move_views.xml b/budget_activity/views/account_move_views.xml
index 603acd95..d244d025 100644
--- a/budget_activity/views/account_move_views.xml
+++ b/budget_activity/views/account_move_views.xml
@@ -11,16 +11,16 @@
             >
                 <field
                     name="activity_id"
-                    attrs="{'required': [('analytic_account_id', '!=', False)]}"
+                    attrs="{'required': [('analytic_account_id', '!=', False), ('parent.not_affect_budget', '=', False)]}"
                 />
             </xpath>
             <xpath
-                expr="//field[@name='line_ids']/tree/field[@name='product_id']"
-                position="after"
+                expr="//field[@name='line_ids']/tree/field[@name='account_id']"
+                position="before"
             >
                 <field
                     name="activity_id"
-                    attrs="{'required': [('analytic_account_id', '!=', False)]}"
+                    attrs="{'required': [('analytic_account_id', '!=', False), ('parent.not_affect_budget', '=', False)]}"
                 />
             </xpath>
         </field>