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

[IMP] deprecated feature: added protection against read/write on non-existing columns. The ORM already throws warnings, no need to crash in addition :) . 

bzr revid: tde@openerp.com-20120809120926-ngl31imcggzz9wzt
This commit is contained in:
Thibault Delavallée 2012-08-09 14:09:26 +02:00
parent 90c695d141
commit 87625fd742
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
openerp/osv/orm.py
View File

@ -3601,7 +3601,7 @@ class BaseModel(object):
# Warn about deprecated fields now that fields_pre and fields_post are computed
for f in fields_pre + fields_post:
field_column = (f != self.CONCURRENCY_CHECK_FIELD and self._all_columns.get(f).column)
field_column = (f != self.CONCURRENCY_CHECK_FIELD and self._all_columns.get(f) and self._all_columns.get(f).column)
if field_column and field_column.deprecated:
_logger.warning('Field %s.%s is deprecated: %s', self._name, f, field_column.deprecated)
@ -3982,7 +3982,7 @@ class BaseModel(object):
direct = []
totranslate = context.get('lang', False) and (context['lang'] != 'en_US')
for field in vals:
field_column = (field != self.CONCURRENCY_CHECK_FIELD and self._all_columns.get(field).column)
field_column = (field != self.CONCURRENCY_CHECK_FIELD and self._all_columns.get(field) and self._all_columns.get(field).column)
if field_column and field_column.deprecated:
_logger.warning('Field %s.%s is deprecated: %s', self._name, field, field_column.deprecated)
if field in self._columns: