From e575211116c099e24ae294e0e5c08f617ae3a9fb Mon Sep 17 00:00:00 2001 From: Xavier Morel Date: Fri, 9 Sep 2011 18:29:29 +0200 Subject: [PATCH] [IMP] add a pair of assertions on the parameter to VARCHAR, just in case bzr revid: xmo@openerp.com-20110909162929-w1l328oh14ccn7f8 --- openerp/osv/orm.py | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/openerp/osv/orm.py b/openerp/osv/orm.py index 6f794469529..9492e3d812b 100644 --- a/openerp/osv/orm.py +++ b/openerp/osv/orm.py @@ -485,6 +485,12 @@ def pg_varchar(size=0): :rtype: str """ if size: + if not isinstance(size, int): + raise TypeError("VARCHAR parameter should be an int, got %s" + % type(size)) + if size < 0: + raise ValueError("VARCHAR parameter can not be negative, got %d" + % size) return 'VARCHAR(%d)' % size return 'VARCHAR'