diff options
| author | Arnold Moyaux <arm@odoo.com> | 2019-10-01 12:58:03 +0000 |
|---|---|---|
| committer | Arnold Moyaux <arm@odoo.com> | 2019-10-01 13:42:50 +0000 |
| commit | d86c2c588367dee319649ab2a5890975c1bcb916 (patch) | |
| tree | fc84c1f7e068dc5ffd7c5a900a959c229c3f11e4 /odoo/addons/base/security | |
| parent | da2b24714616bfbaf3ed7834844618207ec6ca2c (diff) | |
[FIX] base: res_users in multi company
Usecase to reproduce:
- User1 in company A and company B (default company A)
- User2 in company A and company B (default company A)
As user1 select company B in company switcher.
-> You never see user2 as he's hidden by a record rule.
This issue was present in older version when disabling the “Common Contact Book” feature,
because it was then enabling the faulty record rule. Since [1], this faulty record rule is
always enabled.
(Note that res.partner record rules are applied to res.user through _inherits)
We chose to remove the faulty record rules so that the present usecase is now working,
but the drawback is that you now see partners from other companies.
Also remove [2] that try to fix the issue and become useless if the
rule is removed.
[1] commit 25714692b23624d04a43862db26ebcf802b89399
[2] commit d6c65aa4d48ae2bf10830e04a3fb043b06816277
closes odoo/odoo#37712
Signed-off-by: Arnold Moyaux <amoyaux@users.noreply.github.com>
Diffstat (limited to 'odoo/addons/base/security')
| -rw-r--r-- | odoo/addons/base/security/base_security.xml | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/odoo/addons/base/security/base_security.xml b/odoo/addons/base/security/base_security.xml index 2d1e5289ce7..e9288666e57 100644 --- a/odoo/addons/base/security/base_security.xml +++ b/odoo/addons/base/security/base_security.xml @@ -12,12 +12,6 @@ <field name="perm_read" eval="False"/> </record> - <record model="ir.rule" id="res_partner_rule"> - <field name="name">res.partner company</field> - <field name="model_id" ref="base.model_res_partner"/> - <field name="domain_force">['|', '|', ('id', '=', user.partner_id.id), ('company_id', 'in', company_ids), ('company_id', '=', False)]</field> - </record> - <record model="ir.rule" id="res_partner_portal_public_rule"> <field name="name">res_partner: portal/public: read access on my commercial partner</field> <field name="model_id" ref="base.model_res_partner"/> @@ -167,7 +161,7 @@ <field name="name">user rule</field> <field name="model_id" ref="model_res_users"/> <field eval="True" name="global"/> - <field name="domain_force">['|', ('id', '=', user.id), ('company_ids', 'in', company_ids)]</field> + <field name="domain_force">[('company_ids', 'in', company_ids)]</field> </record> </data> </odoo> |