From 9732e2e66c4f46fc5d954280f8367ee34e50ff54 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Nicolas=20=C3=89vrard?= <nicoe@b2ck.com>
Date: Fri, 8 Apr 2022 19:07:13 +0200
Subject: [PATCH] Send only the required fields when fetching a readonly view
 definition

issue11134
review380081002
---
 production.py |  6 ++----
 work.py       | 24 ++++++++----------------
 2 files changed, 10 insertions(+), 20 deletions(-)

diff --git a/production.py b/production.py
index 56bf3ea15f..d849dd950f 100644
--- a/production.py
+++ b/production.py
@@ -25,8 +25,7 @@ class Production(metaclass=PoolMeta):
         domain=[
             ('company', '=', Eval('company', -1)),
             ('warehouse', '=', Eval('warehouse', -1)),
-            ],
-        depends=['routing', 'company', 'warehouse', 'state'])
+            ])
     works = fields.One2Many('production.work', 'production', 'Works',
         states={
             'readonly': Eval('state').in_(
@@ -34,8 +33,7 @@ class Production(metaclass=PoolMeta):
             },
         domain=[
             ('company', '=', Eval('company', -1)),
-            ],
-        depends=['state', 'company'])
+            ])
 
     def get_cost(self, name):
         pool = Pool()
diff --git a/work.py b/work.py
index b1c2ab0a9e..8c2fdeffd3 100644
--- a/work.py
+++ b/work.py
@@ -32,20 +32,17 @@ class WorkCenter(DeactivableMixin, tree(separator=' / '), ModelSQL, ModelView):
         domain=[
             ('company', '=', Eval('company', -1)),
             ('warehouse', '=', Eval('warehouse', -1)),
-            ],
-        depends=['company', 'warehouse'])
+            ])
     children = fields.One2Many('production.work.center', 'parent', 'Children',
         domain=[
             ('company', '=', Eval('company', -1)),
             ('warehouse', '=', Eval('warehouse', -1)),
-            ],
-        depends=['company', 'warehouse'])
+            ])
     category = fields.Many2One('production.work.center.category', 'Category')
     cost_price = fields.Numeric('Cost Price', digits=price_digits,
         states={
             'required': Bool(Eval('cost_method')),
-            },
-        depends=['cost_method'])
+            })
     cost_method = fields.Selection([
             ('', ''),
             ('cycle', 'Per Cycle'),
@@ -53,8 +50,7 @@ class WorkCenter(DeactivableMixin, tree(separator=' / '), ModelSQL, ModelView):
             ], 'Cost Method',
         states={
             'required': Bool(Eval('cost_price')),
-            },
-        depends=['cost_price'])
+            })
     company = fields.Many2One('company.company', 'Company', required=True,
         select=True)
     warehouse = fields.Many2One('stock.location', 'Warehouse', required=True,
@@ -108,8 +104,7 @@ class Work(sequence_ordered(), ModelSQL, ModelView):
         select=True, ondelete='CASCADE',
         domain=[
             ('company', '=', Eval('company', -1)),
-            ],
-        depends=['company'])
+            ])
     work_center_category = fields.Function(fields.Many2One(
             'production.work.center.category', 'Work Center Category'),
         'on_change_with_work_center_category')
@@ -123,13 +118,11 @@ class Work(sequence_ordered(), ModelSQL, ModelView):
             ],
         states={
             'required': ~Eval('state').in_(['request', 'draft']),
-            },
-        depends=['work_center_category', 'company', 'warehouse', 'state'])
+            })
     cycles = fields.One2Many('production.work.cycle', 'work', 'Cycles',
         states={
             'readonly': Eval('state').in_(['request', 'done']),
-            },
-        depends=['state'])
+            })
     active_cycles = fields.One2Many(
         'production.work.cycle', 'work', "Active Cycles",
         readonly=True,
@@ -283,8 +276,7 @@ class WorkCycle(Workflow, ModelSQL, ModelView):
         states={
             'required': Eval('state') == 'done',
             'readonly': Eval('state').in_(['done', 'draft', 'cancelled']),
-            },
-        depends=['state'])
+            })
     cost = fields.Numeric('Cost', digits=price_digits, readonly=True)
     state = fields.Selection([
             ('draft', 'Draft'),