From 31983e0c2e6a307032f6836107b162c4eefbed1b Mon Sep 17 00:00:00 2001 From: David Monjoie Date: Fri, 15 Mar 2019 14:08:21 +0000 Subject: [IMP] web: only display missing view erorr in debug assets This error is there to help developpers realize their view has not been loaded due to an error. However, we have cases where we define actions containing views that only exist in Enterprise to avoid creating an entire bridge module just to add one view to an action. The error was triggered in this case which resulted in a red build for the click all runbot test, even though this was not really a true error, as it was intended. This commit changes the error to only appear in debug=assets such that it will no longer put the click all runbot tour red while still providing help for the developper trying to debug his view since he will need to be in debug=assets to debug his view anyway. closes odoo/odoo#31884 Signed-off-by: Aaron Bohy (aab) --- addons/web/static/src/js/chrome/action_manager_act_window.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'addons/web/static/src') diff --git a/addons/web/static/src/js/chrome/action_manager_act_window.js b/addons/web/static/src/js/chrome/action_manager_act_window.js index 8d13f4d9679..ad5749a16b7 100644 --- a/addons/web/static/src/js/chrome/action_manager_act_window.js +++ b/addons/web/static/src/js/chrome/action_manager_act_window.js @@ -358,7 +358,7 @@ ActionManager.include({ viewID: view[0], Widget: View, }); - } else { + } else if (config.debug === 'assets') { console.error("View type '" + viewType + "' is not present in the view registry."); } }); -- cgit v1.2.3