changeset aac31f00ca6e in modules/sale_shipment_cost:default
details:
https://hg.tryton.org/modules/sale_shipment_cost?cmd=changeset&node=aac31f00ca6e
description:
Send only the required fields when fetching a readonly view definition
issue11134
review380081002
diffstat:
sale.py | 9 ++++-----
stock.py | 12 ++++--------
2 files changed, 8 insertions(+), 13 deletions(-)
diffs (82 lines):
diff -r 297e68b5e987 -r aac31f00ca6e sale.py
--- a/sale.py Wed Apr 06 23:37:44 2022 +0200
+++ b/sale.py Fri Apr 08 19:07:14 2022 +0200
@@ -87,7 +87,7 @@
context={
'company': Eval('company', -1),
},
- depends=['state', 'available_carriers', 'company'])
+ depends={'company'})
available_carriers = fields.Function(
fields.Many2Many('carrier', None, None, 'Available Carriers'),
'on_change_with_available_carriers')
@@ -97,7 +97,7 @@
('shipment', 'On Shipment'),
], 'Shipment Cost Method', states={
'readonly': Eval('state') != 'draft',
- }, depends=['state'])
+ })
@classmethod
def __register__(cls, module):
@@ -326,7 +326,7 @@
'sale_shipment_cost.msg_sale_shipment_cost_unique'),
]
# shipment_cost is needed to compute the unit_price
- cls.unit_price.depends.append('shipment_cost')
+ cls.unit_price.depends.add('shipment_cost')
@fields.depends('shipment_cost', 'unit_price')
def compute_unit_price(self):
@@ -404,8 +404,7 @@
"Amount with Shipment Cost Included",
states={
'invisible': ~Eval('amount'),
- },
- depends=['amount'])
+ })
@classmethod
def default_amount_shipment_cost_included(cls):
diff -r 297e68b5e987 -r aac31f00ca6e stock.py
--- a/stock.py Wed Apr 06 23:37:44 2022 +0200
+++ b/stock.py Fri Apr 08 19:07:14 2022 +0200
@@ -19,8 +19,7 @@
'invisible': (
Eval('cost_edit', False)
| (Eval('cost_method') != 'shipment')),
- },
- depends=['cost_edit', 'cost_method']),
+ }),
'on_change_with_cost_sale_currency_used')
cost_sale_currency = fields.Many2One(
'currency.currency', "Cost Sale Currency",
@@ -30,16 +29,14 @@
| (Eval('cost_method') != 'shipment')),
'required': Bool(Eval('cost_sale')),
'readonly': Eval('state').in_(['done', 'cancelled']),
- },
- depends=['cost_method', 'cost_sale', 'state'])
+ })
cost_sale_used = fields.Function(fields.Numeric(
"Cost Sale", digits=price_digits,
states={
'invisible': (
Eval('cost_edit', False)
| (Eval('cost_method') != 'shipment')),
- },
- depends=['cost_edit', 'cost_method']),
+ }),
'on_change_with_cost_sale_used')
cost_sale = fields.Numeric(
"Cost Sale", digits=price_digits,
@@ -48,8 +45,7 @@
~Eval('cost_edit', False)
| (Eval('cost_method') != 'shipment')),
'readonly': Eval('state').in_(['done', 'cancelled']),
- },
- depends=['cost_method', 'state'])
+ })
cost_invoice_line = fields.Many2One('account.invoice.line',
'Cost Invoice Line', readonly=True)