[IMP] registry: avoid every direct access registry.db, and rename attribute as registry._db

bzr revid: rco@openerp.com-20140409133515-x2ywcm0d2qkdq3hq
This commit is contained in:
Raphael Collet 2014-04-09 15:35:15 +02:00
parent dda229273d
commit 980f6fc4b1
3 changed files with 8 additions and 11 deletions

View File

@ -21,15 +21,12 @@ class test_cr_execute(unittest2.TestCase):
""" """
Try to use iterable but non-list or int params in query parameters. Try to use iterable but non-list or int params in query parameters.
""" """
cr = registry().cursor() with registry().cursor() as cr:
try:
with self.assertRaises(ValueError): with self.assertRaises(ValueError):
cr.execute("SELECT id FROM res_users WHERE login=%s", 'admin') cr.execute("SELECT id FROM res_users WHERE login=%s", 'admin')
with self.assertRaises(ValueError): with self.assertRaises(ValueError):
cr.execute("SELECT id FROM res_users WHERE id=%s", 1) cr.execute("SELECT id FROM res_users WHERE id=%s", 1)
with self.assertRaises(ValueError): with self.assertRaises(ValueError):
cr.execute("SELECT id FROM res_users WHERE id=%s", '1') cr.execute("SELECT id FROM res_users WHERE id=%s", '1')
finally:
cr.close()
# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4: # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:

View File

@ -58,7 +58,7 @@ class Registry(Mapping):
self._init_modules = set() self._init_modules = set()
self.db_name = db_name self.db_name = db_name
self.db = openerp.sql_db.db_connect(db_name) self._db = openerp.sql_db.db_connect(db_name)
# special cursor for test mode; None means "normal" mode # special cursor for test mode; None means "normal" mode
self.test_cr = None self.test_cr = None
@ -79,7 +79,7 @@ class Registry(Mapping):
self._any_cache_cleared = False self._any_cache_cleared = False
cr = self.cursor() cr = self.cursor()
has_unaccent = openerp.modules.db.has_unaccent(cr) has_unaccent = openerp.modules._db.has_unaccent(cr)
if openerp.tools.config['unaccent'] and not has_unaccent: if openerp.tools.config['unaccent'] and not has_unaccent:
_logger.warning("The option --unaccent was given but no unaccent() function was found in database.") _logger.warning("The option --unaccent was given but no unaccent() function was found in database.")
self.has_unaccent = openerp.tools.config['unaccent'] and has_unaccent self.has_unaccent = openerp.tools.config['unaccent'] and has_unaccent
@ -193,7 +193,7 @@ class Registry(Mapping):
def enter_test_mode(self): def enter_test_mode(self):
""" Enter the 'test' mode, where one cursor serves several requests. """ """ Enter the 'test' mode, where one cursor serves several requests. """
assert self.test_cr is None assert self.test_cr is None
self.test_cr = self.db.test_cursor() self.test_cr = self._db.test_cursor()
RegistryManager.enter_test_mode() RegistryManager.enter_test_mode()
def leave_test_mode(self): def leave_test_mode(self):
@ -214,7 +214,7 @@ class Registry(Mapping):
# cursor itself in its method close(). # cursor itself in its method close().
self.test_cr.acquire() self.test_cr.acquire()
return self.test_cr return self.test_cr
return self.db.cursor() return self._db.cursor()
class DummyRLock(object): class DummyRLock(object):
""" Dummy reentrant lock, to be used while running rpc and js tests """ """ Dummy reentrant lock, to be used while running rpc and js tests """
@ -295,7 +295,7 @@ class RegistryManager(object):
registry.base_registry_signaling_sequence = seq_registry registry.base_registry_signaling_sequence = seq_registry
registry.base_cache_signaling_sequence = seq_cache registry.base_cache_signaling_sequence = seq_cache
# This should be a method on Registry # This should be a method on Registry
openerp.modules.load_modules(registry.db, force_demo, status, update_module) openerp.modules.load_modules(registry._db, force_demo, status, update_module)
except Exception: except Exception:
del cls.registries[db_name] del cls.registries[db_name]
raise raise

View File

@ -36,7 +36,7 @@ def get_db_and_pool(db_name, force_demo=False, status=None, update_module=False)
assert openerp.conf.deprecation.openerp_pooler assert openerp.conf.deprecation.openerp_pooler
_logger.warning('openerp.pooler.get_db_and_pool() is deprecated.') _logger.warning('openerp.pooler.get_db_and_pool() is deprecated.')
registry = RegistryManager.get(db_name, force_demo, status, update_module) registry = RegistryManager.get(db_name, force_demo, status, update_module)
return registry.db, registry return registry._db, registry
def restart_pool(db_name, force_demo=False, status=None, update_module=False): def restart_pool(db_name, force_demo=False, status=None, update_module=False):
@ -44,7 +44,7 @@ def restart_pool(db_name, force_demo=False, status=None, update_module=False):
_logger.warning('openerp.pooler.restart_pool() is deprecated.') _logger.warning('openerp.pooler.restart_pool() is deprecated.')
assert openerp.conf.deprecation.openerp_pooler assert openerp.conf.deprecation.openerp_pooler
registry = RegistryManager.new(db_name, force_demo, status, update_module) registry = RegistryManager.new(db_name, force_demo, status, update_module)
return registry.db, registry return registry._db, registry
def get_db(db_name): def get_db(db_name):
"""Return a database connection. The corresponding registry is initialized.""" """Return a database connection. The corresponding registry is initialized."""