FIX set reoute_ids as readonly to avoid triggering inverse method and…

… changing the route_profile_id when adding force_route_profile_id
OCA · Feb 8, 2024 · d506d7b · d506d7b
1 parent 6724f41
commit d506d7b
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/product_route_profile/tests/test_product_route_profile.py b/product_route_profile/tests/test_product_route_profile.py
@@ -55,6 +55,7 @@ def test_2_force_route_profile(self):
         self.product.with_company(
             self.env.company
         ).force_route_profile_id = self.route_profile_2.id
+        self.assertEqual(self.product.route_profile_id, self.route_profile_1)
         self.assertEqual(
             self.product.with_company(self.env.company).route_ids,
             self.route_profile_2.route_ids,

diff --git a/product_route_profile/views/product_template.xml b/product_route_profile/views/product_template.xml
@@ -11,6 +11,12 @@
             <xpath expr="//field[@name='route_ids']/parent::div" position="attributes">
                 <attribute name="attrs">{'invisible': True}</attribute>
             </xpath>
+            <xpath expr="//field[@name='route_ids']" position="attributes">
+                <attribute name="readonly">1</attribute>
+            </xpath>
+            <xpath expr="//label[@for='route_ids']" position="attributes">
+                <attribute name="invisible">1</attribute>
+            </xpath>
             <xpath expr="//group[@name='operations']" position="inside">
                 <field
                     name="route_profile_id"