From 7787e91d68ad7b71cb756d30cbcd66a8ac8d70e8 Mon Sep 17 00:00:00 2001 From: Denis Vermylen Date: Wed, 5 Apr 2017 11:24:02 +0200 Subject: [PATCH] [FIX] models: don't call name_get on falsy records The call of the name_get on records with falsy ids was unnecessary as it wasn't used afterwards and could lead to errors if a name_get doesn't handle falsy records. solves https://github.com/odoo/odoo/pull/16244 --- odoo/models.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/odoo/models.py b/odoo/models.py index 848e9cc9b69..c6e265e4f86 100644 --- a/odoo/models.py +++ b/odoo/models.py @@ -2043,7 +2043,7 @@ class BaseModel(object): def _read_group_resolve_many2one_fields(self, data, fields): many2onefields = {field['field'] for field in fields if field['type'] == 'many2one'} for field in many2onefields: - ids_set = {d[field] for d in data} + ids_set = {d[field] for d in data if d[field]} m2o_records = self.env[self._fields[field].comodel_name].browse(ids_set) data_dict = dict(m2o_records.name_get()) for d in data: