From 57818bb779374597d561f33faa092d0e0061f8a0 Mon Sep 17 00:00:00 2001 From: Denis Ledoux Date: Wed, 28 Aug 2013 14:52:09 +0200 Subject: [PATCH] [FIX]res_config: in _install_modules, result returned by button_immediate_install must be returned, in order to launch the action if there is one. TODO fix,at the moment, if there are missing modules, this return statement prevent their installation bzr revid: dle@openerp.com-20130828125209-5jqwjqhrl17x7kpa --- openerp/addons/base/res/res_config.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/openerp/addons/base/res/res_config.py b/openerp/addons/base/res/res_config.py index 655173a27a2..861931bed5c 100644 --- a/openerp/addons/base/res/res_config.py +++ b/openerp/addons/base/res/res_config.py @@ -46,10 +46,10 @@ class res_config_module_installation_mixin(object): to_install_missing_names.append(name) elif module.state == 'uninstalled': to_install_ids.append(module.id) - + result = None if to_install_ids: - ir_module.button_immediate_install(cr, uid, to_install_ids, context=context) - + result = ir_module.button_immediate_install(cr, uid, to_install_ids, context=context) + #FIXME: if result is not none, the corresponding todo will be skipped because it was just marked done if to_install_missing_names: return { 'type': 'ir.actions.client', @@ -57,7 +57,7 @@ class res_config_module_installation_mixin(object): 'params': {'modules': to_install_missing_names}, } - return None + return result class res_config_configurable(osv.osv_memory): ''' Base classes for new-style configuration items