diff options
| author | Raphael Collet <rco@openerp.com> | 2014-11-03 16:00:50 +0100 |
|---|---|---|
| committer | Raphael Collet <rco@openerp.com> | 2014-11-04 13:47:57 +0100 |
| commit | f2e4a10e1a58c932f9a989e642771f5ae75d756a (patch) | |
| tree | b5d933afe79088ecf5b8a5bc9eae957fb8898b0b /addons/mass_mailing/models/mail_thread.py | |
| parent | 0f52e22906a969a8ca3b2c38acddf5bf8a6386be (diff) | |
[IMP] use model._fields instead of model._all_columns to cover all fields
The old-api model._all_columns contains information about model._columns and
inherited columns. This dictionary is missing new-api computed non-stored
fields, and the new field objects provide a more readable api...
This commit contains the following changes:
- adapt several methods of BaseModel to use fields instead of columns and
_all_columns
- copy all semantic-free attributes of related fields from their source
- add attribute 'group_operator' on integer and float fields
- base, base_action_rule, crm, edi, hr, mail, mass_mailing, pad,
payment_acquirer, share, website, website_crm, website_mail: simply use
_fields instead of _all_columns
- base, decimal_precision, website: adapt qweb rendering methods to use fields
instead of columns
Diffstat (limited to 'addons/mass_mailing/models/mail_thread.py')
| -rw-r--r-- | addons/mass_mailing/models/mail_thread.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/addons/mass_mailing/models/mail_thread.py b/addons/mass_mailing/models/mail_thread.py index 7bfce705043..e4732527f2d 100644 --- a/addons/mass_mailing/models/mail_thread.py +++ b/addons/mass_mailing/models/mail_thread.py @@ -77,7 +77,7 @@ class MailThread(osv.AbstractModel): Mail Returned to Sender) is received for an existing thread. The default behavior is to check is an integer ``message_bounce`` column exists. If it is the case, its content is incremented. """ - if self._all_columns.get('message_bounce'): + if 'message_bounce' in self._fields: for obj in self.browse(cr, uid, ids, context=context): self.write(cr, uid, [obj.id], {'message_bounce': obj.message_bounce + 1}, context=context) |
