From c54a44c70d1bf4e7eed87060efdee3b553743963 Mon Sep 17 00:00:00 2001
From: Raimon Esteve <raimonesteve@gmail.com>
Date: Mon, 21 Mar 2022 16:01:38 +0100
Subject: [PATCH] Check rule only if _check_access is set and enforce companies
 rule issue4080

---
 locale/ca.po                       | 8 ++++----
 locale/es.po                       | 8 ++++----
 operation.xml                      | 8 ++++----
 tests/test_production_operation.py | 3 ++-
 4 files changed, 14 insertions(+), 13 deletions(-)

diff --git a/locale/ca.po b/locale/ca.po
index 8fe5c66..1283487 100644
--- a/locale/ca.po
+++ b/locale/ca.po
@@ -364,12 +364,12 @@ msgid "Wait"
 msgstr "En espera"
 
 msgctxt "model:ir.rule.group,name:rule_group_production_operation"
-msgid "Rule Group Product Operation"
-msgstr "Grup de regles d'operació de producció"
+msgid "User in companies"
+msgstr "Usuari a les empreses"
 
 msgctxt "model:ir.rule.group,name:rule_group_production_operation_tracking"
-msgid "Rule Group Production Operation Tracking"
-msgstr "Grup de regles de registre d'operació de producció"
+msgid "User in companies"
+msgstr "Usuari a les empreses"
 
 msgctxt "model:ir.ui.menu,name:"
 msgid "Operation Types"
diff --git a/locale/es.po b/locale/es.po
index 7bcd9c1..71c173e 100644
--- a/locale/es.po
+++ b/locale/es.po
@@ -365,12 +365,12 @@ msgid "Wait"
 msgstr "Esperando"
 
 msgctxt "model:ir.rule.group,name:rule_group_production_operation"
-msgid "Rule Group Product Operation"
-msgstr "Grupo de reglas de operación de producción"
+msgid "User in companies"
+msgstr "Usuario en las empresas"
 
 msgctxt "model:ir.rule.group,name:rule_group_production_operation_tracking"
-msgid "Rule Group Production Operation Tracking"
-msgstr "Grupo de reglas de registro de operación de producción"
+msgid "User in companies"
+msgstr "Usuario en las empresas"
 
 msgctxt "model:ir.ui.menu,name:"
 msgid "Operation Types"
diff --git a/operation.xml b/operation.xml
index 8b9c0a6..411bee4 100644
--- a/operation.xml
+++ b/operation.xml
@@ -84,13 +84,13 @@
         </record>
 
         <record model="ir.rule.group" id="rule_group_production_operation">
-            <field name="name">Rule Group Product Operation</field>
+            <field name="name">User in companies</field>
             <field name="model" search="[('model', '=', 'production.operation')]"/>
             <field name="global_p" eval="True"/>
         </record>
         <record model="ir.rule" id="rule_production_operation">
             <field name="domain"
-                eval="[('company', '=', Eval('user', {}).get('company', None))]"
+                eval="[('company', 'in', Eval('companies', []))]"
                 pyson="1"/>
             <field name="rule_group" ref="rule_group_production_operation"/>
         </record>
@@ -177,13 +177,13 @@
         </record>
 
         <record model="ir.rule.group" id="rule_group_production_operation_tracking">
-            <field name="name">Rule Group Production Operation Tracking</field>
+            <field name="name">User in companies</field>
             <field name="model" search="[('model', '=', 'production.operation.tracking')]"/>
             <field name="global_p" eval="True"/>
         </record>
         <record model="ir.rule" id="rule_production_operation_tracking">
             <field name="domain"
-                eval="[('company', '=', Eval('user', {}).get('company', None))]"
+                eval="[('company', 'in', Eval('companies', []))]"
                 pyson="1"/>
             <field name="rule_group" ref="rule_group_production_operation_tracking"/>
         </record>
diff --git a/tests/test_production_operation.py b/tests/test_production_operation.py
index e56b439..0b9ca57 100644
--- a/tests/test_production_operation.py
+++ b/tests/test_production_operation.py
@@ -7,9 +7,10 @@ import trytond.tests.test_tryton
 from trytond.tests.test_tryton import ModuleTestCase
 from trytond.tests.test_tryton import doctest_teardown
 from trytond.tests.test_tryton import doctest_checker
+from trytond.modules.company.tests import CompanyTestMixin
 
 
-class ProductionOperationTestCase(ModuleTestCase):
+class ProductionOperationTestCase(CompanyTestMixin, ModuleTestCase):
     'Test Production Operatoon module'
     module = 'production_operation'