openerp-community team mailing list archive

Thread
Date

Re: lp:~openbmsjsc/openobject-addons/addons-extra-trunk-l10n_vn into lp:openobject-addons/extra-trunk

To: mp+64294@xxxxxxxxxxxxxxxxxx
From: Gustavo Adrian Marino <gamarino@xxxxxxxxx>
Date: Fri, 03 Feb 2012 15:12:22 -0000
In-reply-to: <20120203140832.14814.16766.launchpad@ackee.canonical.com>
Reply-to: mp+64294@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

I wish to add something that is not technically wrong, but it could become a potential problem.

You are defining your user account.types on the account.xxxx namespace. I believe that could be a potential cause of problems on multi country databases (that is a multicompany installation with each company using a different template). If all localization templates uses the account.xxx namespace to define their own types, name crashing could become a real problem.

I suggest to keep account.types as locally defined items (that means using the module namespace) in order to prevent the a.m. danger
-- 
https://code.launchpad.net/~openbmsjsc/openobject-addons/addons-extra-trunk-l10n_vn/+merge/64294
Your team OpenERP Community is requested to review the proposed merge of lp:~openbmsjsc/openobject-addons/addons-extra-trunk-l10n_vn into lp:openobject-addons/extra-trunk.