From 41b2baef6d2a511721f544506173ba7510ec2d74 Mon Sep 17 00:00:00 2001
From: Sylvain LE GAL <sylvain.legal@grap.coop>
Date: Tue, 23 Jul 2024 10:49:52 +0200
Subject: [PATCH] [IMP] bi_sql_editor : better display of UI settings

---
 bi_sql_editor/views/view_bi_sql_view.xml | 34 +++++++++++-------------
 1 file changed, 16 insertions(+), 18 deletions(-)

diff --git a/bi_sql_editor/views/view_bi_sql_view.xml b/bi_sql_editor/views/view_bi_sql_view.xml
index f8f36c83b..1a7ce7afb 100644
--- a/bi_sql_editor/views/view_bi_sql_view.xml
+++ b/bi_sql_editor/views/view_bi_sql_view.xml
@@ -93,17 +93,16 @@ License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
                         name="technical_name"
                         attrs="{'readonly': [('state', '!=', 'draft')]}"
                     />
-                    <field name="view_order" />
                     <field name="is_materialized" />
                     <field
                         name="size"
-                        attrs="{'invisible': ['|', ('state', '=', 'draft'), ('is_materialized', '=', False)]}"
-                    />
-                    <field
-                        name="cron_id"
                         attrs="{'invisible': ['|', ('state', 'in', ('draft', 'sql_valid')), ('is_materialized', '=', False)]}"
                     />
                 </group>
+                <group>
+                    <field name="view_order" />
+                    <field name="parent_menu_id" />
+                </group>
             </group>
             <page name="page_sql" position="after">
                 <page
@@ -188,6 +187,10 @@ License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
                         <page string="Extras Information">
                             <group>
                                 <group string="Technical Details">
+                                    <field
+                                name="cron_id"
+                                attrs="{'invisible': ['|', ('state', 'in', ('draft', 'sql_valid')), ('is_materialized', '=', False)]}"
+                            />
                                     <field name="view_name" />
                                     <field name="model_name" />
                                     <field
@@ -195,19 +198,14 @@ License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
                                 attrs="{'invisible': [('state', '=', 'draft')]}"
                             />
                                 </group>
-                                <group string="User Interface">
-                                    <group string="UI Parameters">
-                                        <field name="parent_menu_id" />
-                                    </group>
-                                    <group string="UI Instances">
-                                        <field name="form_view_id" />
-                                        <field name="tree_view_id" />
-                                        <field name="graph_view_id" />
-                                        <field name="pivot_view_id" />
-                                        <field name="search_view_id" />
-                                        <field name="action_id" />
-                                        <field name="menu_id" />
-                                    </group>
+                                <group string="UI Instances">
+                                    <field name="form_view_id" />
+                                    <field name="tree_view_id" />
+                                    <field name="graph_view_id" />
+                                    <field name="pivot_view_id" />
+                                    <field name="search_view_id" />
+                                    <field name="action_id" />
+                                    <field name="menu_id" />
                                 </group>
                             </group>
                         </page>