Code review comment for lp:~camptocamp/openerp-product-attributes/7.0-fix-field-acl-lep

Leonardo Pistone (lepistone) wrote :

Yannick, I agree with your argument on sale managers.

Still, I was trying to make the smallest fix. Sale Managers already have permissions to change attribute.attribute, and attribute.set. But because attribute.attribute _inherits ir.model.field, they actually can't. There is an inconsistency.

I know security based on views is not security, but still, sales manager don't see the "fields" menu.

What do you suggest?

« Back to merge proposal