Odoo计算字段日期比较出错？教你解决默认值陷阱！

2024-07-07 00:29:11

Odoo 计算字段中日期比较难题：@api.depends 与日期类型

在 Odoo 开发中，我们经常需要根据关联字段的变化自动计算某个字段的值。@api.depends 装饰器为我们提供了一种便捷的实现方式。然而，当涉及日期类型的比较时，即使代码逻辑清晰，也可能会遇到一些棘手的问题。本文将深入探讨这一问题，并提供清晰易懂的解决方案，帮助您构建更加健壮的 Odoo 应用程序。

场景再现：采购订单与交货日期

假设您正在使用 Odoo 16 开发一个采购订单模块。您希望在 operations.purchaseorder 模型中添加一个名为 full_delivery_date 的计算字段，用于存储所有关联订单行 (operations.purchaseorderline) 中最晚的交货日期。

您编写了以下代码：

operations.purchaseorder 模型:

full_delivery_date = fields.Date(string="Full Delivery Date (calculated)", compute='_compute_deliver_date', default=lambda self: fields.Date.today())

order_line_ids = fields.One2many('operations.purchaseorderline','purchase_order_id',string="Order Lines")

@api.depends('order_line_ids')
def _compute_deliver_date(self):
    for record in self:
        if record.order_line_ids:
            for order_line in record.order_line_ids:
                if order_line.delivery > record.full_delivery_date:
                    record.full_delivery_date = order_line.delivery

operations.purchaseorderline 模型:

purchase_order_id = fields.Many2one('operations.purchaseorder',string='Purchase Order')
delivery = fields.Date("Promised Delivery Date")

这段代码看起来逻辑清晰，但在实际运行时却抛出错误：TypeError: '>' not supported between instances of 'datetime.date' and 'bool'。究竟是什么原因导致的呢？

深入分析：默认值的陷阱

问题出在 if order_line.delivery > record.full_delivery_date: 这行代码。乍看之下，这行代码只是简单地比较两个日期类型字段的大小。然而，由于 record.full_delivery_date 字段的默认值为 lambda self: fields.Date.today(), 在首次计算 _compute_deliver_date 函数时， record.full_delivery_date 的值实际上是一个函数对象，而非日期类型。将 order_line.delivery 与之进行比较，自然会引发类型错误。

解决方案：确保日期类型一致性

解决这个问题的关键在于确保在进行日期比较之前，record.full_delivery_date 字段存储的是一个有效的日期值。我们可以通过以下两种方式实现：

方法一：修改默认值

将 full_delivery_date 字段的默认值修改为一个固定的日期，例如 fields.Date.min 或 False：

full_delivery_date = fields.Date(string="Full Delivery Date (calculated)", compute='_compute_deliver_date', default=fields.Date.min)

使用 fields.Date.min 可以确保初始值为一个有效的日期类型，避免与订单行的交货日期进行比较时出现类型错误。

方法二：添加判断条件

在进行日期比较之前，先判断 record.full_delivery_date 是否为有效日期：

@api.depends('order_line_ids')
def _compute_deliver_date(self):
    for record in self:
        if record.order_line_ids:
            for order_line in record.order_line_ids:
                if isinstance(record.full_delivery_date, date) and order_line.delivery > record.full_delivery_date:
                    record.full_delivery_date = order_line.delivery

通过 isinstance(record.full_delivery_date, date) 判断，可以确保只有在 record.full_delivery_date 为有效日期时才进行比较，避免了潜在的类型错误。