From a1331c60adb58e009c97b35be7ddc6a768b899bf Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Nicolas=20=C3=89vrard?= Date: Fri, 8 Apr 2022 19:07:13 +0200 Subject: [PATCH] Send only the required fields when fetching a readonly view definition issue11134 review380081002 --- product.py | 6 +----- purchase.py | 10 +++------- 2 files changed, 4 insertions(+), 12 deletions(-) diff --git a/product.py b/product.py index a6e0424622..00f77f9df7 100644 --- a/product.py +++ b/product.py @@ -15,15 +15,13 @@ class PurchaseSecondaryMixin: 'product.uom', "Purchase Secondary UOM", domain=[ ('category', '!=', Eval('default_uom_category')), - ], - depends=['default_uom_category']) + ]) purchase_secondary_uom_factor = fields.Float( "Purchase Secondary UOM Factor", digits=uom_conversion_digits, states={ 'required': Bool(Eval('purchase_secondary_uom')), 'invisible': ~Eval('purchase_secondary_uom'), }, - depends=['purchase_secondary_uom'], help="The coefficient for the formula:\n" "1 (purchase unit) = coefficient (secondary unit)") purchase_secondary_uom_rate = fields.Float( @@ -32,7 +30,6 @@ class PurchaseSecondaryMixin: 'required': Bool(Eval('purchase_secondary_uom')), 'invisible': ~Eval('purchase_secondary_uom'), }, - depends=['purchase_secondary_uom'], help="The coefficient for the formula:\n" "coefficient (purchase unit) = 1 (secondary unit)") purchase_secondary_uom_category = fields.Function( @@ -126,7 +123,6 @@ class Template(PurchaseSecondaryMixin, metaclass=PoolMeta): cls.purchase_secondary_uom.states = { 'invisible': ~Eval('purchasable', False), } - cls.purchase_secondary_uom.depends.append('purchasable') class Product(metaclass=PoolMeta): diff --git a/purchase.py b/purchase.py index 3c6de92e7f..6b09bd24a3 100644 --- a/purchase.py +++ b/purchase.py @@ -16,8 +16,7 @@ class Line(metaclass=PoolMeta): 'invisible': ((Eval('type') != 'line') | ~Eval('secondary_unit')), 'readonly': Eval('purchase_state') != 'draft', - }, - depends=['type', 'secondary_unit', 'purchase_state']), + }), 'on_change_with_secondary_quantity', setter='set_secondary') secondary_unit = fields.Many2One( 'product.uom', "Secondary Unit", ondelete='RESTRICT', @@ -31,17 +30,14 @@ class Line(metaclass=PoolMeta): | (~Eval('secondary_uom_factor') & ~Eval('secondary_uom_rate'))), 'readonly': Eval('purchase_state') != 'draft', - }, - depends=['product_secondary_uom_category', 'type', - 'secondary_uom_factor', 'secondary_uom_rate', 'purchase_state']) + }) secondary_unit_price = fields.Function(Monetary( "Secondary Unit Price", currency='currency', digits=price_digits, states={ 'invisible': ((Eval('type') != 'line') | ~Eval('secondary_unit')), 'readonly': Eval('purchase_state') != 'draft', - }, - depends=['type', 'secondary_unit', 'purchase_state']), + }), 'on_change_with_secondary_unit_price', setter='set_secondary') secondary_uom_factor = fields.Float("Secondary UOM Factor")