Merge lp:~openerp-dev/openobject-server/6.0-opw-583074-pna into lp:openobject-server/6.0

Proposed by Pinakin Nayi (OpenERP)
Status: Rejected
Rejected by: Olivier Dony (Odoo)
Proposed branch: lp:~openerp-dev/openobject-server/6.0-opw-583074-pna
Merge into: lp:openobject-server/6.0
Diff against target: 12 lines (+1/-1)
1 file modified
bin/addons/base/security/base_security.xml (+1/-1)
To merge this branch: bzr merge lp:~openerp-dev/openobject-server/6.0-opw-583074-pna
Reviewer Review Type Date Requested Status
Olivier Dony (Odoo) Disapprove
Naresh(OpenERP) Pending
Review via email: mp+140618@code.launchpad.net

Description of the change

Hello Sir,

I solved bug of Multi-comapnies incorrect access right in view of partner list.
Steps to reproduce :
https://bugs.launchpad.net/openobject-addons/+bug/1090049

Thanks,
pna

To post a comment you must log in.
Revision history for this message
Olivier Dony (Odoo) (odo-openerp) wrote :

The bug is invalid, and changing the default access rule for res.partner is not a good idea, please see comment #5 on bug 1090049.

review: Disapprove

Unmerged revisions

3651. By Pinakin Nayi (OpenERP)

[FIX]res_partner : multi company

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'bin/addons/base/security/base_security.xml'
2--- bin/addons/base/security/base_security.xml 2011-03-03 15:59:18 +0000
3+++ bin/addons/base/security/base_security.xml 2012-12-19 10:26:24 +0000
4@@ -53,7 +53,7 @@
5 <field name="global" eval="True"/>
6 <!-- Show partners from ancestors and descendants companies (or company-less), this is usually a better
7 default for multicompany setups. -->
8- <field name="domain_force">['|','|',('company_id.child_ids','child_of',[user.company_id.id]),('company_id','child_of',[user.company_id.id]),('company_id','=',False)]</field>
9+ <field name="domain_force">['|','|',('company_id.child_ids','child_of',[user.company_id.id]),('company_id','in',[c.id for c in user.company_ids]),('company_id','=',False)]</field>
10 </record>
11
12 <record model="ir.rule" id="multi_company_default_rule">