【ODOO】来了解一下browse方法

错误抛出

前端时间在生产环境上线了一个定时任务，目的是采集客户信息，以用来分析客户数据。2022-05-30 系统告警出现了错误。下面是错误信息：

2022-05-30 14:40:46,332 76 ERROR momo_prod odoo.addons.base.models.ir_cron: Call from cron Partner信息采集定时任务 for server action #417 failed in Job #17
Traceback (most recent call last):File '/opt/momo/odoo/api.py', line 793, in getreturn field_cache[record._ids[0]]
KeyError: 12
During handling of the above exception, another exception occurred:
Traceback (most recent call last):File '/opt/momo/odoo/fields.py', line 972, in __get__value = env.cache.get(record, self)File '/opt/momo/odoo/api.py', line 796, in getraise CacheMiss(record, field)
odoo.exceptions.CacheMiss: 'res.partner(75432,).group_id'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):File '/opt/momo/odoo/addons/base/models/ir_cron.py', line 110, in _callbackself.env['ir.actions.server'].browse(server_action_id).run()File '/opt/momo/odoo/addons/base/models/ir_actions.py', line 632, in runres = runner(run_self, eval_context=eval_context)File '/opt/momo/odoo/addons/base/models/ir_actions.py', line 501, in _run_action_code_multisafe_eval(self.code.strip(), eval_context, mode='exec', nocopy=True)  # nocopy allows to return 'action'File '/opt/momo/odoo/tools/safe_eval.py', line 330, in safe_evalreturn unsafe_eval(c, globals_dict, locals_dict)File '', line 1, in <module>File '/opt/momo/addons/momo_cron/models/cron_partner.py', line 199, in _action_cron_do_collectioncollection_item.handle_record()File '/opt/momo/addons/momo_cron/models/cron_partner.py', line 119, in handle_collectionparent_name = partner_res.group_id.nameFile '/opt/momo/odoo/fields.py', line 2485, in __get__return super().__get__(records, owner)File '/opt/sps/odoo/fields.py', line 1004, in __get___('(Record: %s, User: %s)') % (record, env.uid),
odoo.exceptions.MissingError: Record does not exist or has been deleted.
(Record: res.partner(75432,), User: 1)

简单分析

定位到报错代码位置（line 119）：

parent_name = partner_res.group_id.name

再看最后的错误信息：

odoo.exceptions.MissingError: Record does not exist or has been deleted.
(Record: res.partner(75432,), User: 1)

结合可知，错误并不是因为字段 group_id 可能没有值导致的问题，而是因为记录 partner_res 不存在或已删除导致的。

但这不对呀，我是判断了 partner_res 是否存在的：

    partner_res = res_partner_obj.browse(self.partner_id)if partner_res:parent_name = partner_res.group_id.nameelse:parent_name = ""

如果 partner_res 不存在，就不应该走 partner_res.group_id.name ，更不应该报错。打个断点跟一下。

Debug

打上断点跟了一下不咋滴，这一跟简直跟出了一条惊天大案！

问题定位到了这句看起来非常普通的代码上：

partner_res = res_partner_obj.browse(self.partner_id)

断点过程发现 self.partner_id 是有值的，但数据库中确实不存在 id 为现 75432 的记录。怀疑问题出现在browse方法上，browse方法可能压根儿没走数据库。为了验证这个问题，我将问题代码简单抽出来复现一下。

问题复现

TIP: 已知在数据库中，表res_partner中不存在 id 为 999999 的记录，通过使用 browse 查找记录。

def match_record_by_browse(self):"""演示 browse 查询记录集:return:"""browse_result = self.env["res.partner"].browse(999999)print("browse_result", browse_result)if browse_result:print("browse_result yes")else:print("browse_result no")

终端输出

browse_result res.partner(999999,)
browse_result yes

可以看到数据库中，没有 id 为 999999 的记录，依然输出了 res.partner(999999,) 。

browse() API

查看下browse源码:

    ## Instance creation## An instance represents an ordered collection of records in a given# execution environment. The instance object refers to the environment, and# the records themselves are represented by their cache dictionary. The 'id'# of each record is found in its corresponding cache dictionary.## This design has the following advantages:#  - cache access is direct and thus fast;#  - one can consider records without an 'id' (see new records);#  - the global cache is only an index to "resolve" a record 'id'.#
@classmethoddef _browse(cls, env, ids, prefetch_ids):""" Create a recordset instance.
:param env: an environment:param ids: a tuple of record ids:param prefetch_ids: a collection of record ids (for prefetching)"""records = object.__new__(cls)records.env = envrecords._ids = idsrecords._prefetch_ids = prefetch_idsreturn records
def browse(self, ids=None):""" browse([ids]) -> records
Returns a recordset for the ids provided as parameter in the currentenvironment.
.. code-block:: python
self.browse([7, 18, 12])res.partner(7, 18, 12)
:param ids: id(s):type ids: int or list(int) or None:return: recordset"""if not ids:ids = ()elif ids.__class__ in IdType:ids = (ids,)else:ids = tuple(ids)return self._browse(self.env, ids, ids)

王德发！browse方法真的没有访问数据库，只是将提供的 id 包装成了当前环境的实例。目的是为了快速读取数据。

实例表示给定执行环境中的有序记录集合。实例对象引用环境，而记录本身由它们的缓存字典表示。每条记录的“id”都可以在其对应的缓存字典中找到。

这种设计具有以下优点：

缓存访问是直接的，因此速度很快。

可以考虑没有“id”的记录。

全局缓存只是“解析”记录“id”的索引。

这么来看，browse方法并不关心传入的 id 在相关表中是不是存在。它仅仅是为了将给定的 id 包装成ORM可用的实例。

问题解决

这是一个browse方法使用的问题。将browse方法替换为search方法，通过search方法访问数据库，查找是否真正存在即可解决此问题。

partner_res = res_partner_obj.search([("id", "=", self.partner_id)])