openerp-dev-web team mailing list archive

["Vo Minh Thu \(OpenERP\)" <vmt@xxxxxxxxxxx>]

Vo Minh Thu (OpenERP) has proposed merging lp:~openerp-dev/openobject-server/trunk-bug-716715-db-user-name-vmt into lp:openobject-server.

Requested reviews:
  OpenERP Core Team (openerp)
Related bugs:
  #716715 [trunk 3353] db_user parameter dont work
  https://bugs.launchpad.net/bugs/716715

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-server/trunk-bug-716715-db-user-name-vmt/+merge/52560
-- 
https://code.launchpad.net/~openerp-dev/openobject-server/trunk-bug-716715-db-user-name-vmt/+merge/52560
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-server/trunk-bug-716715-db-user-name-vmt.

=== modified file 'openerp/sql_db.py'
--- openerp/sql_db.py	2011-01-18 19:26:37 +0000
+++ openerp/sql_db.py	2011-03-08 14:08:55 +0000
@@ -358,14 +358,13 @@
         except Exception:
             return False
 
-
-_dsn = ''
-for p in ('host', 'port', 'user', 'password'):
-    cfg = tools.config['db_' + p]
-    if cfg:
-        _dsn += '%s=%s ' % (p, cfg)
-
 def dsn(db_name):
+    _dsn = ''
+    for p in ('host', 'port', 'user', 'password'):
+        cfg = tools.config['db_' + p]
+        if cfg:
+            _dsn += '%s=%s ' % (p, cfg)
+
     return '%sdbname=%s' % (_dsn, db_name)
 
 def dsn_are_equals(first, second):