From 3bee6ebb0a8b1f8909af810aa2cc5c24e040d286 Mon Sep 17 00:00:00 2001 From: Christophe Simonis Date: Fri, 6 Jan 2017 17:55:04 +0100 Subject: [PATCH] [FIX] core: effectively warn on invalid custom views Since new-api migration of `ir.ui.view`, the method that check views raises exceptions instead of returning a boolean. Show them as warning. Related to #14521 --- openerp/modules/loading.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/openerp/modules/loading.py b/openerp/modules/loading.py index 92d174dcd23..be94697ad1d 100644 --- a/openerp/modules/loading.py +++ b/openerp/modules/loading.py @@ -402,8 +402,10 @@ def load_modules(db, force_demo=False, status=None, update_module=False): if update_module: Views = registry['ir.ui.view'] for model in registry.models.keys(): - if not Views._validate_custom_views(cr, SUPERUSER_ID, model): - _logger.warning('Invalid custom view(s) for model %s', model) + try: + Views._validate_custom_views(cr, SUPERUSER_ID, model) + except Exception as e: + _logger.warning('invalid custom view(s) for model %s: %s', model, tools.ustr(e)) if report.failures: _logger.error('At least one test failed when loading the modules.')