changeset e1cf976199ab in modules/incoterm:default
details: https://hg.tryton.org/modules/incoterm?cmd=changeset&node=e1cf976199ab
description:
        Send only the required fields when fetching a readonly view definition

        issue11134
        review380081002
diffstat:

 common.py |  14 +++++---------
 party.py  |   4 ++--
 2 files changed, 7 insertions(+), 11 deletions(-)

diffs (75 lines):

diff -r 2ce3e6a1e86d -r e1cf976199ab common.py
--- a/common.py Wed Apr 06 23:37:42 2022 +0200
+++ b/common.py Fri Apr 08 19:07:13 2022 +0200
@@ -26,11 +26,10 @@
     @classmethod
     def __setup__(cls):
         super().__setup__()
-        readonly, depends = cls._incoterm_readonly_state()
+        readonly = cls._incoterm_readonly_state()
         cls.incoterm.states = {
             'readonly': readonly,
             }
-        cls.incoterm.depends = depends
 
         cls.incoterm_location.states = {
             'readonly': readonly,
@@ -40,16 +39,15 @@
         cls.incoterm_location.search_context = {
             'related_party': related_party,
             }
-        cls.incoterm_location.depends = (
-            depends + ['incoterm'] + related_party_depends)
+        cls.incoterm_location.depends = {'incoterm'} | related_party_depends
 
     @classmethod
     def _incoterm_readonly_state(cls):
-        return ~Eval('state').in_(['draft']), ['state']
+        return ~Eval('state').in_(['draft'])
 
     @classmethod
     def _incoterm_related_party(cls):
-        return Eval('party'), ['party']
+        return Eval('party'), {'party'}
 
     @property
     def incoterm_name(self):
@@ -73,16 +71,14 @@
     @classmethod
     def __setup__(cls):
         super().__setup__()
-        readonly, depends = cls._incoterm_readonly_state()
+        readonly = cls._incoterm_readonly_state()
         cls.incoterm.domain = [
             If(~readonly,
                 ('id', 'in', Eval('available_incoterms', [])),
                 ()),
             ]
-        cls.incoterm.depends += depends + ['available_incoterms']
         cls.incoterm_location.states['required'] = (
             Eval('incoterm_location_required', False))
-        cls.incoterm_location.depends.append('incoterm_location_required')
 
     @fields.depends('company', 'party', methods=['_get_incoterm_pattern'])
     def on_change_with_available_incoterms(self, name=None):
diff -r 2ce3e6a1e86d -r e1cf976199ab party.py
--- a/party.py  Wed Apr 06 23:37:42 2022 +0200
+++ b/party.py  Fri Apr 08 19:07:13 2022 +0200
@@ -94,7 +94,7 @@
         context={
             'company': Eval('company', -1),
             },
-        depends=['company'])
+        depends={'company'})
     company = fields.Many2One('company.company', "Company")
     type = fields.Selection([
             ('purchase', "Purchase"),
@@ -111,7 +111,7 @@
             ('party.distance', 'ASC NULLS LAST'),
             ('id', None),
             ],
-        depends=['party'])
+        depends={'party'})
 
     @classmethod
     def __setup__(cls):

Reply via email to