From 0211537664f37ee0ded4495fbb42ba94ce94c227 Mon Sep 17 00:00:00 2001
From: "matthias.lotz" <matthias.lotz@hobbyhimmel.de>
Date: Thu, 29 May 2025 16:37:37 +0200
Subject: [PATCH] [IMP] open_workshop: restructure partner form tab for machine
 access and safety briefing

- Combined machine access list and HTML overview into a unified tab "Offene Werkstatt (Hobbyhimmel)"
- Grouped security briefing fields into a compact two-column layout
- Improved usability by keeping related data in one place and reducing click depth
---
 views/res_partner_view.xml | 43 +++++++++++++++++---------------------
 1 file changed, 19 insertions(+), 24 deletions(-)

diff --git a/views/res_partner_view.xml b/views/res_partner_view.xml
index 01358fa..c96f9e0 100644
--- a/views/res_partner_view.xml
+++ b/views/res_partner_view.xml
@@ -8,35 +8,30 @@
     <field name="arch" type="xml">
       <xpath expr="//page[@name='sales_purchases']" position="before">
 
-        <!-- Tab 1: HOBBYHIMMEL Basis -->
-        <page name="ows_basic" string="HOBBYHIMMEL Basis">
-          <group name="container_row_2">
-            <group string="Sicherheit">
+        <page name="ows_machine_access" string="Offene Werkstatt (Hobbyhimmel)">
+          <group name="container_row_2" string="Sicherheitseinweisung">
+            <group>
               <field name="security_briefing"/>
+            </group>
+            <group>
               <field name="security_id"/>
-            </group>
-            <group string="Zugang">
-              <field name="rfid_card"/>
-            </group>
+            </group>  
+          </group>          
+          <group string="Maschinenfreigaben">
+            <field name="machine_access_ids" context="{'default_partner_id': id}" nolabel="1">
+              <tree>
+                <field name="machine_id"/>
+                <field name="date_granted"/>
+                <field name="date_expiry"/>
+                <field name="granted_by_pos"/>
+              </tree>
+            </field>
+          </group>        
+          <group string="Maschinenfreigaben Übersicht">
+            <field name="machine_access_html" readonly="1" widget="html"  nolabel="1"/>
           </group>
         </page>
 
-        <!-- Tab 2: HOBBYHIMMEL Einweisungen (HTML) -->
-        <page name="ows_machine_access_html" string="HOBBYHIMMEL Einweisungen">
-          <field name="machine_access_html" readonly="1" widget="html"/>
-        </page>
-
-        <!-- Tab 3: Einweisungen (Liste) -->
-        <page name="ows_machine_access_list" string="Einweisungen (Liste)">
-          <field name="machine_access_ids" context="{'default_partner_id': active_id}">
-            <tree>
-              <field name="machine_id"/>
-              <field name="date_granted"/>
-              <field name="date_expiry"/>
-              <field name="granted_by_pos"/>
-            </tree>
-          </field>
-        </page>
 
       </xpath>
     </field>