From a1b29f38344e09e5b59c8af348717f684fc3cf1e Mon Sep 17 00:00:00 2001
From: Ryan Cole <admin@ryanc.me>
Date: Mon, 24 Feb 2025 14:29:41 +1300
Subject: [PATCH] [FIX] web_responsive: Fix blank screen on search (support
 menus w/o xmlid)

If a menu item is created manually via the Odoo UI, then it (likely) won't have an xmlid. As a result, the `t-foreach` fails, resulting in the whole Odoo UI crashing (a blank white screen is shown). This code should use `id` as the `t-key` instead, as that is *always* available, even for menu items created manually.
---
 .../src/components/menu_canonical_searchbar/searchbar.xml     | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/web_responsive/static/src/components/menu_canonical_searchbar/searchbar.xml b/web_responsive/static/src/components/menu_canonical_searchbar/searchbar.xml
index 023864a0f6df..1dcff5ad7e3e 100644
--- a/web_responsive/static/src/components/menu_canonical_searchbar/searchbar.xml
+++ b/web_responsive/static/src/components/menu_canonical_searchbar/searchbar.xml
@@ -25,7 +25,7 @@
                 class="list-unstyled search-list"
                 t-ref="searchItems"
             >
-                <t t-foreach="state.rootItems" t-as="menu" t-key="menu.xmlid">
+                <t t-foreach="state.rootItems" t-as="menu" t-key="menu.id">
                     <li t-attf-class="search-item {{highlighted(menu_index)}}">
                         <a
                             t-attf-class="search-item__link"
@@ -59,7 +59,7 @@
                 >
                     <hr class="w-100" />
                 </li>
-                <t t-foreach="state.subItems" t-as="menu" t-key="menu.xmlid">
+                <t t-foreach="state.subItems" t-as="menu" t-key="menu.id">
                     <li t-attf-class="search-item {{highlighted(menu_index, true)}}">
                         <a
                             t-attf-class="search-item__link"