From 27d2a81b31df56f405932e99e903f3c576f35add Mon Sep 17 00:00:00 2001
From: laurentbarontini <l.barontini@open-squared.ch>
Date: Wed, 29 Jul 2026 09:23:30 +0200
Subject: [PATCH] Group physical position reports

---
 modules/purchase_trade/ctrm_reporting.xml   |  2 +-
 modules/purchase_trade/global_reporting.xml | 26 +++++++++++++--------
 2 files changed, 17 insertions(+), 11 deletions(-)

diff --git a/modules/purchase_trade/ctrm_reporting.xml b/modules/purchase_trade/ctrm_reporting.xml
index 20ea31b..62b66ea 100644
--- a/modules/purchase_trade/ctrm_reporting.xml
+++ b/modules/purchase_trade/ctrm_reporting.xml
@@ -329,7 +329,7 @@
             icon="tradon-reporting-positions"/>
         <menuitem
             name="Physical Position"
-            parent="menu_ctrm_positions"
+            parent="menu_ctrm_positions_physical_group"
             sequence="10"
             action="act_ctrm_position_physical"
             id="menu_ctrm_position_physical"/>
diff --git a/modules/purchase_trade/global_reporting.xml b/modules/purchase_trade/global_reporting.xml
index 40a8ed6..4e3decf 100644
--- a/modules/purchase_trade/global_reporting.xml
+++ b/modules/purchase_trade/global_reporting.xml
@@ -151,34 +151,40 @@
             id="menu_ctrm_positions"
             icon="tradon-reporting-positions"/>
         <menuitem
-            name="Long &amp; Short"
+            name="Physical Position"
             parent="menu_ctrm_positions"
-            action="act_ctrm_open_position"
             sequence="10"
+            id="menu_ctrm_positions_physical_group"
+            icon="tradon-reporting-positions"/>
+        <menuitem
+            name="Long &amp; Short"
+            parent="menu_ctrm_positions_physical_group"
+            action="act_ctrm_open_position"
+            sequence="20"
             id="menu_ctrm_positions_long_short"/>
         <menuitem
             name="By Product &amp; Strategy"
-            parent="menu_ctrm_positions"
+            parent="menu_ctrm_positions_physical_group"
             action="act_ctrm_long_short_product_strategy"
-            sequence="20"
+            sequence="40"
             id="menu_ctrm_positions_long_short_product_strategy"/>
         <menuitem
             name="By Commodity &amp; Direction"
-            parent="menu_ctrm_positions"
+            parent="menu_ctrm_positions_physical_group"
             action="act_ctrm_long_short_commodity_direction"
-            sequence="30"
+            sequence="50"
             id="menu_ctrm_positions_long_short_commodity_direction"/>
         <menuitem
             name="By Period &amp; Product"
-            parent="menu_ctrm_positions"
+            parent="menu_ctrm_positions_physical_group"
             action="act_ctrm_long_short_period_product"
-            sequence="40"
+            sequence="30"
             id="menu_ctrm_positions_long_short_period_product"/>
         <menuitem
             name="By Pricing Status"
-            parent="menu_ctrm_positions"
+            parent="menu_ctrm_positions_physical_group"
             action="act_ctrm_long_short_pricing_status"
-            sequence="50"
+            sequence="60"
             id="menu_ctrm_positions_long_short_pricing_status"/>
         <menuitem
             name="Other"