odoo
/
odoo
2
0
Fork 0
Code Pull Requests Releases Activity

[iMP] error message simplified for required values 

bzr revid: fp@openerp.com-20130224135248-wdxg8d3fi79lp88z
This commit is contained in:
Fabien Pinckaers 2013-02-24 14:52:48 +01:00
parent fb00c370b2 8f91937bab
commit b831c5aa15
2 changed files with 3 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
openerp/osv/orm.py
View File

@ -5353,8 +5353,7 @@ def convert_pgerror_23502(model, fields, info, e):
message = _(u"Missing required value for the field '%s'.") % field_name
field = fields.get(field_name)
if field:
message = _(u"%s This might be '%s' in the current model, or a field "
u"of the same name in an o2m.") % (message, field['string'])
message = _(u"Missing required value for the field '%s' (%s)") % (field['string'], field_name)
return {
'message': message,
'field': field_name,

8
openerp/tests/addons/test_impex/tests/test_load.py
View File

@ -393,18 +393,14 @@ class test_required_string_field(ImporterCase):
def test_empty(self):
result = self.import_(['value'], [[]])
self.assertEqual(result['messages'], [message(
u"Missing required value for the field 'value'. This might be "
u"'unknown' in the current model, or a field of the same name in "
u"an o2m.")])
u"Missing required value for the field 'value' (unknown)")])
self.assertIs(result['ids'], False)
@mute_logger('openerp.sql_db', 'openerp.osv.orm')
def test_not_provided(self):
result = self.import_(['const'], [['12']])
self.assertEqual(result['messages'], [message(
u"Missing required value for the field 'value'. This might be "
u"'unknown' in the current model, or a field of the same name in "
u"an o2m.")])
u"Missing required value for the field 'unknown'")])
self.assertIs(result['ids'], False)
class test_text(ImporterCase):