openerp-dev-web team mailing list archive

["Anup\(OpenERP\)" <ach@xxxxxxxxxxx>]

Anup(OpenERP) has proposed merging lp:~openerp-dev/openobject-server/6.0-opw-partner_lang_size-ach into lp:openobject-server/6.0.

Requested reviews:
  Olivier Dony (OpenERP) (odo)
  Jay Vora (OpenERP) (jvo-openerp)

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-server/6.0-opw-partner_lang_size-ach/+merge/52041

Hello,

    
   This is an issue with Partner Language. Install the language which has the code's size greater than 5 characters For e.g. Serbian(Latin). Create a new partner and select the language Serbian(Latin) and this will raise the error. DataError: value too long for type character varying(5).


The solution fixes the problem.

Thanks.
-- 
https://code.launchpad.net/~openerp-dev/openobject-server/6.0-opw-partner_lang_size-ach/+merge/52041
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-server/6.0-opw-partner_lang_size-ach.

=== modified file 'bin/addons/base/res/partner/partner.py'
--- bin/addons/base/res/partner/partner.py	2011-01-17 15:17:57 +0000
+++ bin/addons/base/res/partner/partner.py	2011-03-03 11:09:09 +0000
@@ -116,7 +116,7 @@
         'parent_id': fields.many2one('res.partner','Parent Partner'),
         'child_ids': fields.one2many('res.partner', 'parent_id', 'Partner Ref.'),
         'ref': fields.char('Reference', size=64, select=1),
-        'lang': fields.selection(_lang_get, 'Language', size=5, help="If the selected language is loaded in the system, all documents related to this partner will be printed in this language. If not, it will be english."),
+        'lang': fields.selection(_lang_get, 'Language', help="If the selected language is loaded in the system, all documents related to this partner will be printed in this language. If not, it will be english."),
         'user_id': fields.many2one('res.users', 'Salesman', help='The internal user that is in charge of communicating with this partner if any.'),
         'vat': fields.char('VAT',size=32 ,help="Value Added Tax number. Check the box if the partner is subjected to the VAT. Used by the VAT legal statement."),
         'bank_ids': fields.one2many('res.partner.bank', 'partner_id', 'Banks'),

=== modified file 'bin/osv/orm.py'
--- bin/osv/orm.py	2011-02-16 14:21:28 +0000
+++ bin/osv/orm.py	2011-03-03 11:09:09 +0000
@@ -2525,6 +2525,8 @@
                                 ('numeric', 'float', get_pg_type(f)[1], '::'+get_pg_type(f)[1]),
                                 ('float8', 'float', get_pg_type(f)[1], '::'+get_pg_type(f)[1]),
                             ]
+                            if f_pg_type == 'varying' or f._table=='res_partner':
+                                print ":::filed",f._table,f.string
                             if f_pg_type == 'varchar' and f._type == 'char' and f_pg_size < f.size:
                                 cr.execute('ALTER TABLE "%s" RENAME COLUMN "%s" TO temp_change_size' % (self._table, k))
                                 cr.execute('ALTER TABLE "%s" ADD COLUMN "%s" VARCHAR(%d)' % (self._table, k, f.size))