Review: Needs Fixing
Olivier,

thanks for your reply. A perfectly agree with you when you say

> So if you make sure that the rules of your database only permit seeing
> the sequences of the current user company, 

i.e. i'd be very happy to set up a rule on ir.sequence for my needs.

The problem is that in that the l.9 of the diff says

company_ids = self.pool.get('res.company').search(cr, uid, [], context=context)

that uses the record rules of the company object, not the ir.sequence object. I 
think that is wrong.

My proposal is to forget all SQL here and just do a simple 
self.pool.get('ir.sequence').search

That would follow rules for ir.sequence, and everyone would be able to set them 
up according to their needs.

Thanks!
-- 
https://code.launchpad.net/~openerp-dev/openobject-server/6.0-opw-10420-ira/+merge/66546
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-server/6.0-opw-10420-ira.

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to