← Back to team overview

credativ team mailing list archive

[Bug 901357] [NEW] ir.property multi company

 

Public bug reported:

Hello,

I'm having issues with ir.property in a multi-company setup. 
I have two companies with company B a child of company A.

I'm confused with the record rule that must be on ir.property:

In trunk it is now
['|',('company_id','child_of',[user.company_id.id]),('company_id','=',False)]
since revision 3634, to fix bug lp:812843

However, that version triggers bug lp:691541 again. That bug was fixed by setting the rule to:
['|',('company_id','=',user.company_id.id),('company_id','=',False)]
in revision 3145.

So it seems we can't have bug 812843 and 691541 fixed at the same time.

My feeling is that the latest version (ie with child_of) is correct, and
that bug 691541 should be fixed by exploiting 'force_company' in
property._fnct_read, like it is done in ir_property._get_domain.

What do you think? If you think it's the right approach, I can provide a
patch.

Best regards,

-sbi

** Affects: openobject-server
     Importance: Undecided
         Status: New

-- 
You received this bug notification because you are a member of OpenERP
Framework Experts, which is subscribed to OpenERP Server.
https://bugs.launchpad.net/bugs/901357

Title:
  ir.property multi company

Status in OpenERP Server:
  New

Bug description:
  Hello,

  I'm having issues with ir.property in a multi-company setup. 
  I have two companies with company B a child of company A.

  I'm confused with the record rule that must be on ir.property:

  In trunk it is now
  ['|',('company_id','child_of',[user.company_id.id]),('company_id','=',False)]
  since revision 3634, to fix bug lp:812843

  However, that version triggers bug lp:691541 again. That bug was fixed by setting the rule to:
  ['|',('company_id','=',user.company_id.id),('company_id','=',False)]
  in revision 3145.

  So it seems we can't have bug 812843 and 691541 fixed at the same
  time.

  My feeling is that the latest version (ie with child_of) is correct,
  and that bug 691541 should be fixed by exploiting 'force_company' in
  property._fnct_read, like it is done in ir_property._get_domain.

  What do you think? If you think it's the right approach, I can provide
  a patch.

  Best regards,

  -sbi

To manage notifications about this bug go to:
https://bugs.launchpad.net/openobject-server/+bug/901357/+subscriptions


Follow ups

References