From 5d7bba4eb7706aebff2090d20e630ed656c27b18 Mon Sep 17 00:00:00 2001 From: Raphael Collet Date: Thu, 15 Jan 2015 11:43:23 +0100 Subject: [PATCH] [FIX] loading: move code that should not be executed on simply installed modules --- openerp/modules/loading.py | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/openerp/modules/loading.py b/openerp/modules/loading.py index f80332d8dd5..1694fdbccf7 100644 --- a/openerp/modules/loading.py +++ b/openerp/modules/loading.py @@ -162,13 +162,6 @@ def load_module_graph(cr, graph, status=None, perform_checks=True, skip_modules= registry.setup_models(cr, partial=True) init_module_models(cr, package.name, models) - # Can't put this line out of the loop: ir.module.module will be - # registered by init_module_models() above. - modobj = registry['ir.module.module'] - - if perform_checks: - modobj.check(cr, SUPERUSER_ID, [module_id]) - idref = {} mode = 'update' @@ -176,6 +169,13 @@ def load_module_graph(cr, graph, status=None, perform_checks=True, skip_modules= mode = 'init' if hasattr(package, 'init') or hasattr(package, 'update') or package.state in ('to install', 'to upgrade'): + # Can't put this line out of the loop: ir.module.module will be + # registered by init_module_models() above. + modobj = registry['ir.module.module'] + + if perform_checks: + modobj.check(cr, SUPERUSER_ID, [module_id]) + if package.state=='to upgrade': # upgrading the module information modobj.write(cr, SUPERUSER_ID, [module_id], modobj.get_values_from_terp(package.data))