From f6c84d709a84fa9769bd95d08bfeb89d2a5eb0a5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?R=C3=A9mi=20Rahir?= <rar@odoo.com>
Date: Wed, 25 May 2022 19:28:33 +0000
Subject: [PATCH] [FIX] project: prevent menu_service from loading
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Continuity of b859d78c. The assets generated for project sharing feature
suffer from the same issue than `point_of_sale`. In master, we have to
properly declare every file we need instead of realying on the whole
`web.assets_backend`.

Part of task 2860257

closes odoo/odoo#92577

X-original-commit: 45db27c3230a7c341206e9f990e8e5a133342c9f
Related: odoo/enterprise#27914
Signed-off-by: Lucas Lefèvre (lul) <lul@odoo.com>
Signed-off-by: Rémi Rahir (rar) <rar@odoo.com>
---
 addons/project/__manifest__.py                     | 1 -
 addons/project/views/project_sharing_templates.xml | 3 +++
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/addons/project/__manifest__.py b/addons/project/__manifest__.py
index 85e4b1b4c31..0aa3d5970b9 100644
--- a/addons/project/__manifest__.py
+++ b/addons/project/__manifest__.py
@@ -109,7 +109,6 @@
         ],
         'project.webclient': [
             ('include', 'web.assets_backend'),
-            ('remove', 'web/static/src/webclient/menus/*.js'),
 
             # Remove Longpolling bus and packages needed this bus
             ('remove', 'bus/static/src/js/services/assets_watchdog_service.js'),
diff --git a/addons/project/views/project_sharing_templates.xml b/addons/project/views/project_sharing_templates.xml
index 9633d84b2ce..9e9f380c79c 100644
--- a/addons/project/views/project_sharing_templates.xml
+++ b/addons/project/views/project_sharing_templates.xml
@@ -18,6 +18,9 @@
             <t t-set="head_project_sharing">
                 <script type="text/javascript">
                     odoo.__session_info__ = <t t-out="json.dumps(session_info)"/>;
+                    // Prevent the menu_service to load anything. In an ideal world, Project Sharing assets would only contain
+                    // what is genuinely necessary, and not the whole backend.
+                    odoo.loadMenusPromise = Promise.resolve();
                     odoo.loadTemplatesPromise = fetch(`/web/webclient/qweb/${odoo.__session_info__.cache_hashes.qweb}?bundle=project.assets_qweb`).then(doc => doc.text());
                 </script>
                 <base target="_parent"/>