From f4fef55e2b04ce44bb9519667a6aa82dc1418969 Mon Sep 17 00:00:00 2001 From: Xavier Morel Date: Mon, 1 Aug 2011 10:54:57 +0200 Subject: [PATCH] [IMP] don't manually manage extracted indexes when enumerate() can do it for us bzr revid: xmo@openerp.com-20110801085457-bw0x6af0xwqmf7e5 --- openerp/modules/loading.py | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/openerp/modules/loading.py b/openerp/modules/loading.py index 6bb45e77598..b24f4888424 100644 --- a/openerp/modules/loading.py +++ b/openerp/modules/loading.py @@ -153,13 +153,12 @@ def load_module_graph(cr, graph, status=None, perform_checks=True, skip_modules= processed_modules = [] loaded_modules = [] - statusi = 0 pool = pooler.get_pool(cr.dbname) migrations = openerp.modules.migration.MigrationManager(cr, graph) logger.notifyChannel('init', netsvc.LOG_DEBUG, 'loading %d packages..' % len(graph)) # register, instantiate and initialize models for each modules - for package in graph: + for index, package in enumerate(graph): module_name = package.name module_id = package.id @@ -174,7 +173,7 @@ def load_module_graph(cr, graph, status=None, perform_checks=True, skip_modules= if hasattr(package, 'init') or hasattr(package, 'update') or package.state in ('to install', 'to upgrade'): init_module_models(cr, package.name, models) - status['progress'] = float(statusi) / len(graph) + status['progress'] = float(index) / len(graph) # Can't put this line out of the loop: ir.module.module will be # registered by init_module_models() above. @@ -197,7 +196,7 @@ def load_module_graph(cr, graph, status=None, perform_checks=True, skip_modules= load_update_xml(cr, module_name, idref, mode) load_data(cr, module_name, idref, mode) if hasattr(package, 'demo') or (package.dbdemo and package.state != 'installed'): - status['progress'] = (float(statusi)+0.75) / len(graph) + status['progress'] = (index + 0.75) / len(graph) load_demo_xml(cr, module_name, idref, mode) load_demo(cr, module_name, idref, mode) cr.execute('update ir_module_module set demo=%s where id=%s', (True, module_id)) @@ -224,7 +223,6 @@ def load_module_graph(cr, graph, status=None, perform_checks=True, skip_modules= delattr(package, kind) cr.commit() - statusi += 1 cr.commit()